cmake: add missing SYSTEM when include external directories

jcarpent · jcarpent · commit 7e1e8f0fda73 · 2022-12-23T10:24:39.000+01:00
diff --git a/bindings/python/CMakeLists.txt b/bindings/python/CMakeLists.txt
@@ -52,7 +52,7 @@ function(CREATE_PYTHON_TARGET target_name COMPILE_OPTIONS dependencies)
   target_compile_definitions(${target_name}
                              PRIVATE PYTHON_MODULE_NAME=${target_name})
   target_include_directories(
-    ${target_name} PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)
+    ${target_name} SYSTEM PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)
   set_target_properties(
     ${target_name}
     PROPERTIES OUTPUT_NAME ${target_name}
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -56,7 +56,8 @@ add_test_cflags(
   "-DTEST_SERIALIZATION_FOLDER=\\\\\"${CMAKE_CURRENT_BINARY_DIR}/serialization-data\\\\\""
 )
 target_include_directories(
-  test-cpp-serialization PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)
+  test-cpp-serialization SYSTEM
+  PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)
 
 if(NOT CMAKE_BUILD_TYPE STREQUAL "Debug" AND NOT MSVC)
   proxsuite_test(dense_maros_meszaros src/dense_maros_meszaros.cpp)

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,8 @@ add_test_cflags(`
`56`	`56`	`"-DTEST_SERIALIZATION_FOLDER=\\\\\"${CMAKE_CURRENT_BINARY_DIR}/serialization-data\\\\\""`
`57`	`57`	`)`
`58`	`58`	`target_include_directories(`
`59`		`- test-cpp-serialization PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)`
	`59`	`+ test-cpp-serialization SYSTEM`
	`60`	`+ PRIVATE ${PROJECT_SOURCE_DIR}/external/cereal/include)`
`60`	`61`
`61`	`62`	`if(NOT CMAKE_BUILD_TYPE STREQUAL "Debug" AND NOT MSVC)`
`62`	`63`	`proxsuite_test(dense_maros_meszaros src/dense_maros_meszaros.cpp)`