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 | |
| 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.
| -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 |