Re: [bug-gettext] bootstrapping coreutils: autopoint from gettext 0.18.3.1 fails

[Eric Blake]

On 12/04/2013 03:18 AM, Daiki Ueno wrote:
> Pádraig Brady <address@hidden> writes:
> 
>> Any update on this.
>> I don't see the above patch upstream.
>> We're about to release a new coreutils release and I'd like
>> to be using the latest version of all tools if possible.
> 
> I've just pushed this to maint and master, with Eric's suggestion to
> ignore m4_esyscmd as well.  Sorry for the delay.

Hey, while we're at it, I noticed that you hard-code the use of the
first 'autom4te' on $PATH.  It might be nicer to honor the user's choice
of $AUTOM4TE, the way autoreconf does (so that if the user is
specifically requesting a non-default build of the autotools not first
on their PATH, they end up using the same toolset across the entire
build rather than a mix-and-match of one autom4te during the probe and a
different one during the remaining actions).  In particular, with your
current hard-coding of a PATH search for 'autom4te', I cannot do
'AUTOM4TE=/usr/bin/autom4te-2.69 ./bootstrap' to avoid the broken BSD
autom4te wrapper that eats stdin, but instead have to create a temporary
autom4te and inject it earlier in my PATH to have it forward to the real
autom4te-2.69 that I intend to use.

Something like this:

diff --git i/gettext-tools/misc/autopoint.in
w/gettext-tools/misc/autopoint.in
index 0de1322..3b66417 100644
--- i/gettext-tools/misc/autopoint.in
+++ w/gettext-tools/misc/autopoint.in
@@ -153,7 +153,7 @@ dnl macros which needs to be traced without aclocal.m4
 m4_define([AM_GNU_GETTEXT], [])
 m4_define([AM_GNU_GETTEXT_VERSION], [])
 ' \
-  | autom4te --no-cache --language=Autoconf-without-aclocal-m4 \
+  | ${AUTOM4TE:-autom4te} --no-cache
--language=Autoconf-without-aclocal-m4 \
     --trace="$1":\$% - "$2" 2>/dev/null
 }

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature