From 51cf2951d0b27c0ca653c02c0bb433c7cc30d002 Mon Sep 17 00:00:00 2001 From: Ben Clayton Date: Tue, 7 Sep 2021 11:23:36 +0100 Subject: [PATCH] CMake: Remove installation target CMake installation target has been broken for years. cppdap is only internally used and tested as a library that is included as a sub-module of another project. In order for a package to be installable, it really needs a stable ABI / API, which we've made no explicit guarantees about. cppdap is small enough that it can easily be included in a third_party directory of another project. Remove the broken, unmaintained installation rules. Fixes: #54 --- CMakeLists.txt | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 47cb5df..9db4ebf 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -36,7 +36,6 @@ option_if_not_defined(CPPDAP_ASAN "Build dap with address sanitizer" OFF) option_if_not_defined(CPPDAP_MSAN "Build dap with memory sanitizer" OFF) option_if_not_defined(CPPDAP_TSAN "Build dap with thread sanitizer" OFF) option_if_not_defined(CPPDAP_INSTALL_VSCODE_EXAMPLES "Build and install dap examples into vscode extensions directory" OFF) -option_if_not_defined(CPPDAP_INSTALL "Create dap install target" OFF) ########################################################### # Directories @@ -184,30 +183,6 @@ cppdap_set_target_options(cppdap) target_link_libraries(cppdap "${CPPDAP_OS_LIBS}") -# install -if(CPPDAP_INSTALL) - include(GNUInstallDirs) - - install(DIRECTORY ${CPPDAP_INCLUDE_DIR}/cppdap - DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} - USE_SOURCE_PERMISSIONS - ) - - install(TARGETS cppdap - EXPORT cppdap-targets - ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} - LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} - RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} - INCLUDES DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} - ) - - install(EXPORT cppdap-targets - FILE cppdap-config.cmake - NAMESPACE cppdap:: - DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/cppdap - ) -endif(CPPDAP_INSTALL) - # tests if(CPPDAP_BUILD_TESTS) set(DAP_TEST_LIST