chore(cmake): standardize variable naming and packaging configuration

Author: ronan-fdev

README.md

@@ -45,7 +45,7 @@ nfx-datetime is a modern C++20 library providing high-precision temporal operati
 
 ### ⚡ Performance Optimized
 
-- Sub-nanosecond arithmetic operations
+- High-precision arithmetic operations (100-nanosecond resolution)
 - Highly optimized parsing
 - Efficient string formatting
 - Zero-cost abstractions with constexpr support
@@ -174,7 +174,7 @@ The complete API documentation is available online at:
 cmake .. -DCMAKE_BUILD_TYPE=Release -DNFX_DATETIME_BUILD_DOCUMENTATION=ON
 
 # Build the documentation
-cmake --build . --target documentation
+cmake --build . --target nfx-datetime-documentation
 ```
 
 #### Requirements
@@ -495,9 +495,11 @@ See the [CHANGELOG.md](CHANGELOG.md) for a detailed history of changes, new feat
 
 This project is licensed under the MIT License.
 
-## Dependencies
+## Acknowledgments
+
+- Inspired by .NET's DateTime API design
 
-### Runtime Dependencies
+## Dependencies
 
 ### Development Dependencies
 

TODO.md

@@ -1,10 +1,23 @@
 # nfx-datetime TODO
 
-- NIL
+Project roadmap and task tracking for the nfx-datetime library.
 
 ### Todo
 
-- NIL
+- [ ] ZonedDateTime Class: Full IANA timezone database support with automatic DST transitions
+- [ ] Week Calculations: ISO week number, week of year, US week numbering
+- [ ] Custom Format Strings: Strftime-style formatting (e.g., `%Y-%m-%d %H:%M:%S`)
+- [ ] Custom Format Parsing: Parse non-ISO 8601 date strings with format specifiers
+- [ ] Locale-Aware Formatting: Localized month/day names, date formats per locale
+- [ ] Business Day Operations: Add/subtract working days, check if business day
+- [ ] Date Ranges: `DateRange` class with iteration and containment checks
+- [ ] Age Calculation: Calculate years/months/days between dates with proper handling
+- [ ] Stopwatch/Timer: High-precision timing utilities for performance measurement
+- [ ] Astronomy Functions: Sunrise/sunset times, moon phases, solstices/equinoxes
+- [ ] HTTP Date Headers: RFC 7231 date format parsing and formatting
+- [ ] Leap Second Support: TAI time scale, leap second table
+- [ ] Fuzzing Tests: Add fuzzing for parser robustness
+- [ ] SIMD Optimizations: Vectorize date calculations where applicable
 
 ### In Progress
 

benchmark/CMakeLists.txt

@@ -15,9 +15,9 @@ endif()
 # Benchmarks source files
 #----------------------------------------------
 
-set(BENCHMARK_SOURCES)
+set(benchmark_sources)
 
-list(APPEND BENCHMARK_SOURCES
+list(APPEND benchmark_sources
 	BM_DateTime.cpp
 	BM_DateTimeOffset.cpp
 	BM_TimeSpan.cpp
@@ -27,7 +27,7 @@ list(APPEND BENCHMARK_SOURCES
 # Configure benchmark executables
 #----------------------------------------------
 
-foreach(benchmark_source ${BENCHMARK_SOURCES})
+foreach(benchmark_source ${benchmark_sources})
 	get_filename_component(benchmark_target_name ${benchmark_source} NAME_WE)
 	if(NOT TARGET ${benchmark_target_name})
 		add_executable(${benchmark_target_name} ${benchmark_source})

cmake/nfx-datetime-config.cmake.in

@@ -26,8 +26,6 @@ set(NFX_DATETIME_VERSION_PATCH "@PROJECT_VERSION_PATCH@")
 set_and_check(NFX_DATETIME_INCLUDE_DIR "@PACKAGE_CMAKE_INSTALL_INCLUDEDIR@")
 set_and_check(NFX_DATETIME_LIB_DIR "@PACKAGE_CMAKE_INSTALL_LIBDIR@")
 
-# Our library dependencies (none - self-contained library)
-
 # Include the targets file
 include("${CMAKE_CURRENT_LIST_DIR}/nfx-datetime-targets.cmake")
 

cmake/nfxDateTimeDependencies.cmake

@@ -2,9 +2,9 @@
 # nfx-datetime - Dependencies configuration
 #==============================================================================
 
-#----------------------------
+#----------------------------------------------
 # Output configuration
-#----------------------------
+#----------------------------------------------
 
 set(_SAVED_CMAKE_REQUIRED_QUIET     ${CMAKE_REQUIRED_QUIET})
 set(_SAVED_CMAKE_MESSAGE_LOG_LEVEL  ${CMAKE_MESSAGE_LOG_LEVEL})
@@ -83,9 +83,9 @@ if(NFX_DATETIME_BUILD_BENCHMARKS)
 	endif()
 endif()
 
-#----------------------------
+#----------------------------------------------
 # Dependency fetching
-#----------------------------
+#----------------------------------------------
 
 if(NFX_DATETIME_BUILD_TESTS)
 	if(NOT GTest_FOUND)
@@ -99,9 +99,9 @@ if(NFX_DATETIME_BUILD_BENCHMARKS)
 	endif()
 endif()
 
-#----------------------------
+#----------------------------------------------
 # Cleanup
-#----------------------------
+#----------------------------------------------
 
 set(CMAKE_REQUIRED_QUIET ${_SAVED_CMAKE_REQUIRED_QUIET})
 set(CMAKE_MESSAGE_LOG_LEVEL ${_SAVED_CMAKE_MESSAGE_LOG_LEVEL})

cmake/nfxDateTimeInstall.cmake

@@ -39,19 +39,19 @@ install(
 # Install library targets
 #----------------------------------------------
 
-set(INSTALL_TARGETS)
+set(install_targets)
 
 if(NFX_DATETIME_BUILD_SHARED)
-	list(APPEND INSTALL_TARGETS ${PROJECT_NAME})
+	list(APPEND install_targets ${PROJECT_NAME})
 endif()
 
 if(NFX_DATETIME_BUILD_STATIC)
-	list(APPEND INSTALL_TARGETS ${PROJECT_NAME}-static)
+	list(APPEND install_targets ${PROJECT_NAME}-static)
 endif()
 
-if(INSTALL_TARGETS)
+if(install_targets)
 	install(
-		TARGETS ${INSTALL_TARGETS}
+		TARGETS ${install_targets}
 		EXPORT nfx-datetime-targets
 		ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
 			COMPONENT Development
@@ -132,11 +132,15 @@ install(
 	RENAME "LICENSE-${PROJECT_NAME}.txt"
 )
 
-install(
-	DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/licenses/"
-	DESTINATION "${CMAKE_INSTALL_DOCDIR}/licenses"
-	FILES_MATCHING PATTERN "LICENSE-*"
-)
+file(GLOB LICENSE_FILES "${CMAKE_CURRENT_SOURCE_DIR}/licenses/LICENSE-*")
+foreach(LICENSE_FILE ${LICENSE_FILES})
+	get_filename_component(LICENSE_NAME ${LICENSE_FILE} NAME)
+	install(
+		FILES ${LICENSE_FILE}
+		DESTINATION "${CMAKE_INSTALL_DOCDIR}/licenses"
+		RENAME "${LICENSE_NAME}.txt"
+	)
+endforeach()
 
 #----------------------------------------------
 # Install documentation
@@ -169,4 +173,4 @@ if(NFX_DATETIME_BUILD_DOCUMENTATION)
 	endif()
 endif()
 
-message(STATUS "Installation configured for targets: ${INSTALL_TARGETS}")
+message(STATUS "Installation configured for targets: ${install_targets}")

cmake/nfxDateTimePackaging.cmake

@@ -21,9 +21,9 @@ endif()
 
 # --- Detect processor count for parallel packaging ---
 include(ProcessorCount)
-ProcessorCount(N)
-if(NOT N EQUAL 0)
-	set(CPACK_THREADS ${N})
+ProcessorCount(processor_count)
+if(NOT processor_count EQUAL 0)
+	set(CPACK_THREADS ${processor_count})
 else()
 	set(CPACK_THREADS 1)
 endif()
@@ -90,6 +90,12 @@ set(CPACK_PACKAGE_INSTALL_REGISTRY_KEY  "nfx-datetime")
 # Platform-specific generators
 #----------------------------------------------
 
+# --- Find packaging tools ---
+if(UNIX AND NOT APPLE)
+	find_program(DPKG_CMD dpkg)
+	find_program(RPM_CMD rpm)
+endif()
+
 # --- Generators ---
 if(NOT DEFINED CPACK_GENERATOR)
 	set(CPACK_GENERATOR "")
@@ -105,7 +111,6 @@ if(NOT DEFINED CPACK_GENERATOR)
 
 	# --- DEB packages ---
 	if(NFX_DATETIME_PACKAGE_DEB AND UNIX AND NOT APPLE)
-		find_program(DPKG_CMD dpkg)
 		if(DPKG_CMD)
 			set(CPACK_GENERATOR "${CPACK_GENERATOR};DEB")
 		else()
@@ -115,7 +120,6 @@ if(NOT DEFINED CPACK_GENERATOR)
 
 	# --- RPM packages ---
 	if(NFX_DATETIME_PACKAGE_RPM AND UNIX AND NOT APPLE)
-		find_program(RPM_CMD rpm)
 		if(RPM_CMD)
 			set(CPACK_GENERATOR "${CPACK_GENERATOR};RPM")
 		else()
@@ -126,25 +130,15 @@ if(NOT DEFINED CPACK_GENERATOR)
 	# --- WiX packages ---
 	if(NFX_DATETIME_PACKAGE_WIX AND WIN32)
 		if(NOT WIX_CANDLE OR NOT WIX_LIGHT)
-			set(WIX_SEARCH_PATHS
-				"C:/Program Files (x86)/WiX Toolset v3.14/bin"
-				"C:/Program Files (x86)/WiX Toolset v3.11/bin"
-				"C:/Program Files (x86)/WiX Toolset v3.10/bin"
-				"$ENV{WIX}bin"
-			)
-			
-			find_program(WIX_CANDLE NAMES candle.exe candle PATHS ${WIX_SEARCH_PATHS} NO_DEFAULT_PATH)
-			find_program(WIX_LIGHT NAMES light.exe light PATHS ${WIX_SEARCH_PATHS} NO_DEFAULT_PATH)
+			find_program(WIX_CANDLE NAMES candle.exe candle HINTS "$ENV{WIX}/bin")
+			find_program(WIX_LIGHT NAMES light.exe light HINTS "$ENV{WIX}/bin")
 		endif()
 
 		if(WIX_CANDLE AND WIX_LIGHT)
 			set(CPACK_GENERATOR "${CPACK_GENERATOR};WIX")
 			message(STATUS "WiX found: ${WIX_CANDLE} - Windows MSI installer generation enabled")
 		else()
-			message(STATUS "WiX not found - install WiX Toolset v3.x for MSI installer support")
-			if(DEFINED WIX_SEARCH_PATHS)
-				message(STATUS "  Searched paths: ${WIX_SEARCH_PATHS}")
-			endif()
+			message(STATUS "WiX not found - install WiX Toolset and ensure WIX environment variable is set")
 		endif()
 	endif()
 
@@ -162,13 +156,12 @@ message(STATUS "  Output dir: ${CPACK_PACKAGE_DIRECTORY}")
 
 if("DEB" IN_LIST CPACK_GENERATOR AND UNIX AND NOT APPLE)
 	if(NOT DEFINED CPACK_DEBIAN_PACKAGE_ARCHITECTURE)
-		find_program(DPKG_CMD dpkg)
 		if(DPKG_CMD)
 			execute_process(COMMAND ${DPKG_CMD} --print-architecture
-				OUTPUT_VARIABLE DETECTED_DEB_ARCH
+				OUTPUT_VARIABLE detected_deb_arch
 				OUTPUT_STRIP_TRAILING_WHITESPACE
 				ERROR_QUIET)
-			set(CPACK_DEBIAN_PACKAGE_ARCHITECTURE "${DETECTED_DEB_ARCH}")
+			set(CPACK_DEBIAN_PACKAGE_ARCHITECTURE "${detected_deb_arch}")
 		endif()
 	endif()
 
@@ -178,10 +171,10 @@ endif()
 if("RPM" IN_LIST CPACK_GENERATOR AND UNIX AND NOT APPLE)
 	if(NOT DEFINED CPACK_RPM_PACKAGE_ARCHITECTURE)
 		execute_process(COMMAND uname -m
-			OUTPUT_VARIABLE DETECTED_RPM_ARCH
+			OUTPUT_VARIABLE detected_rpm_arch
 			OUTPUT_STRIP_TRAILING_WHITESPACE
 			ERROR_QUIET)
-		set(CPACK_RPM_PACKAGE_ARCHITECTURE "${DETECTED_RPM_ARCH}")
+		set(CPACK_RPM_PACKAGE_ARCHITECTURE "${detected_rpm_arch}")
 	endif()
 
 	message(STATUS "RPM package architecture: ${CPACK_RPM_PACKAGE_ARCHITECTURE}")
@@ -194,7 +187,7 @@ endif()
 # --- DEB package settings ---
 if("DEB" IN_LIST CPACK_GENERATOR AND UNIX AND NOT APPLE)
 	set(CPACK_DEBIAN_PACKAGE_NAME          ${CPACK_PACKAGE_NAME})
-	set(CPACK_DEBIAN_FILE_NAME             DEB-DEFAULT)
+	set(CPACK_DEBIAN_FILE_NAME             "DEB-DEFAULT")
 	set(CPACK_DEBIAN_PACKAGE_VERSION       ${PROJECT_VERSION})
 	set(CPACK_DEBIAN_PACKAGE_ARCHITECTURE  ${CPACK_DEBIAN_PACKAGE_ARCHITECTURE})
 	set(CPACK_DEBIAN_PACKAGE_MAINTAINER    ${CPACK_PACKAGE_VENDOR})
@@ -205,9 +198,9 @@ if("DEB" IN_LIST CPACK_GENERATOR AND UNIX AND NOT APPLE)
 	set(CPACK_DEBIAN_PACKAGE_HOMEPAGE      ${CPACK_PACKAGE_HOMEPAGE_URL})
 
 	# --- Core runtime dependencies  ---
-	set(DEB_DEPENDS "libc6, libstdc++6, libgcc-s1")
+	set(deb_depends "libc6, libstdc++6, libgcc-s1")
 
-	set(CPACK_DEBIAN_PACKAGE_DEPENDS "${DEB_DEPENDS}")
+	set(CPACK_DEBIAN_PACKAGE_DEPENDS "${deb_depends}")
 	message(STATUS "DEB dependencies: ${CPACK_DEBIAN_PACKAGE_DEPENDS}")
 endif()
 
@@ -227,9 +220,9 @@ if("RPM" IN_LIST CPACK_GENERATOR AND UNIX AND NOT APPLE)
 	set(CPACK_RPM_PACKAGE_URL           ${CPACK_PACKAGE_HOMEPAGE_URL})
 
 	# --- Core runtime dependencies ---
-	set(RPM_REQUIRES "glibc, libstdc++")
+	set(rpm_requires "glibc, libstdc++")
 
-	set(CPACK_RPM_PACKAGE_REQUIRES "${RPM_REQUIRES}")
+	set(CPACK_RPM_PACKAGE_REQUIRES "${rpm_requires}")
 	message(STATUS "RPM dependencies: ${CPACK_RPM_PACKAGE_REQUIRES}")
 endif()
 

cmake/nfxDateTimeSources.cmake

@@ -3,12 +3,12 @@
 #==============================================================================
 
 #----------------------------------------------
-# Conditional headers and sources
+# Source files
 #----------------------------------------------
 
-set(PRIVATE_SOURCES)
+set(private_sources)
 
-list(APPEND PRIVATE_SOURCES
+list(APPEND private_sources
 	${NFX_DATETIME_SOURCE_DIR}/DateTime.cpp
 	${NFX_DATETIME_SOURCE_DIR}/DateTimeOffset.cpp
 	${NFX_DATETIME_SOURCE_DIR}/TimeSpan.cpp

cmake/nfxDateTimeTargets.cmake

@@ -11,7 +11,7 @@ if(NFX_DATETIME_BUILD_SHARED)
 	add_library(${PROJECT_NAME} SHARED)
 	target_sources(${PROJECT_NAME}
 		PRIVATE
-			${PRIVATE_SOURCES}
+			${private_sources}
 	)
 
 	set_target_properties(${PROJECT_NAME} PROPERTIES
@@ -27,7 +27,7 @@ if(NFX_DATETIME_BUILD_STATIC)
 	add_library(${PROJECT_NAME}-static STATIC)
 	target_sources(${PROJECT_NAME}-static
 		PRIVATE
-			${PRIVATE_SOURCES}
+			${private_sources}
 	)
 
 	set_target_properties(${PROJECT_NAME}-static PROPERTIES

samples/CMakeLists.txt

@@ -15,17 +15,17 @@ endif()
 # Samples source files
 #----------------------------------------------
 
-set(SAMPLE_SOURCES)
+set(sample_sources)
 
-list(APPEND SAMPLE_SOURCES
+list(APPEND sample_sources
 	Sample_DateTime.cpp
 )
 
 #----------------------------------------------
 # Configure samples executables
 #----------------------------------------------
 
-foreach(sample_source ${SAMPLE_SOURCES})
+foreach(sample_source ${sample_sources})
 	get_filename_component(sample_target_name ${sample_source} NAME_WE)
 
 	if(NOT TARGET ${sample_target_name})