From 4f2471026d69f8eeffa1b9403d96aca5e050399b Mon Sep 17 00:00:00 2001 From: David Levine Date: Wed, 25 Jan 2012 20:48:55 -0600 Subject: [PATCH] Updated config test to add -Wextra to be consistent with others. So, it's back! --- configure.ac | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index d73f376..37d7c23 100644 --- a/configure.ac +++ b/configure.ac @@ -215,11 +215,10 @@ if test "$nmh_cv_has_unusedmacros" = 'yes'; then fi AC_SUBST(DISABLE_UNUSED_MACROS_WARNING)dnl -AC_CACHE_CHECK([whether compiler supports -Wextra], [nmh_cv_wextra], +AC_CACHE_CHECK([whether compiler supports -Wextra], [nmh_cv_has_wextra], [nmh_saved_cflags="$CFLAGS" CFLAGS="$CFLAGS -Wextra -Wno-clobbered" - AC_COMPILE_IFELSE([AC_LANG_PROGRAM()], - [nmh_cv_wextras=' -Wextra -Wno-clobbered'], [echo $ECHO_N "no$ECHO_C"]) + AC_TRY_COMPILE([],[],nmh_cv_has_wextra=yes,nmh_cv_has_wextra=no) CFLAGS="$nmh_saved_cflags"]) AC_CACHE_CHECK([whether compiler supports -Wno-pointer-sign], [nmh_cv_has_noptrsign], @@ -237,9 +236,17 @@ dnl also use -Wno-pointer-sign, because gcc 4 now produces a lot of new dnl warnings which are probably mostly spurious and which in any case we dnl don't want to deal with now. if test "$nmh_cv_has_noptrsign" = "yes"; then - nmh_gcc_warnflags="-Wall$nmh_cv_wextra -Wno-pointer-sign" + if test "$nmh_cv_has_wextra" = "yes"; then + nmh_gcc_warnflags="-Wall -Wextra -Wno-clobbered -Wno-pointer-sign" + else + nmh_gcc_warnflags="-Wall -Wno-pointer-sign" + fi else - nmh_gcc_warnflags="-Wall$nmh_cv_wextra" + if test "$nmh_cv_has_wextra" = "yes"; then + nmh_gcc_warnflags="-Wall -Wextra -Wno-clobbered" + else + nmh_gcc_warnflags="-Wall" + fi fi if test -n "$auto_cflags"; then -- 1.7.10.4