Really enable -Werror
authorBlue Swirl <blauwirbel@gmail.com>
Sat, 13 Jun 2009 15:37:55 +0000 (15:37 +0000)
committerBlue Swirl <blauwirbel@gmail.com>
Sat, 13 Jun 2009 15:37:55 +0000 (15:37 +0000)
487fefdb.. did not actually enable -Werror despite the claims made by
configure output.

Signed-off-by: Blue Swirl <blauwirbel@gmail.com>

configure

index 48c8949..85bc598 100755 (executable)
--- a/configure
+++ b/configure
@@ -513,8 +513,21 @@ if test "$debug" = "no" ; then
 fi
 CFLAGS="$CFLAGS -Wall -Wundef -Wendif-labels -Wwrite-strings -Wmissing-prototypes -Wstrict-prototypes -Wredundant-decls"
 LDFLAGS="$LDFLAGS -g"
+
+# Consult white-list to determine whether to enable werror
+# by default.  Only enable by default for git builds
+if test -z "$werror" ; then
+    z_version=`cut -f3 -d. $source_path/VERSION`
+    if test "$z_version" = "50" -a \
+        "$linux" = "yes" ; then
+        werror="yes"
+    else
+        werror="no"
+    fi
+fi
+
 if test "$werror" = "yes" ; then
-CFLAGS="$CFLAGS -Werror"
+    CFLAGS="$CFLAGS -Werror"
 fi
 
 if test "$solaris" = "no" ; then
@@ -653,18 +666,6 @@ if test ! -x "$(which cgcc 2>/dev/null)"; then
     sparse="no"
 fi
 
-# Consult white-list to determine whether to enable werror
-# by default.  Only enable by default for git builds
-if test -z "$werror" ; then
-    z_version=`cut -f3 -d. $source_path/VERSION`
-    if test "$z_version" = "50" -a \
-       "$linux" = "yes" ; then
-       werror="yes"
-    else
-       werror="no"
-    fi
-fi
-
 #
 # Solaris specific configure tool chain decisions
 #