Change exported install name to beman::execution26 (#109)

* Change exported install name to beman::execution26

* adjust the header update script and add missing headers

---------

Co-authored-by: Dietmar Kühl <[email protected]>

Author: maikel

CMakeLists.txt

@@ -12,10 +12,10 @@ if(${CMAKE_SOURCE_DIR} STREQUAL ${CMAKE_BINARY_DIR})
 endif()
 
 set(TARGET_NAME execution26)
-set(TARGET_NAMESPACE beman) # FIXME: not used in install(EXPORT ...) CK?
+set(TARGET_NAMESPACE beman)
 set(TARGET_PREFIX ${TARGET_NAMESPACE}.${TARGET_NAME})
 set(TARGET_LIBRARY ${PROJECT_NAME})
-set(TARGET_ALIAS ${TARGET_LIBRARY}::${TARGET_LIBRARY})
+set(TARGET_ALIAS ${TARGET_NAMESPACE}::${TARGET_NAME})
 set(TARGET_PACKAGE_NAME ${PROJECT_NAME}-config)
 set(TARGETS_EXPORT_NAME ${PROJECT_NAME}-targets)
 
@@ -40,7 +40,7 @@ if(CMAKE_BUILD_TYPE STREQUAL Debug)
   # uncomment to enable the options. Some of them accept one or more inputs:
   project_options(
     PREFIX
-    ${PROJECT_NAME}
+    ${TARGET_NAME}
     ENABLE_CACHE
     ENABLE_CLANG_TIDY
     # NO! ENABLE_VS_ANALYSIS

bin/update-cmake-headers.py

@@ -29,7 +29,7 @@ def get_headers(dir):
 }
 
 file_set_re = re.compile(" *FILE_SET.*")
-section_re = re.compile(" *\${TARGET_LIBRARY}_(?P<section>.*)_headers$")
+section_re = re.compile(" *\${TARGET_NAME}_(?P<section>.*)_headers$")
 header_re = re.compile(" *\${PROJECT_SOURCE_DIR}/include/beman/.*/.*\.hpp")
 
 if len(sys.argv) != 2:

examples/CMakeLists.txt

@@ -21,5 +21,5 @@ foreach(EXAMPLE ${EXAMPLES})
   set(EXAMPLE_TARGET ${TARGET_PREFIX}.examples.${EXAMPLE})
   add_executable(${EXAMPLE_TARGET})
   target_sources(${EXAMPLE_TARGET} PRIVATE ${EXAMPLE}.cpp)
-  target_link_libraries(${EXAMPLE_TARGET} PRIVATE ${TARGET_LIBRARY})
+  target_link_libraries(${EXAMPLE_TARGET} PRIVATE ${TARGET_NAMESPACE}::${TARGET_NAME})
 endforeach()

src/beman/execution26/CMakeLists.txt

@@ -3,22 +3,22 @@
 # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 # cmake-format: on
 
-add_library(${TARGET_LIBRARY} STATIC)
-add_library(${TARGET_ALIAS} ALIAS ${TARGET_LIBRARY})
+add_library(${TARGET_NAME} STATIC)
+add_library(${TARGET_ALIAS} ALIAS ${TARGET_NAME})
 
 if(CMAKE_BUILD_TYPE STREQUAL Debug)
-  target_link_libraries(${TARGET_LIBRARY} PUBLIC $<BUILD_INTERFACE:${TARGET_LIBRARY}_project_options>)
-  target_link_libraries(${TARGET_LIBRARY} PUBLIC $<BUILD_INTERFACE:${TARGET_LIBRARY}_project_warnings>)
+  target_link_libraries(${TARGET_NAME} PUBLIC $<BUILD_INTERFACE:${TARGET_NAME}_project_options>)
+  target_link_libraries(${TARGET_NAME} PUBLIC $<BUILD_INTERFACE:${TARGET_NAME}_project_warnings>)
 endif()
 
 include(CMakePrintHelpers)
-cmake_print_variables(TARGET_ALIAS TARGET_LIBRARY TARGET_PREFIX PROJECT_SOURCE_DIR)
+cmake_print_variables(TARGET_ALIAS TARGET_NAME TARGET_PREFIX PROJECT_SOURCE_DIR)
 
 target_sources(
-  ${TARGET_LIBRARY}
+  ${TARGET_NAME}
   PRIVATE execution.cpp
   PUBLIC FILE_SET
-         ${TARGET_LIBRARY}_public_headers
+         ${TARGET_NAME}_public_headers
          TYPE
          HEADERS
          BASE_DIRS
@@ -29,7 +29,7 @@ target_sources(
          ${PROJECT_SOURCE_DIR}/include/beman/execution26/functional.hpp
          ${PROJECT_SOURCE_DIR}/include/beman/execution26/stop_token.hpp
   PUBLIC FILE_SET
-         ${TARGET_LIBRARY}_detail_headers
+         ${TARGET_NAME}_detail_headers
          TYPE
          HEADERS
          BASE_DIRS
@@ -198,28 +198,28 @@ target_sources(
 )
 
 # cmake-format: off
-get_property(DETAIL_HEADER_FILES TARGET ${TARGET_LIBRARY} PROPERTY HEADER_SET_${TARGET_LIBRARY}_detail_headers)
+get_property(DETAIL_HEADER_FILES TARGET ${TARGET_NAME} PROPERTY HEADER_SET_${TARGET_NAME}_detail_headers)
 source_group("Header Files\\detail" FILES ${DETAIL_HEADER_FILES})
 
-set_target_properties(${TARGET_LIBRARY} PROPERTIES VERIFY_INTERFACE_HEADER_SETS ON)
+set_target_properties(${TARGET_NAME} PROPERTIES VERIFY_INTERFACE_HEADER_SETS ON)
 
-target_compile_features(${TARGET_LIBRARY} PUBLIC
+target_compile_features(${TARGET_NAME} PUBLIC
   "$<$<COMPILE_FEATURES:cxx_std_26>:cxx_std_26>"
   "$<$<NOT:$<COMPILE_FEATURES:cxx_std_26>>:cxx_std_23>"
 )
 
 install(
-  TARGETS ${TARGET_LIBRARY}
+  TARGETS ${TARGET_NAME}
   EXPORT ${TARGETS_EXPORT_NAME}1
   ARCHIVE DESTINATION lib/$<CONFIG>
-  FILE_SET ${TARGET_LIBRARY}_public_headers
-  FILE_SET ${TARGET_LIBRARY}_detail_headers
+  FILE_SET ${TARGET_NAME}_public_headers
+  FILE_SET ${TARGET_NAME}_detail_headers
 )
 # cmake-format: on
 
 install(
   EXPORT ${TARGETS_EXPORT_NAME}1
   FILE ${TARGETS_EXPORT_NAME}.cmake
   DESTINATION "${INSTALL_CONFIGDIR}"
-  NAMESPACE ${TARGET_LIBRARY}::
+  NAMESPACE ${TARGET_NAMESPACE}::
 )

tests/beman/execution26/CMakeLists.txt

@@ -131,7 +131,7 @@ remove_definitions(-DNDEBUG) # NOTE: we want ASSERT statements in Release too! C
 foreach(test ${execution_tests})
   set(TEST_EXE ${TARGET_PREFIX}.${test})
   add_executable(${TEST_EXE} ${test}.cpp)
-  target_link_libraries(${TEST_EXE} PRIVATE beman_execution26::beman_execution26)
+  target_link_libraries(${TEST_EXE} PRIVATE beman::execution26)
   add_test(NAME ${TEST_EXE} COMMAND $<TARGET_FILE:${TEST_EXE}>)
 endforeach()