Output PDB files for Windows builds, and stop installing googletest (#18)

* Output PDB files for Debug and RelWithDebInfo configurations

* Stop installing googletest

CMakeLists.txt

@@ -4,9 +4,10 @@
 cmake_minimum_required (VERSION 3.11)
 
 project(AccessorFramework
-    VERSION 1.0
+    VERSION 1.0.0
     DESCRIPTION "A framework for using Accessors"
     LANGUAGES CXX
+    HOMEPAGE_URL "https://github.com/microsoft/AccessorFramework"
 )
 
 set(CMAKE_CXX_STANDARD 14)
@@ -15,6 +16,10 @@ set(CMAKE_CXX_EXTENSIONS OFF)
 
 option(BUILD_TESTS "Build test executable (on by default)" ON)
 
+if(NOT DEFINED CMAKE_DEBUG_POSTFIX)
+  set(CMAKE_DEBUG_POSTFIX "d")
+endif()
+
 add_library(AccessorFramework
     ${PROJECT_SOURCE_DIR}/src/Accessor.cpp
     ${PROJECT_SOURCE_DIR}/src/AccessorImpl.cpp
@@ -28,6 +33,7 @@ add_library(AccessorFramework
 )
 
 add_library(AccessorFramework::AccessorFramework ALIAS AccessorFramework)
+
 if(${VERBOSE})
   target_compile_definitions(AccessorFramework PRIVATE VERBOSE=${VERBOSE})
 endif()
@@ -47,38 +53,61 @@ endif()
 
 set_target_properties(AccessorFramework PROPERTIES
     ARCHIVE_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib
+    COMPILE_PDB_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/bin
     LIBRARY_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib
+    PDB_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/bin
+    RUNTIME_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/bin
+)
+
+get_target_property(AccessorFramework_NAME AccessorFramework NAME)
+get_target_property(AccessorFramework_DEBUG_POSTFIX AccessorFramework DEBUG_POSTFIX)
+set_target_properties(AccessorFramework PROPERTIES
+    COMPILE_PDB_NAME ${AccessorFramework_NAME}
+    COMPILE_PDB_NAME_DEBUG ${AccessorFramework_NAME}${AccessorFramework_DEBUG_POSTFIX}
+    PDB_NAME ${AccessorFramework_NAME}
+    PDB_NAME_DEBUG ${AccessorFramework_NAME}${AccessorFramework_DEBUG_POSTFIX}
 )
 
 # Install
 
 install(TARGETS AccessorFramework
-  CONFIGURATIONS Debug
     EXPORT AccessorFrameworkTargets
-  ARCHIVE DESTINATION ${CMAKE_INSTALL_PREFIX}/lib
-  LIBRARY DESTINATION ${CMAKE_INSTALL_PREFIX}/lib
-  RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX}/bin
-  COMPONENT library
+    ARCHIVE DESTINATION lib
+    LIBRARY DESTINATION lib
+    RUNTIME DESTINATION bin
 )
 
-install(DIRECTORY ${PROJECT_SOURCE_DIR}/include/AccessorFramework
-  DESTINATION ${CMAKE_INSTALL_PREFIX}/include
+install(DIRECTORY ${PROJECT_SOURCE_DIR}/include/AccessorFramework DESTINATION ${CMAKE_INSTALL_PREFIX}/include)
+
+if(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
+    get_target_property(AccessorFramework_PDB_OUTPUT_DIRECTORY AccessorFramework PDB_OUTPUT_DIRECTORY)
+    get_target_property(AccessorFramework_PDB_NAME AccessorFramework PDB_NAME)
+    get_target_property(AccessorFramework_PDB_NAME_DEBUG AccessorFramework PDB_NAME_DEBUG)
+    install(FILES
+        "${AccessorFramework_PDB_OUTPUT_DIRECTORY}/${AccessorFramework_PDB_NAME_DEBUG}.pdb"
+        "${AccessorFramework_PDB_OUTPUT_DIRECTORY}/${AccessorFramework_PDB_NAME}.pdb"
+        DESTINATION bin
+        OPTIONAL
     )
-
-# Package
-
-if(WIN32 AND NOT CYGWIN)
-  set(INSTALL_CONFIGDIR ${CMAKE_INSTALL_PREFIX}/cmake)
-else()
-  set(INSTALL_CONFIGDIR ${CMAKE_INSTALL_PREFIX}/lib/cmake/AccessorFramework)
 endif()
 
 install(EXPORT AccessorFrameworkTargets
     FILE AccessorFrameworkTargets.cmake
     NAMESPACE AccessorFramework::
-  DESTINATION ${INSTALL_CONFIGDIR}
+    DESTINATION lib/cmake/AccessorFramework
 )
 
+# Export
+
+export(EXPORT AccessorFrameworkTargets
+  FILE ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkTargets.cmake
+  NAMESPACE AccessorFramework::
+)
+
+export(PACKAGE AccessorFramework)
+
+# Package
+
 include(CMakePackageConfigHelpers)
 write_basic_package_version_file(
     ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkConfigVersion.cmake
@@ -89,22 +118,17 @@ write_basic_package_version_file(
 configure_package_config_file(
     ${PROJECT_SOURCE_DIR}/cmake/AccessorFrameworkConfig.cmake.in
     ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkConfig.cmake
-  INSTALL_DESTINATION ${INSTALL_CONFIGDIR}
+    INSTALL_DESTINATION lib/cmake/AccessorFramework
 )
 
 install(
     FILES
     ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkConfig.cmake
     ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkConfigVersion.cmake
-  DESTINATION ${INSTALL_CONFIGDIR}
+    DESTINATION lib/cmake/AccessorFramework
 )
 
-export(EXPORT AccessorFrameworkTargets
-  FILE ${PROJECT_BINARY_DIR}/cmake/AccessorFrameworkTargets.cmake
-  NAMESPACE AccessorFramework::
-)
-
-export(PACKAGE AccessorFramework)
+# Tests
 
 if (BUILD_TESTS)
     add_definitions(-DUSE_GTEST)

test/CMakeLists.txt

@@ -3,6 +3,10 @@
 
 cmake_minimum_required (VERSION 3.11)
 
+# Replace install() to do-nothing macro to avoid installing googletest
+macro(install)
+endmacro()
+
 include(FetchContent)
 FetchContent_Declare(
     googletest
@@ -25,9 +29,15 @@ add_executable(AccessorFrameworkTests
 )
 
 target_link_libraries(AccessorFrameworkTests
+    PRIVATE
     gtest
     gmock_main
     AccessorFramework::AccessorFramework
 )
 
 add_test(NAME AccessorFrameworkTests COMMAND AccessorFrameworkTests)
+
+# Restore original install() behavior
+macro(install)
+    _install(${ARGN})
+endmacro()