Revert compiler flag variables in CMake config.
This is a partial revert of d59586630e.
This commit is contained in:
parent
57805dfd6a
commit
aae9bbb003
@ -118,9 +118,6 @@ include_directories(${YAML_CPP_SOURCE_DIR}/include)
|
|||||||
###
|
###
|
||||||
### General compilation settings
|
### General compilation settings
|
||||||
###
|
###
|
||||||
set(yaml_c_flags ${CMAKE_C_FLAGS})
|
|
||||||
set(yaml_cxx_flags ${CMAKE_CXX_FLAGS})
|
|
||||||
|
|
||||||
if(BUILD_SHARED_LIBS)
|
if(BUILD_SHARED_LIBS)
|
||||||
set(LABEL_SUFFIX "shared")
|
set(LABEL_SUFFIX "shared")
|
||||||
else()
|
else()
|
||||||
@ -179,7 +176,7 @@ if(CMAKE_CXX_COMPILER_ID MATCHES "GNU" OR
|
|||||||
set(GCC_EXTRA_OPTIONS "${GCC_EXTRA_OPTIONS} ${FLAG_TESTED}")
|
set(GCC_EXTRA_OPTIONS "${GCC_EXTRA_OPTIONS} ${FLAG_TESTED}")
|
||||||
endif()
|
endif()
|
||||||
#
|
#
|
||||||
set(yaml_cxx_flags "-Wall ${GCC_EXTRA_OPTIONS} -pedantic -Wno-long-long -std=c++11 ${yaml_cxx_flags}")
|
set(yaml_cxx_flags "-Wall ${GCC_EXTRA_OPTIONS} -pedantic -Wno-long-long -std=c++11")
|
||||||
|
|
||||||
### Make specific
|
### Make specific
|
||||||
if(${CMAKE_BUILD_TOOL} MATCHES make OR ${CMAKE_BUILD_TOOL} MATCHES gmake)
|
if(${CMAKE_BUILD_TOOL} MATCHES make OR ${CMAKE_BUILD_TOOL} MATCHES gmake)
|
||||||
@ -217,7 +214,7 @@ if(MSVC)
|
|||||||
endif()
|
endif()
|
||||||
|
|
||||||
# correct linker options
|
# correct linker options
|
||||||
foreach(flag_var yaml_c_flags yaml_cxx_flags)
|
foreach(flag_var CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
|
||||||
foreach(config_name "" DEBUG RELEASE MINSIZEREL RELWITHDEBINFO)
|
foreach(config_name "" DEBUG RELEASE MINSIZEREL RELWITHDEBINFO)
|
||||||
set(var_name "${flag_var}")
|
set(var_name "${flag_var}")
|
||||||
if(NOT "${config_name}" STREQUAL "")
|
if(NOT "${config_name}" STREQUAL "")
|
||||||
@ -243,7 +240,7 @@ if(MSVC)
|
|||||||
# /W3 = set warning level; see http://msdn.microsoft.com/en-us/library/thxezb7y.aspx
|
# /W3 = set warning level; see http://msdn.microsoft.com/en-us/library/thxezb7y.aspx
|
||||||
# /wd4127 = disable warning C4127 "conditional expression is constant"; see http://msdn.microsoft.com/en-us/library/6t66728h.aspx
|
# /wd4127 = disable warning C4127 "conditional expression is constant"; see http://msdn.microsoft.com/en-us/library/6t66728h.aspx
|
||||||
# /wd4355 = disable warning C4355 "'this' : used in base member initializer list"; http://msdn.microsoft.com/en-us/library/3c594ae3.aspx
|
# /wd4355 = disable warning C4355 "'this' : used in base member initializer list"; http://msdn.microsoft.com/en-us/library/3c594ae3.aspx
|
||||||
set(yaml_cxx_flags "/W3 /wd4127 /wd4355 ${yaml_cxx_flags}")
|
set(CMAKE_CXX_FLAGS "/W3 /wd4127 /wd4355 ${CMAKE_CXX_FLAGS}")
|
||||||
endif()
|
endif()
|
||||||
|
|
||||||
|
|
||||||
@ -273,7 +270,7 @@ set(_INSTALL_DESTINATIONS
|
|||||||
###
|
###
|
||||||
add_library(yaml-cpp ${library_sources})
|
add_library(yaml-cpp ${library_sources})
|
||||||
set_target_properties(yaml-cpp PROPERTIES
|
set_target_properties(yaml-cpp PROPERTIES
|
||||||
COMPILE_FLAGS "${yaml_c_flags} ${yaml_cxx_flags}"
|
COMPILE_FLAGS "${yaml_cxx_flags}"
|
||||||
)
|
)
|
||||||
|
|
||||||
set_target_properties(yaml-cpp PROPERTIES
|
set_target_properties(yaml-cpp PROPERTIES
|
||||||
|
|||||||
@ -27,7 +27,7 @@ add_executable(run-tests
|
|||||||
${test_headers}
|
${test_headers}
|
||||||
)
|
)
|
||||||
set_target_properties(run-tests PROPERTIES
|
set_target_properties(run-tests PROPERTIES
|
||||||
COMPILE_FLAGS "${yaml_c_flags} ${yaml_cxx_flags} ${yaml_test_flags}"
|
COMPILE_FLAGS "${yaml_test_flags}"
|
||||||
)
|
)
|
||||||
target_link_libraries(run-tests yaml-cpp gmock)
|
target_link_libraries(run-tests yaml-cpp gmock)
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user