* 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 2b01c56f8e8da16cbb526a2e595eb65b78f0a002..b32fbff3bbd7775d74175eb1348cad622d62de5e 100644 (file)
--- 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 fece10c6bc8dd685fda954dbb6761855a13d2a48..839173511192717eea150e0beee5a49417107dc1 100644 (file)
--- 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                              \