Set cxx extensions and std required flags per target

benvenutti · Aug 25, 2024 · 45d6794 · 45d6794
1 parent f23fa40
commit 45d6794
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 6 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -34,9 +34,6 @@ if(NOT CMAKE_BUILD_TYPE)
   set(CMAKE_BUILD_TYPE Release FORCE)
 endif()
 
-set(CXX_STANDARD_REQUIRED ON)
-set(CXX_EXTENSIONS OFF)
-
 # testing:
 
 if(BUILD_TESTS)

diff --git a/app-qtwidget/CMakeLists.txt b/app-qtwidget/CMakeLists.txt
@@ -14,7 +14,13 @@ qt_add_executable(
 
 target_compile_features(core8-gui-qtwidget PUBLIC cxx_std_20)
 
-set_target_properties(core8-gui-qtwidget PROPERTIES COMPILE_WARNING_AS_ERROR ON)
+set_target_properties(
+  core8-gui-qtwidget
+  PROPERTIES
+    CXX_STANDARD_REQUIRED ON
+    CXX_EXTENSIONS OFF
+    COMPILE_WARNING_AS_ERROR ON
+)
 
 target_link_libraries(
   core8-gui-qtwidget

diff --git a/model/CMakeLists.txt b/model/CMakeLists.txt
@@ -16,7 +16,13 @@ add_library(
 
 target_compile_features(core8-model PUBLIC cxx_std_20)
 
-set_target_properties(core8-model PROPERTIES COMPILE_WARNING_AS_ERROR ON)
+set_target_properties(
+  core8-model
+  PROPERTIES
+    CXX_STANDARD_REQUIRED ON
+    CXX_EXTENSIONS OFF
+    COMPILE_WARNING_AS_ERROR ON
+)
 
 configure_file(
   ${CMAKE_CURRENT_SOURCE_DIR}/include/model/Version.hpp.in

diff --git a/model/tests/CMakeLists.txt b/model/tests/CMakeLists.txt
@@ -22,7 +22,13 @@ add_executable(
 
 target_compile_features(model_tests PUBLIC cxx_std_20)
 
-set_target_properties(model_tests PROPERTIES COMPILE_WARNING_AS_ERROR ON)
+set_target_properties(
+  model_tests
+  PROPERTIES
+    CXX_STANDARD_REQUIRED ON
+    CXX_EXTENSIONS OFF
+    COMPILE_WARNING_AS_ERROR ON
+)
 
 target_link_libraries(model_tests PRIVATE core8-model Catch2::Catch2WithMain)