-Wformat is needed by gcc 4.8 (closes #17547)

diff --git a/Misc/NEWS b/Misc/NEWS
index 2c094208c63cc2129890109a607442aade91a489..3e177ca834ab7289d03762d27ebbde4b627e12bd 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -95,6 +95,9 @@ Library
 Build
 -----
 
+- Issue #17547: In configure, explicitly pass -Wformat for the benefit for GCC
+  4.8.
+
 - Issue #17682: Add the _io module to Modules/Setup.dist (commented out).
 
 - Issue #17086: Search the include and library directories provided by the

diff --git a/configure b/configure
index 528aa512ae09b527cc38d715bec9a26c00cf02c2..dc0dfd04ad1b5b55762958b9542583f672fadc8e 100755 (executable)
--- a/configure
+++ b/configure
@@ -6253,7 +6253,7 @@ then
   { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether gcc supports ParseTuple __format__" >&5
 $as_echo_n "checking whether gcc supports ParseTuple __format__... " >&6; }
   save_CFLAGS=$CFLAGS
-  CFLAGS="$CFLAGS -Werror"
+  CFLAGS="$CFLAGS -Werror -Wformat"
   cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 

diff --git a/configure.ac b/configure.ac
index cb2a958fbfe8f2f64bed2116413b36bac73141e1..30f5bf404a37fa1bc2eb38c43ae6f584dbc5497b 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -1326,7 +1326,7 @@ if test "$GCC" = "yes"
 then
   AC_MSG_CHECKING(whether gcc supports ParseTuple __format__)
   save_CFLAGS=$CFLAGS
-  CFLAGS="$CFLAGS -Werror"
+  CFLAGS="$CFLAGS -Werror -Wformat"
   AC_COMPILE_IFELSE([
     AC_LANG_PROGRAM([[void f(char*,...)__attribute((format(PyArg_ParseTuple, 1, 2)));]], [[]])
   ],[