Some -Werror related cleanup.
authorSascha Wildner <saw@online.de>
Wed, 16 Sep 2009 13:44:36 +0000 (15:44 +0200)
committerSascha Wildner <saw@online.de>
Wed, 16 Sep 2009 13:44:36 +0000 (15:44 +0200)
Instead of enabling WARNS_WERROR by default, enable -Werror by default
in bsd.sys.mk and if the platform is i386. Get rid of WARNS_WERROR
completely.

etc/defaults/make.conf
share/man/man5/make.conf.5
share/mk/bsd.sys.mk

index 065de82..3a08915 100644 (file)
@@ -68,10 +68,6 @@ BDECFLAGS=   -W -Wall -ansi -pedantic -Wbad-function-cast -Wcast-align \
                -Wmissing-prototypes -Wnested-externs -Wpointer-arith \
                -Wredundant-decls -Wshadow -Wstrict-prototypes -Wwrite-strings
 #
-# WARNS_WERROR causes -Werror to be added when WARNS is in effect.
-# Set NO_WERROR to disable.
-WARNS_WERROR=  yes
-#
 # To compile just the kernel with special optimizations, you should use
 # this instead of CFLAGS (which is not applicable to kernel builds anyway).
 # There is very little to gain by using higher optimization levels, and doing
index c0e0f2f..ca0bd78 100644 (file)
@@ -25,7 +25,7 @@
 .\" $FreeBSD: src/share/man/man5/make.conf.5,v 1.12.2.30 2003/05/18 17:05:55 brueffer Exp $
 .\" $DragonFly: src/share/man/man5/make.conf.5,v 1.38 2008/11/03 00:25:45 pavalos Exp $
 .\"
-.Dd August 11, 2009
+.Dd September 16, 2009
 .Dt MAKE.CONF 5
 .Os
 .Sh NAME
@@ -196,18 +196,6 @@ Set this to the flag to pass the
 command.
 If set to a blank value, components will be installed with debugging
 symbols.
-.It Va WARNS_WERROR
-Causes
-.Fl Werror
-to be added to
-.Va CWARNFLAGS
-when WARNS is in effect.
-.Va WARNS_WERROR
-is enabled by default.
-Set
-.Va NO_WERROR
-to disable
-.Fl Werror .
 .It Va WANT_HESIOD
 .Pq Vt bool
 Set this to build
@@ -224,8 +212,6 @@ Set this if you don't want
 .Fl Werror
 to be added to
 .Va CWARNFLAGS .
-See also
-.Va WARNS_WERROR .
 .It Va STATIC_LOCALES
 .Pq Vt str
 Set to a list of locales to build into statically-linked binaries.
index a008d66..37fd0d4 100644 (file)
@@ -24,7 +24,7 @@ CFLAGS                += -std=${CSTD}
 . if defined(WARNS)
 .  if ${WARNS} >= 1
 CWARNFLAGS     +=      -Wsystem-headers
-.   if defined(WARNS_WERROR) && !defined(NO_WERROR)
+.   if !defined(NO_WERROR) && ${MACHINE_ARCH} == "i386"
 CWARNFLAGS     +=      -Werror
 .   endif
 .  endif
@@ -55,7 +55,7 @@ WFORMAT               =       1
 . if defined(WFORMAT)
 .  if ${WFORMAT} > 0
 CWARNFLAGS     +=      -Wformat=2 -Wno-format-extra-args
-.   if defined(WARNS_WERROR) && !defined(NO_WERROR)
+.   if !defined(NO_WERROR) && ${MACHINE_ARCH} == "i386"
 CWARNFLAGS     +=      -Werror
 .   endif
 .  endif