Merge pull request #226 from robertmaynard/improve_install_rules

Correct issues when `INSTALL_CUCO` is set to OFF

Author: PointKernel

CMakeLists.txt

@@ -111,12 +111,9 @@ endif(BUILD_EXAMPLES)
 ###################################################################################################
 # - Install targets -------------------------------------------------------------------------------
 
-if(INSTALL_CUCO)
-    install(TARGETS cuco EXPORT cuco-exports)
-    install(DIRECTORY include/cuco/ DESTINATION include/cuco)
-    install(FILES ${CUCO_BINARY_DIR}/include/cuco/version_config.hpp DESTINATION include/cuco)
+install(TARGETS cuco EXPORT cuco-exports)
 
-    set(doc_string
+set(doc_string
     [=[
 Provide targets for cuCollections.
 
@@ -131,27 +128,30 @@ structures tailored for efficient use with GPUs.
 
 ]=])
 
-    # build install targets
-    rapids_export(
-        INSTALL cuco
-        EXPORT_SET cuco-exports
-        GLOBAL_TARGETS cuco
-        NAMESPACE cuco::
-        DOCUMENTATION doc_string)
-
-    set(code_string
-    [=[
+set(code_string
+[=[
 if(NOT TARGET cuco::Thrust)
-    thrust_create_target(cuco::Thrust FROM_OPTIONS)
+thrust_create_target(cuco::Thrust FROM_OPTIONS)
 endif()
 ]=])
 
-    # build export targets
+# build directory cuco-config generation
+rapids_export(
+    BUILD cuco
+    EXPORT_SET cuco-exports
+    GLOBAL_TARGETS cuco
+    NAMESPACE cuco::
+    DOCUMENTATION doc_string
+    FINAL_CODE_BLOCK code_string)
+
+if(INSTALL_CUCO)
+    install(DIRECTORY include/cuco/ DESTINATION include/cuco)
+    install(FILES ${CUCO_BINARY_DIR}/include/cuco/version_config.hpp DESTINATION include/cuco)
+    # install directory cuco-config generation
     rapids_export(
-        BUILD cuco
+        INSTALL cuco
         EXPORT_SET cuco-exports
         GLOBAL_TARGETS cuco
         NAMESPACE cuco::
-        DOCUMENTATION doc_string
-        FINAL_CODE_BLOCK code_string)
+        DOCUMENTATION doc_string)
 endif()

benchmarks/CMakeLists.txt

@@ -25,13 +25,15 @@ CPMAddPackage(
     # Additionally, attempting to set the CMAKE_CXX_VERSION here doesn't propogate to the feature test build
     # Therefore, we just disable the feature test and assume platforms we care about have a regex impl available
     "RUN_HAVE_STD_REGEX 0" #
+    "BENCHMARK_ENABLE_INSTALL OFF"
 )
 
 CPMAddPackage(
   NAME nvbench
   GITHUB_REPOSITORY NVIDIA/nvbench
   GIT_TAG main
   GIT_SHALLOW TRUE
+  EXCLUDE_FROM_ALL YES
 )
 
 ###################################################################################################