Address rabbit comments; Improve style consistency.

Author: davidlion

CMakeLists.txt

@@ -38,7 +38,7 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS
     FORCE
 )
 
-set(YSTDLIB_LIBRARIES
+set(ystdlib_LIBRARIES
     "containers"
     "error_handling"
     "io_interface"
@@ -48,8 +48,8 @@ set(YSTDLIB_LIBRARIES
 )
 
 message(STATUS "Building the following libraries:")
-foreach(lib IN LISTS YSTDLIB_LIBRARIES)
-    message(" - ${lib}")
+foreach(LIB IN LISTS ystdlib_LIBRARIES)
+    message(" - ${LIB}")
 endforeach()
 
 if(ystdlib_IS_TOP_LEVEL)

README.md

@@ -72,12 +72,12 @@ cmake -S . -B ./build
 cmake --build ./build
 ```
 
-To build a subset of libraries, set the variable `YSTDLIB_LIBRARIES` to a semicolon(`;`) separated
+To build a subset of libraries, set the variable `ystdlib_LIBRARIES` to a semicolon(`;`) separated
 list of library names. The library names match their [directory name in src/](./src/ystdlib).
 For example:
 
 ```shell
-cmake -S . -B ./build -DYSTDLIB_LIBRARIES="containers;io_interface"
+cmake -S . -B ./build -Dystdlib_LIBRARIES="containers;io_interface"
 cmake --build ./build
 ```
 

cmake/ystdlib-config.cmake.in

@@ -7,19 +7,19 @@ include(CMakeFindDependencyMacro)
 # If no component is specified we require all libraries.
 # We guard against repeatedly including targets to support multiple find_package calls.
 if(NOT ystdlib_FIND_COMPONENTS)
-    set(YSTDLIB_LIBRARIES_LIST "@YSTDLIB_LIBRARIES@")
-    foreach(lib IN LISTS YSTDLIB_LIBRARIES_LIST)
-        if(NOT TARGET ystdlib::${lib})
-            include("${CMAKE_CURRENT_LIST_DIR}/libs/${lib}-config.cmake")
+    set(ystdlib_LIBRARIES_LIST "@ystdlib_LIBRARIES@")
+    foreach(LIB IN LISTS ystdlib_LIBRARIES_LIST)
+        if(NOT TARGET ystdlib::${LIB})
+            include("${CMAKE_CURRENT_LIST_DIR}/libs/${LIB}-config.cmake")
         endif()
     endforeach()
 else()
-    foreach(lib IN LISTS ystdlib_FIND_COMPONENTS)
-        if(NOT TARGET "ystdlib::${lib}")
-            if(ystdlib_FIND_REQUIRED_${lib})
-                include("${CMAKE_CURRENT_LIST_DIR}/libs/${lib}-config.cmake")
+    foreach(LIB IN LISTS ystdlib_FIND_COMPONENTS)
+        if(NOT TARGET "ystdlib::${LIB}")
+            if(ystdlib_FIND_REQUIRED_${LIB})
+                include("${CMAKE_CURRENT_LIST_DIR}/libs/${LIB}-config.cmake")
             else()
-                include("${CMAKE_CURRENT_LIST_DIR}/libs/${lib}-config.cmake" OPTIONAL)
+                include("${CMAKE_CURRENT_LIST_DIR}/libs/${LIB}-config.cmake" OPTIONAL)
             endif()
         endif()
     endforeach()

src/ystdlib/CMakeLists.txt

@@ -1,7 +1,7 @@
-if(NOT YSTDLIB_LIBRARIES)
-    message(FATAL_ERROR "YSTDLIB_LIBRARIES must be defined and non-empty.")
+if(NOT DEFINED ystdlib_LIBRARIES OR ystdlib_LIBRARIES STREQUAL "")
+    message(FATAL_ERROR "ystdlib_LIBRARIES must be defined and non-empty.")
 endif()
 
-foreach(lib IN LISTS YSTDLIB_LIBRARIES)
-    add_subdirectory("${lib}")
+foreach(LIB IN LISTS ystdlib_LIBRARIES)
+    add_subdirectory("${LIB}")
 endforeach()

src/ystdlib/io_interface/CMakeLists.txt

@@ -2,7 +2,7 @@ if(TARGET ystdlib::io_interface)
     return()
 endif()
 
-if(NOT wrapped_facade_headers IN_LIST YSTDLIB_LIBRARIES)
+if(NOT wrapped_facade_headers IN_LIST ystdlib_LIBRARIES)
     add_subdirectory(
         "${CMAKE_CURRENT_LIST_DIR}/../wrapped_facade_headers"
         "${CMAKE_CURRENT_BINARY_DIR}/implicit_dep/wrapped_facade_headers"