I *did* intend to enable $(WERROR_CFLAGS), but didn't notice a vestigial change set that left it commented out. Thanks, Pádraig.
Now, I configure coreutils with --enable-gcc-warnings on modern-enough systems. I'll push this later today. >From 3b3e21f7d4108dcfb1a54b147b29c5b0286918d9 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyer...@redhat.com> Date: Mon, 22 Dec 2008 18:21:59 +0100 Subject: [PATCH] build: do enable $(WERROR_CFLAGS) for src/ files. MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit This reverts the more recent commit (of two) entitled "build: add configure-time --enable-gcc-warnings option; avoid warnings". Pádraig Brady spotted the contradiction between the log message and actual change. This reverts commit 292d68565a34b237cd2bf586ace545b7cd3dbfcf. --- src/Makefile.am | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 68c5f75..819b874 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -26,7 +26,7 @@ no_install__progs = \ build_if_possible__progs = \ chroot df hostid nice pinky stty su uname uptime users who -AM_CFLAGS = $(WARN_CFLAGS) # $(WERROR_CFLAGS) +AM_CFLAGS = $(WARN_CFLAGS) $(WERROR_CFLAGS) EXTRA_PROGRAMS = \ $(no_install__progs) \ -- 1.6.1.rc3.359.g43db1 _______________________________________________ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils
