|
From: | Pádraig Brady |
Subject: | Re: enable -Werror for lib/ in coreutils |
Date: | Tue, 27 Oct 2009 13:14:07 +0000 |
User-agent: | Thunderbird 2.0.0.6 (X11/20071008) |
Jim Meyering wrote: > Subject: [PATCH 2/2] build (--enable-gcc-warnings): enable gcc's -Werror also > in lib/ > > * configure.ac (GNULIB_WARN_CFLAGS): Define. > * lib/Makefile.am (AM_CFLAGS): Use $(GNULIB_WARN_CFLAGS) > rather than $(WARN_CFLAGS) and add $(WERROR_CFLAGS). > * gl/lib/isnan.c.diff: New file. > * gl/lib/regcomp.c.diff: New file. > * gl/lib/regex_internal.c.diff: New file. > * gl/lib/regexec.c.diff: New file. > * gl/lib/strftime.c.diff: New file. > * gl/lib/unicodeio.c.diff: New file. Are you maintaining diffs or patching gnulib directly. Seems like it might be more trouble than it's worth to maintain diffs. If patching directly then modules/regex would need to depend on verify. Also is verify.h needed here? > diff --git a/gl/lib/regcomp.c.diff b/gl/lib/regcomp.c.diff > + > ++#include "intprops.h" ++#include "verify.h" ? cheers, Pádraig.
[Prev in Thread] | Current Thread | [Next in Thread] |