From: Antonio Ospite <ospite@studenti.unina.it>
Date: Thu, 10 May 2012 12:58:26 +0000 (+0200)
Subject: Don't set -pedantic-errors in CMAKE_C_FLAGS, it breaks check_symbol_exists()
X-Git-Tag: v0.1.3~37^2~2
X-Git-Url: https://git.ao2.it/libam7xxx.git/commitdiff_plain/281f9e80f6fd38866b895f1d2509fbe006520e22?ds=sidebyside

Don't set -pedantic-errors in CMAKE_C_FLAGS, it breaks check_symbol_exists()

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
---

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 23627fa..3e525cc 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -68,21 +68,27 @@ if (CMAKE_COMPILER_IS_GNUCC)
     -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)
-    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)
+
+    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()
 
-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)