Merge pull request #38 from rcsoccersim/fix/cmake_compile_options

Replace CMAKE_CXX_FLAGS variable with target_compile_options
rcsoccersim · Apr 9, 2022 · f7067b6 · f7067b6
2 parents f7a61e9 + adcc887
commit f7067b6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -13,7 +13,6 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
 if(NOT CMAKE_BUILD_TYPE)
   set(CMAKE_BUILD_TYPE "Release" CACHE STRING "" FORCE)
 endif()
-set(CMAKE_CXX_FLAGS "-W -Wall")
 
 include(GNUInstallDirs)
 

diff --git a/rcss/rcg/CMakeLists.txt b/rcss/rcg/CMakeLists.txt
@@ -21,6 +21,11 @@ target_include_directories(rcssrcg
   ${PROJECT_BINARY_DIR}
   )
 
+target_compile_options(rcssrcg
+  PRIVATE
+  -W -Wall
+  )
+
 
 set_target_properties(rcssrcg PROPERTIES
   VERSION 17.0.0

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -51,6 +51,11 @@ target_compile_definitions(rcssmonitor
   HAVE_CONFIG_H
   )
 
+target_compile_options(rcssmonitor
+  PRIVATE
+  -W -Wall
+  )
+
 set_target_properties(rcssmonitor
   PROPERTIES
   RUNTIME_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}