bug-gnulib
[Top][All Lists]
Advanced

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

Re: a saner bootstrap script


From: Gary V. Vaughan
Subject: Re: a saner bootstrap script
Date: Sun, 16 Oct 2011 18:28:27 +0700

Hi Bruno,

On 16 Oct 2011, at 15:58, Bruno Haible wrote:
> Gary V. Vaughan wrote:
>> If there is a bug in gnulib-tool, or autopoint that puts unnecessary
>> 'intl/' references into Makefiles when the presence of
>> AM_GNU_GETTEXT_VERSION in configure.ac is a declaration that says there
>> is no need for local 'intl/', then I think the bug should be fixed there
>> rather than patched around in bootstrap.  Unfortunately, some difference
>> in our environments is preventing me from exploring further since I can't
>> reproduce your error on my machine :(
> 
> It doesn't come from gnulb-tool nor autopoint, but from the gnulib module
> called 'gettext'. The -Iintl option there is generally safe, except that
> coreutils activates lots of GCC warnings plus -Werror.

Thanks for the info.  Then I think the right approach is for coreutils to
use gl/modules/gettext.diff to make the gettext it imports compatible with
the gcc options it is using.  Although it would be easy enough to add a
small hook function to bootstrap.conf for now and then remove it later,
coreutils already makes use of local gnulib patches, and using that mechanism
to fix this bootstrap problem also effectively creates a builtin warning
when upstream gnulib changes (when the gnulib gettext module follows) which
has the benefit of reducing the tendency to let cruft collect in
bootstrap.conf.

I guess that my gcc is rejecting the configure-time addition of whatever
flag is causing the abort on Jim's machine... I've added this patch to my
github mirror of coreutils:

  
https://github.com/gvvaughan/GNU-coreutils/commit/d49887540fffc19cf955c7e6b0656dfed4028806

Jim, please consider pulling it into coreutils master as a better fix than
twidding Makefiles after the fact in bootstrap.conf.

> This -Iintl option will disappear with gettext version 0.19 - because
> then "gettextize --intl" will not work any more; it's already deprecated.

And then, when coreutils moves to gettext-0.19, it's easy to remove the
diff file at the same time.

Cheers,
-- 
Gary V. Vaughan (gary AT gnu DOT org)


reply via email to

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