Tidy cmake, separate apply compilation flags (#2014)

This will extract compilation flag setting code to separate file

Signed-off-by: SaeHie Park <saehie.park@samsung.com>

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 1fc340b..efec620 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -14,21 +14,9 @@ include("cmake/option/identify_platform.cmake")
 # platform specific options
 include("cmake/option/option_${TARGET_PLATFORM}.cmake")
 
-# add common flags
-foreach(FLAG ${FLAGS_COMMON})
-  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${FLAG}")
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${FLAG}")
-endforeach()
-
-# add c flags
-foreach(FLAG ${FLAGS_CONLY})
-  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${FLAG}")
-endforeach()
-
-# add cxx flags
-foreach(FLAG ${FLAGS_CXXONLY})
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${FLAG}")
-endforeach()
+# apply compilation flags
+# note: this should be placed after cmake/option/option_xxx.cmake files
+include("cmake/ApplyCompileFlags.cmake")
 
 # Configuration flags
 include("cmake/CfgOptionFlags.cmake")

diff --git a/cmake/ApplyCompileFlags.cmake b/cmake/ApplyCompileFlags.cmake
new file mode 100644 (file)
index 0000000..e2124bb
--- /dev/null
+++ b/cmake/ApplyCompileFlags.cmake
@@ -0,0 +1,15 @@
+# add common flags
+foreach(FLAG ${FLAGS_COMMON})
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${FLAG}")
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${FLAG}")
+endforeach()
+
+# add c flags
+foreach(FLAG ${FLAGS_CONLY})
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${FLAG}")
+endforeach()
+
+# add cxx flags
+foreach(FLAG ${FLAGS_CXXONLY})
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${FLAG}")
+endforeach()