From: Jim Meyering <jim@meyering.net>
Date: Thu, 1 Mar 2007 10:06:58 +0000 (+0100)
Subject: * Makefile.maint (warn_cflags): Hoist, adding "-Dlint -O".
X-Git-Tag: COREUTILS-6_9~56
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=262963212bc15e89b38eabc27c12572b5025ba70;p=thirdparty%2Fcoreutils.git

* Makefile.maint (warn_cflags): Hoist, adding "-Dlint -O".
(my-distcheck): Use the new variable, instead of too-long literal.
---

diff --git a/ChangeLog b/ChangeLog
index 2b01c56f8e..b32fbff3bb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2007-03-01  Jim Meyering  <jim@meyering.net>
 
+	* Makefile.maint (warn_cflags): Hoist, adding "-Dlint -O".
+	(my-distcheck): Use the new variable, instead of too-long literal.
+
 	Make "make syntax-check" rules less coreutils-specific.
 	* Makefile.maint (sc_cast_of_x_alloc_return_value): Use CVS_LIST_EXCEPT.
 	(sc_cast_of_alloca_return_value): Likewise.
diff --git a/Makefile.maint b/Makefile.maint
index fece10c6bc..8391735111 100644
--- a/Makefile.maint
+++ b/Makefile.maint
@@ -513,6 +513,8 @@ null_AM_MAKEFLAGS = \
   AUTOHEADER=false \
   MAKEINFO=false
 
+warn_cflags = -Dlint -O -Werror -Wall -Wformat -Wshadow -Wpointer-arith
+
 # Detect format-string/arg-list mismatches that would normally be obscured
 # by the use of _().  The --disable-nls effectively defines away that macro,
 # and building with CFLAGS='-Wformat -Werror' causes any format warning to be
@@ -528,7 +530,7 @@ my-distcheck: $(local-check) $(release_archive_dir)/$(prev-tgz)
 	GZIP=$(GZIP_ENV) $(AMTAR) -C $(t) -zxf $(distdir).tar.gz
 	cd $(t)/$(distdir)				\
 	  && ./configure --disable-nls			\
-	  && $(MAKE) CFLAGS='-Werror -Wall -Wformat -Wshadow -Wpointer-arith' \
+	  && $(MAKE) CFLAGS='$(warn_cflags)'		\
 	      AM_MAKEFLAGS='$(null_AM_MAKEFLAGS)'	\
 	  && $(MAKE) dvi				\
 	  && $(MAKE) check				\