diff options
author | Dan Williams <dcbw@redhat.com> | 2011-04-26 13:55:52 -0500 |
---|---|---|
committer | Dan Williams <dcbw@redhat.com> | 2011-04-26 13:55:52 -0500 |
commit | db9b1df0e47996ff8aaea468a11e1e97f64ee126 (patch) | |
tree | 2e29fa4df6384618a880c1944139eb36b516b9b9 /m4 | |
parent | aeb7123368c4f4ee26023cf94ee169f2882ac979 (diff) | |
download | NetworkManager-db9b1df0e47996ff8aaea468a11e1e97f64ee126.tar.gz |
build: don't use -Wstrict-prototypes (bgo #584947)
First, -Wstrict-prototypes wasn't actually getting used because
-Werror was already in CFLAGS and AC_TRY_COMPILE doesn't produce
main() functions with valid strict prototypes. Suck. But even
fixing that, the WiMAX SDK won't build with the flag, so just rip
it out.
Diffstat (limited to 'm4')
-rw-r--r-- | m4/compiler_warnings.m4 | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/m4/compiler_warnings.m4 b/m4/compiler_warnings.m4 index 55ecfe1bc4..575ea9a61e 100644 --- a/m4/compiler_warnings.m4 +++ b/m4/compiler_warnings.m4 @@ -2,13 +2,13 @@ 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) +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 -Wshadow -Wmissing-declarations -Wmissing-prototypes \ - -Wdeclaration-after-statement -Wstrict-prototypes \ + -Wdeclaration-after-statement \ -Wfloat-equal -Wno-unused-parameter -Wno-sign-compare \ -fno-strict-aliasing -Wno-unused-but-set-variable \ -Wundef; do @@ -26,6 +26,7 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then unset SAVE_CFLAGS done unset option + CFLAGS="$CFLAGS -Werror" else AC_MSG_RESULT(no) fi |