From: John Mark Bell <jmb@netsurf-browser.org>
Date: Fri, 6 Nov 2009 07:05:12 +0000 (-0000)
Subject: -Wextra is only understood by GCC 3.4.6 or later.
X-Git-Url: https://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=e394a3b924dfe18b03a81e9c028082c286be697c;p=libsvgtiny.git

-Wextra is only understood by GCC 3.4.6 or later.
It is a more descriptive synonym for -W.

Use -W instead which is understood by all common versions of GCC
(i.e. GCC 2.95 supports it, which is the oldest GCC version we have to support)

Additionally, conditionalise our -Werror usage on whether we're building for
BeOS/Haiku. On these platforms, the standard library headers result in warnings.
GCC 2 is quite happy to build this code on other platforms, so shouldn't be
the trigger for dropping -Werror (even if BeOS is the only target where GCC2 is
likely to be used).


svn path=/trunk/libsvgtiny/; revision=9670
---

diff --git a/Makefile b/Makefile
index 2f74aa1..4bbcb12 100644
--- a/Makefile
+++ b/Makefile
@@ -10,11 +10,12 @@ include build/makefiles/Makefile.tools
 TESTRUNNER := $(ECHO)
 
 # Toolchain flags
-WARNFLAGS := -Wall -Wundef -Wpointer-arith -Wcast-align \
+WARNFLAGS := -Wall -W -Wundef -Wpointer-arith -Wcast-align \
 	-Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes \
-	-Wmissing-declarations -Wnested-externs -Werror -pedantic
-ifneq ($(GCCVER),2)
-  WARNFLAGS := $(WARNFLAGS) -Wextra
+	-Wmissing-declarations -Wnested-externs -pedantic
+# BeOS/Haiku standard library headers create warnings
+ifneq ($(TARGET),beos)
+  WARNFLAGS := $(WARNFLAGS) -Werror
 endif
 CFLAGS := -D_BSD_SOURCE -I$(CURDIR)/include/ \
 	-I$(CURDIR)/src $(WARNFLAGS) $(CFLAGS)