summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndre Heinecke <aheinecke@gnupg.org>2019-05-03 10:56:03 +0200
committerAndre Heinecke <aheinecke@gnupg.org>2019-05-03 10:56:03 +0200
commit33b13d1c290d7ce35c636cce4265b512fb8e64c3 (patch)
tree9e90e92b6c8f577373935baa291d17020897dc40
parent0ed81498147723f0e9282df5ae08b8b8e40edd2f (diff)
downloadgpgme-33b13d1c290d7ce35c636cce4265b512fb8e64c3.tar.gz
Always use maintainer mode -Wno cflags
* configure.ac (CFLAGS): Move -Wno flags out of the maintainer mode. -- It makes sense to have more warnings in maintainer mode and not more warnings in release builds. GnuPG-Bug-Id: T4477
-rw-r--r--configure.ac33
1 files changed, 18 insertions, 15 deletions
diff --git a/configure.ac b/configure.ac
index e8a08158..d69cc82d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -599,27 +599,30 @@ AM_SUBST_NOTMAKE(API__SSIZE_T)
# Checks for compiler features.
if test "$GCC" = yes; then
CFLAGS="$CFLAGS -Wall -Wcast-align -Wshadow -Wstrict-prototypes"
+ CFLAGS="$CFLAGS -Wno-format-y2k"
+
+ # If -Wno-missing-field-initializers is supported we can expect a
+ # a larger set of warning options.
+ AC_MSG_CHECKING([if gcc supports -Wno-missing-field-initializers])
+ _gcc_cflags_save=$CFLAGS
+ CFLAGS="-Wno-missing-field-initializers"
+ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([],[])],_gcc_wopt=yes,_gcc_wopt=no)
+ AC_MSG_RESULT($_gcc_wopt)
+ CFLAGS=$_gcc_cflags_save;
+ if test x"$_gcc_wopt" = xyes ; then
+ CFLAGS="$CFLAGS -Wno-missing-field-initializers"
+ CFLAGS="$CFLAGS -Wno-sign-compare"
+ CFLAGS="$CFLAGS -Wno-format-zero-length"
+ CFLAGS="$CFLAGS -Wno-format-truncation"
+ CFLAGS="$CFLAGS -Wno-sizeof-pointer-div"
+ fi
if test "$USE_MAINTAINER_MODE" = "yes"; then
- CFLAGS="$CFLAGS -Wformat -Wno-format-y2k -Wformat-security"
-
- # If -Wno-missing-field-initializers is supported we can enable a
- # a bunch of really useful warnings.
- AC_MSG_CHECKING([if gcc supports -Wno-missing-field-initializers])
- _gcc_cflags_save=$CFLAGS
- CFLAGS="-Wno-missing-field-initializers"
- AC_COMPILE_IFELSE([AC_LANG_PROGRAM([],[])],_gcc_wopt=yes,_gcc_wopt=no)
- AC_MSG_RESULT($_gcc_wopt)
- CFLAGS=$_gcc_cflags_save;
if test x"$_gcc_wopt" = xyes ; then
CFLAGS="$CFLAGS -W -Wextra -Wbad-function-cast"
CFLAGS="$CFLAGS -Wwrite-strings"
CFLAGS="$CFLAGS -Wdeclaration-after-statement"
- CFLAGS="$CFLAGS -Wno-missing-field-initializers"
- CFLAGS="$CFLAGS -Wno-sign-compare"
- CFLAGS="$CFLAGS -Wno-format-zero-length"
- CFLAGS="$CFLAGS -Wno-format-truncation"
- CFLAGS="$CFLAGS -Wno-sizeof-pointer-div"
fi
+ CFLAGS="$CFLAGS -Wformat -Wformat-security"
CXXFLAGS="$CXXFLAGS -Wall -Wextra -Wno-shadow"
AC_MSG_CHECKING([if gcc supports -Wpointer-arith])