[Top][All Lists]

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

Re: enable -Werror for lib/ in coreutils

From: Pádraig Brady
Subject: Re: enable -Werror for lib/ in coreutils
Date: Tue, 27 Oct 2009 13:14:07 +0000
User-agent: Thunderbird (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" ?


reply via email to

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