From 6e55b50996ffb4e9198b6e2fd23e7596a3fb4109 Mon Sep 17 00:00:00 2001 From: BestITUserEUW Date: Mon, 29 Sep 2025 21:26:05 +0200 Subject: [PATCH] use target_compile_definitions for REFLECTCPP_USE_STD_EXPECTED instead of CMAKE_CXX_FLAGS --- CMakeLists.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index a023b6f0..717adbff 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -26,7 +26,6 @@ option(REFLECTCPP_USE_BUNDLED_DEPENDENCIES "Use the bundled dependencies" ON) option(REFLECTCPP_USE_STD_EXPECTED "Use std::expected instead of the built-in Result type (requires C++-23)" OFF) if(REFLECTCPP_USE_STD_EXPECTED) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DREFLECTCPP_USE_STD_EXPECTED") if(NOT DEFINED CMAKE_CXX_STANDARD) set(CMAKE_CXX_STANDARD 23) endif() @@ -160,6 +159,10 @@ else() target_compile_options(reflectcpp PRIVATE $<$:-Wall -Wextra>) endif() +if(REFLECTCPP_USE_STD_EXPECTED) + target_compile_definitions(reflectcpp PUBLIC REFLECTCPP_USE_STD_EXPECTED) +endif() + set(REFLECT_CPP_SOURCES src/reflectcpp.cpp )