Re: AM_INTL_SUBDIR

[Paul Eggert]

Bruno Haible <address@hidden> writes:

> In bison/bootstrap I can see the following code:
>
>   echo "$0: patching m4/gettext.m4 so that AM_INTL_SUBDIR is empty ..."
>   sed '
>     /^AC_DEFUN(\[AM_INTL_SUBDIR],/,/^]/c\
>       AC_DEFUN([AM_INTL_SUBDIR], [])
>   ' m4/gettext.m4 >m4/gettext_gl.m4 || exit
>
> Why is this necessary? bison does not include the intl directory, so this
> macro is unused. With 'aclocal' from version 1.8 or newer, unused macros
> don't drag in other macros any more. Any other problem I'm overlooking?

No, it's just old cruft that predates 1.8.  Thanks for reporting it.
I installed this:

2005-09-08  Paul Eggert  <address@hidden>

        * bootstrap: Remove AM_INTL_SUBDIR hack; no longer needed with
        recent versions of Automake.  Reported by Bruno Haible.

Index: bootstrap
===================================================================
RCS file: /cvsroot/bison/bison/bootstrap,v
retrieving revision 1.24
retrieving revision 1.25
diff -p -u -r1.24 -r1.25
--- bootstrap   25 Aug 2005 18:49:57 -0000      1.24
+++ bootstrap   8 Sep 2005 18:01:54 -0000       1.25
@@ -154,12 +154,6 @@ for gnulib_file in $gnulib_files; do
   cp -p $GNULIB_SRCDIR/$gnulib_file $dest || exit
 done
 
-echo "$0: patching m4/gettext.m4 so that AM_INTL_SUBDIR is empty ..."
-sed '
-  /^AC_DEFUN(\[AM_INTL_SUBDIR],/,/^]/c\
-    AC_DEFUN([AM_INTL_SUBDIR], [])
-' m4/gettext.m4 >m4/gettext_gl.m4 || exit
-
 
 # Get translations.