From b2add2f77f8474efc3a9b1a4e733aa6747c42b6e Mon Sep 17 00:00:00 2001 From: Joaquim Casals <jcasals@iri.upc.edu> Date: Tue, 7 May 2019 08:35:51 +0200 Subject: [PATCH] Reverse order of CMAKE_CXX_FLAGS initialization The old order caused cmake in Mac OSX to not detect c++11. This fixes the problem. --- CMakeLists.txt | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index a16ca6601..6bed07c68 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -26,12 +26,6 @@ message(STATUS "Configured to compile in ${CMAKE_BUILD_TYPE} mode.") SET(CMAKE_CXX_FLAGS_DEBUG "-g -Wall -D_REENTRANT") SET(CMAKE_CXX_FLAGS_RELEASE "-O3 -D_REENTRANT") -if(UNIX) - # GCC is not strict enough by default, so enable most of the warnings. - set(CMAKE_CXX_FLAGS - "${CMAKE_CXX_FLAGS} -Werror=all -Werror=extra -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers") -endif(UNIX) - #Set compiler according C++11 support include(CheckCXXCompilerFlag) CHECK_CXX_COMPILER_FLAG("-std=c++11" COMPILER_SUPPORTS_CXX11) @@ -46,6 +40,12 @@ else() message(STATUS "The compiler ${CMAKE_CXX_COMPILER} has no C++11 support. Please use a different C++ compiler.") endif() +if(UNIX) + # GCC is not strict enough by default, so enable most of the warnings. + set(CMAKE_CXX_FLAGS + "${CMAKE_CXX_FLAGS} -Werror=all -Werror=extra -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers") +endif(UNIX) + #OPTION(BUILD_DOC "Build Documentation" OFF) OPTION(BUILD_TESTS "Build Unit tests" ON) -- GitLab