Re: [bug-gettext] AC_CONFIG_MACRO_DIR{,S} & ACLOCAL_AMFLAGS

[Pavel Raiskup]

On Mon, 2013-03-04 at 18:43 +0900, Daiki Ueno wrote:
> Daiki Ueno <address@hidden> writes:
>
>> Pavel Raiskup <address@hidden> writes:
>>
>>>   autom4te --language Autoconf-without-aclocal-m4
>>> --trace=AC_CONFIG_MACRO_DIR configure.ac
>>>
>>>   // and for the up2date autoconf (not yet released) should be used (See
>>>   // autoconf's manual from git)
>>>   autom4te --language Autoconf-without-aclocal-m4
>>> --trace=AC_CONFIG_MACRO_DIR_TRACE configure.ac
>>>
>>> Are there any problems to be dependant on autoconf because of the
>>> 'autom4te' utility?  There is a lot of tracing done in gettext.
>>> (Note that this is not a blocker.)
>>
>> Well, I'm not sure :) but this 'autom4te' usage looks very helpful.  So
>> let's refactor using it, after installing this patch.
>
> Okay, I'm attaching a patch to use 'autom4te' and a patch for autopoint.
> Testing and comments would be appreciated.

Ad the 'autom4te' - this seems to be much better and cleaner to me!
Thanks.  One nit from me would be that the name of $autoconf variable is
not luckily chosen because it is not running autoconf.  I would chose
something like $tracer (or $autom4te)?  But it is a nit - don't see any
blocker in these patches.

I have tested it in a standard use-case and autopoint placed all needed
m4s to the AC_CONFIG_MACRO_DIR{,S}'s specified directory.

Pavel