]> granicus.if.org Git - json-c/commitdiff
Properly append to CMAKE_C_FLAGS string
authorJaap Keuter <jaap.keuter@xs4all.nl>
Thu, 12 Dec 2019 20:28:03 +0000 (21:28 +0100)
committerJaap Keuter <jaap.keuter@xs4all.nl>
Thu, 12 Dec 2019 20:28:03 +0000 (21:28 +0100)
Contrary to other CMAKE variables the CMAKE_C_FLAGS variable is the
composed string of flags for the C compiler. It is therefore not a list
to append to. Current implementation results in these incorrect CFLAGS,
e.g., "-O2 -g -fblahblah;-UNDEBUG". Extending the CFLAGS this way
results in the proper CFLAGS, e.g., "-O2 -g -fblahblah -UNDEBUG".

tests/CMakeLists.txt

index 72679db5d2ea390bf0ea04413a925f9cf88eba2c..15e325860b4d67ad9d60e377ae0f1fae978c3311 100644 (file)
@@ -54,6 +54,6 @@ endforeach(TESTNAME)
 
 # Make sure NDEBUG is always undefined for tests
 if (UNIX OR MINGW OR CYGWIN)
-    list(APPEND CMAKE_C_FLAGS -UNDEBUG)
+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -UNDEBUG")
 endif()