bug-coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] build: do enable $(WERROR_CFLAGS) for src/ files.


From: Jim Meyering
Subject: [PATCH] build: do enable $(WERROR_CFLAGS) for src/ files.
Date: Tue, 23 Dec 2008 15:59:50 +0100

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 <address@hidden>
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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]