diff options
author | Iain Lane <iainl@gnome.org> | 2018-04-24 12:31:44 +0100 |
---|---|---|
committer | Aleksander Morgado <aleksander@aleksander.es> | 2018-04-24 18:08:29 +0200 |
commit | 5072db3418ce5799a75a577612b30de97ca700f0 (patch) | |
tree | aaa2e709597457413212ca672bdefdcf6b5a7b1c | |
parent | 7e94928301a971488ea53b17dd7d5ef37f468b1d (diff) |
m4/compiler_warnings.m4: Let -Werror be disabled
It's annoying for distributors to build with -Werror, since it means
that compiler upgrades can break the build. Let's let them disable it,
but keep it enabled by default.
-rw-r--r-- | m4/compiler_warnings.m4 | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/m4/compiler_warnings.m4 b/m4/compiler_warnings.m4 index abc2da77..f498a282 100644 --- a/m4/compiler_warnings.m4 +++ b/m4/compiler_warnings.m4 @@ -1,11 +1,11 @@ AC_DEFUN([NM_COMPILER_WARNINGS], [AC_ARG_ENABLE(more-warnings, - AS_HELP_STRING([--enable-more-warnings], [Maximum compiler warnings]), - set_more_warnings="$enableval",set_more_warnings=yes) -AC_MSG_CHECKING(for more warnings, including -Werror) + AS_HELP_STRING([--enable-more-warnings], [Possible values: no/yes/error]), + set_more_warnings="$enableval",set_more_warnings=error) +AC_MSG_CHECKING(for more warnings) if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then AC_MSG_RESULT(yes) - CFLAGS="-Wall -Werror -std=gnu89 $CFLAGS" + CFLAGS="-Wall -std=gnu89 $CFLAGS" for option in -Wmissing-declarations -Wmissing-prototypes \ -Wdeclaration-after-statement -Wstrict-prototypes \ @@ -26,6 +26,9 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then unset SAVE_CFLAGS done unset option + if test "x$set_more_warnings" = xerror; then + CFLAGS="$CFLAGS -Werror" + fi else AC_MSG_RESULT(no) fi |