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

[Daiki Ueno]

Hi,

Bernhard Voelker <address@hidden> writes:

>   ./bootstrap: autopoint --force
>   sh: build-aux/git-version-gen: No such file or directory
>   /usr/bin/m4:configure.ac:444: cannot open `m4/cu-progs.m4': \
>     No such file or directory
>   autom4te: /usr/bin/m4 failed with exit status: 1
>   autopoint: *** Missing version: please specify in configure.ac \
>     through a line 'AM_GNU_GETTEXT_VERSION(x.yy.zz)' the gettext \
>     version the package is using
>   autopoint: *** Stop.
>
> So m4 seems to be used earlier in the process,
> to parse the AN_GNU_GETTEXT_VERSION from configure.ac,
> before we have things setup for m4.

Thanks for the report.  It looks like another oversight in the tracing
code in autopoint.  Attaching a patch.

> Bug
>   http://savannah.gnu.org/bugs/?40083
> seems to be related, but the fix was to just discard stderr.

Yeah, maybe we should consider borrowing the tracing code from libtool.
That should be more reliable and avoid duplicate effort like this.

Regards,
-- 
Daiki Ueno

>From a76771d100be7c7a81c23c2e6b4a615c23aeaf79 Mon Sep 17 00:00:00 2001
From: Daiki Ueno <address@hidden>
Date: Tue, 19 Nov 2013 12:08:10 +0900
Subject: [PATCH] autopoint: disable m4_include when tracing

* autopoint.in (func_trace): Disable m4_include.
* gettextize.in (func_trace): Likewise.
Reported by Bernhard Voelker in
<https://lists.gnu.org/archive/html/bug-gettext/2013-11/msg00011.html>.
---
 gettext-tools/misc/ChangeLog     | 7 +++++++
 gettext-tools/misc/autopoint.in  | 3 ++-
 gettext-tools/misc/gettextize.in | 3 ++-
 3 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/gettext-tools/misc/ChangeLog b/gettext-tools/misc/ChangeLog
index 6ac8663..909e68f 100644
--- a/gettext-tools/misc/ChangeLog
+++ b/gettext-tools/misc/ChangeLog
@@ -1,3 +1,10 @@
+2013-11-19  Daiki Ueno  <address@hidden>
+
+       * autopoint.in (func_trace): Disable m4_include.
+       * gettextize.in (func_trace): Likewise.
+       Reported by Bernhard Voelker in
+       <https://lists.gnu.org/archive/html/bug-gettext/2013-11/msg00011.html>.
+
 2013-10-16  Daiki Ueno  <address@hidden>
 
        * autopoint.in: Use 'm4' as the default macro directory if it is
diff --git a/gettext-tools/misc/autopoint.in b/gettext-tools/misc/autopoint.in
index d8c7d74..0f8bbfb 100644
--- a/gettext-tools/misc/autopoint.in
+++ b/gettext-tools/misc/autopoint.in
@@ -141,12 +141,13 @@ fi
 func_trace ()
 {
   echo '\
-dnl ignore M4sugar macros which may abort autom4te
+dnl disable macros which may abort autom4te
 m4_undefine([m4_assert])
 m4_undefine([m4_fatal])
 m4_undefine([m4_warn])
 m4_undefine([m4_errprintn])
 m4_undefine([m4_exit])
+m4_undefine([m4_include])
 dnl macros which needs to be traced without aclocal.m4
 m4_define([AM_GNU_GETTEXT], [])
 m4_define([AM_GNU_GETTEXT_VERSION], [])
diff --git a/gettext-tools/misc/gettextize.in b/gettext-tools/misc/gettextize.in
index 9b0b465..bdc1089 100644
--- a/gettext-tools/misc/gettextize.in
+++ b/gettext-tools/misc/gettextize.in
@@ -141,12 +141,13 @@ fi
 func_trace ()
 {
   echo '\
-dnl ignore M4sugar macros which may abort autom4te
+dnl disable macros which may abort autom4te
 m4_undefine([m4_assert])
 m4_undefine([m4_fatal])
 m4_undefine([m4_warn])
 m4_undefine([m4_errprintn])
 m4_undefine([m4_exit])
+m4_undefine([m4_include])
 dnl macros which needs to be traced without aclocal.m4
 m4_define([AM_GNU_GETTEXT], [])
 m4_define([AM_GNU_GETTEXT_VERSION], [])
-- 
1.8.3.1