fable: Simplify CMakeLists.txt to not use ${target} variable

Author: cassava

fable/CMakeLists.txt

@@ -10,15 +10,12 @@ set(FABLE_VERSION_U32 0 CACHE STRING "Fable version as (MAJOR<<16)|(MINOR<<8)|PA
 include(GNUInstallDirs)
 
 # Library ------------------------------------------------------------
-set(namespace fable)
-set(target fable)
-
 find_package(Boost COMPONENTS headers filesystem REQUIRED)
 find_package(fmt REQUIRED)
 find_package(nlohmann_json REQUIRED)
 
-file(GLOB ${target}_PUBLIC_HEADERS "include/**/*.hpp")
-add_library(${target}
+file(GLOB fable_PUBLIC_HEADERS "include/**/*.hpp")
+add_library(fable
     # find src -type f -name "*.cpp" \! -name "*_test.cpp"
     src/fable/conf.cpp
     src/fable/confable.cpp
@@ -34,22 +31,22 @@ add_library(${target}
     src/fable/utility.cpp
 
     # For IDE integration
-    ${${target}_PUBLIC_HEADERS}
+    ${fable_PUBLIC_HEADERS}
 )
-add_library(${namespace}::${target} ALIAS ${target})
-set_target_properties(${target} PROPERTIES
+add_library(fable::fable ALIAS fable)
+set_target_properties(fable PROPERTIES
     CXX_STANDARD 17
     CXX_STANDARD_REQUIRED ON
     VERSION ${FABLE_VERSION}
 )
 configure_file(src/fable/version.hpp.in include/fable/version.hpp @ONLY)
-target_include_directories(${target}
+target_include_directories(fable
   PUBLIC
     "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>"
     "$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/include>"
     "$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>"
 )
-target_link_libraries(${target}
+target_link_libraries(fable
   PUBLIC
     Boost::headers
     Boost::filesystem
@@ -59,9 +56,9 @@ target_link_libraries(${target}
 
 option(FABLE_ALLOW_COMMENTS "Allow comments when parsing JSON?" ON)
 if(FABLE_ALLOW_COMMENTS)
-    target_compile_definitions(${target} PRIVATE PARSE_JSON_WITH_COMMENTS=true)
+    target_compile_definitions(fable PRIVATE PARSE_JSON_WITH_COMMENTS=true)
 else()
-    target_compile_definitions(${target} PRIVATE PARSE_JSON_WITH_COMMENTS=false)
+    target_compile_definitions(fable PRIVATE PARSE_JSON_WITH_COMMENTS=false)
 endif()
 
 # Testing ------------------------------------------------------------
@@ -95,15 +92,16 @@ if(BUILD_TESTING)
     target_link_libraries(test-fable
         GTest::gtest
         GTest::gtest_main
-        Boost::boost
-        ${target}
+        Boost::headers
+        Boost::filesystem
+        fable
     )
     gtest_add_tests(TARGET test-fable)
 endif()
 
 # Installation -------------------------------------------------------
-install(TARGETS ${target}
-    EXPORT ${namespace}Targets
+install(TARGETS fable
+    EXPORT fableTargets
     LIBRARY
     ARCHIVE
     RUNTIME
@@ -114,18 +112,18 @@ install(DIRECTORY include/
 install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/include/
     DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}
 )
-install(EXPORT ${namespace}Targets
-    NAMESPACE ${namespace}::
-    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${namespace}
+install(EXPORT fableTargets
+    NAMESPACE fable::
+    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/fable
 )
 include(CMakePackageConfigHelpers)
 write_basic_package_version_file(
-    "${namespace}ConfigVersion.cmake"
+    "fableConfigVersion.cmake"
     VERSION ${FABLE_VERSION}
     COMPATIBILITY AnyNewerVersion
 )
 install(FILES
-    "cmake/${namespace}Config.cmake"
-    "${CMAKE_CURRENT_BINARY_DIR}/${namespace}ConfigVersion.cmake"
-    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${namespace}
+    "cmake/fableConfig.cmake"
+    "${CMAKE_CURRENT_BINARY_DIR}/fableConfigVersion.cmake"
+    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/fable
 )