From e00a1f0dffe58a36d30a93a073da31562a9ffa57 Mon Sep 17 00:00:00 2001 From: John Mark Bell Date: Fri, 6 Nov 2009 07:01:06 +0000 Subject: -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/libcss/; revision=9665 --- Makefile | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index c9b1342..9d46dff 100644 --- a/Makefile +++ b/Makefile @@ -10,11 +10,12 @@ include build/makefiles/Makefile.tools TESTRUNNER := $(PERL) build/testtools/testrunner.pl # 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) -- cgit v1.2.3