Don't set -pedantic-errors in CMAKE_C_FLAGS, it breaks check_symbol_exists()
authorAntonio Ospite <ospite@studenti.unina.it>
Thu, 10 May 2012 12:58:26 +0000 (14:58 +0200)
committerAntonio Ospite <ospite@studenti.unina.it>
Thu, 10 May 2012 13:58:00 +0000 (15:58 +0200)
Set -pedantic-errors in more specific variables like
CMAKE_C_FLAGS_DEBUG, CMAKE_C_FLAGS_RELEASE and
CMAKE_C_FLAGS_RELWITHDEBINFO.

See also http://public.kitware.com/Bug/view.php?id=13208

CMakeLists.txt

index 23627fa..3e525cc 100644 (file)
@@ -68,21 +68,27 @@ if (CMAKE_COMPILER_IS_GNUCC)
     -Wwrite-strings
     -fstack-protector
     --param=ssp-buffer-size=4)
     -Wwrite-strings
     -fstack-protector
     --param=ssp-buffer-size=4)
+endif()
+
+set(CMAKE_C_FLAGS_DEBUG "-O0 -ggdb -DDEBUG=1")
+set(CMAKE_C_FLAGS_RELEASE "-O2")
+set(CMAKE_C_FLAGS_RELWITHDEBINFO "-O2 -g")
 
 
+if (CMAKE_COMPILER_IS_GNUCC)
   if (STRICT_COMPILATION_CHECKS)
   if (STRICT_COMPILATION_CHECKS)
-    add_flags(CMAKE_C_FLAGS
+    add_flags(STRICT_FLAGS
       -Werror
       # NOTE: Vanilla libusb-1.0.8 can't live with -pedantic-errors
       -pedantic-errors
       # NOTE: GCC >= 4.6 is needed for -Wunused-but-set-variable
       -Wunused-but-set-variable)
       -Werror
       # NOTE: Vanilla libusb-1.0.8 can't live with -pedantic-errors
       -pedantic-errors
       # NOTE: GCC >= 4.6 is needed for -Wunused-but-set-variable
       -Wunused-but-set-variable)
+
+    add_flags(CMAKE_C_FLAGS_DEBUG ${STRICT_FLAGS})
+    add_flags(CMAKE_C_FLAGS_RELEASE ${STRICT_FLAGS})
+    add_flags(CMAKE_C_FLAGS_RELWITHDEBINFO ${STRICT_FLAGS})
   endif()
 endif()
 
   endif()
 endif()
 
-set(CMAKE_C_FLAGS_DEBUG "-O0 -ggdb -DDEBUG=1")
-set(CMAKE_C_FLAGS_RELEASE "-O2")
-set(CMAKE_C_FLAGS_RELWITHDEBINFO "-O2 -g")
-
 # Add library project
 add_subdirectory(src)
 add_subdirectory(examples)
 # Add library project
 add_subdirectory(src)
 add_subdirectory(examples)