Makefile: Split warnings into two sets (WARN_CFLAGS and WARN_CXXFLAGS)
authorCarl Worth <cworth@cworth.org>
Wed, 2 Dec 2009 00:42:50 +0000 (16:42 -0800)
committerCarl Worth <cworth@cworth.org>
Wed, 2 Dec 2009 00:42:50 +0000 (16:42 -0800)
Some C++ compilers complain about -Wmissing-declarations not being
valid, so avoid passing it except when compiling a C file.

Makefile

index 7a9e44c629b833b27e3fbfe54fcd8657e49d78b2..6a2ecea58a604b2549e8b5330280686a8f6fb82d 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,7 +1,9 @@
-# Default FLAGS, (can be overridden by user such as "make CFLAGS=-O2")
-WARN_FLAGS=-Wall -Wextra -Wmissing-declarations -Wwrite-strings -Wswitch-enum
+# Default FLAGS, (can be overridden by user such as "make CFLAGS=-g")
 CFLAGS=-O2
 
+WARN_CXXFLAGS=-Wall -Wextra -Wwrite-strings -Wswitch-enum
+WARN_CFLAGS=$(WARN_CXXFLAGS) -Wmissing-declarations
+
 # Additional programs that are used during the compilation process.
 EMACS ?= emacs
 # Lowercase to avoid clash with GZIP environment variable for passing
@@ -25,8 +27,8 @@ all_deps = Makefile Makefile.local Makefile.config \
                   lib/Makefile lib/Makefile.local
 
 # Now smash together user's values with our extra values
-override CFLAGS += $(WARN_FLAGS) $(extra_cflags)
-override CXXFLAGS += $(WARN_FLAGS) $(extra_cflags) $(extra_cxxflags)
+override CFLAGS += $(WARN_CFLAGS) $(extra_cflags)
+override CXXFLAGS += $(WARN_CXXFLAGS) $(extra_cflags) $(extra_cxxflags)
 
 override LDFLAGS += \
        $(shell pkg-config --libs glib-2.0 gmime-2.4 talloc) \