Re: [PATCH 1/2] aclocal: multiple local m4 macro dirs with AC_CONFIG_MACRO_DIR

2012-07-04 Thread Stefano Lattarini
Hi Eric. On 07/04/2012 06:42 PM, Eric Blake wrote: > On 07/04/2012 09:24 AM, Stefano Lattarini wrote: >> It can be done either passing several arguments to a single invocation: >> >> AC_CONFIG_MACRO_DIR([dir1 dir2]) > > I would recommend against this, > >> >> or issuing more invocations: >>

Re: [PATCH 1/2] aclocal: multiple local m4 macro dirs with AC_CONFIG_MACRO_DIR

2012-07-04 Thread Eric Blake
On 07/04/2012 09:24 AM, Stefano Lattarini wrote: > It can be done either passing several arguments to a single invocation: > > AC_CONFIG_MACRO_DIR([dir1 dir2]) I would recommend against this, > > or issuing more invocations: > > AC_CONFIG_MACRO_DIR([dir1]) > AC_CONFIG_MACRO_DIR([di

[PATCH 2/2] aclocal: diagnose non-existing directories in AC_CONFIG_MACRO_DIR better

2012-07-04 Thread Stefano Lattarini
This new implementation ensures that any directory (possibly excluding the first one, if the '--install' option is used) that is declared with AC_CONFIG_MACRO_DIR and that is non-existent will cause an error from aclocal. * aclocal.in (scan_m4_dirs): Add a new argument, telling whether it's OK for

[PATCH 1/2] aclocal: multiple local m4 macro dirs with AC_CONFIG_MACRO_DIR

2012-07-04 Thread Stefano Lattarini
It can be done either passing several arguments to a single invocation: AC_CONFIG_MACRO_DIR([dir1 dir2]) or issuing more invocations: AC_CONFIG_MACRO_DIR([dir1]) AC_CONFIG_MACRO_DIR([dir2]) This will allow projects to use several m4 macro local dirs. This is especially important fo

[PATCH 0/2] aclocal: allow multiple local m4 macro dirs with AC_CONFIG_MACRO_DIR

2012-07-04 Thread Stefano Lattarini
For master. Stefano Lattarini (2): aclocal: multiple local m4 macro dirs with AC_CONFIG_MACRO_DIR aclocal: diagnose non-existing directories in AC_CONFIG_MACRO_DIR better aclocal.in | 58 +- t/aclocal-macrodir.tap | 158 +

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Stefano Lattarini
On 07/04/2012 01:28 PM, Eric Blake wrote: > On 07/04/2012 05:03 AM, Eric Blake wrote: > >> Would it also make sense to allow multiple calls to AC_CONFIG_MACRO_DIR >> to stack, as in: >> >> AC_CONFIG_MACRO_DIR([dir1]) >> AC_CONFIG_MACRO_DIR([dir2]) >> >> And should we mention that the first directo

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Stefano Lattarini
On 07/04/2012 01:28 PM, Eric Blake wrote: > On 07/04/2012 05:03 AM, Eric Blake wrote: > >> Would it also make sense to allow multiple calls to AC_CONFIG_MACRO_DIR >> to stack, as in: >> >> AC_CONFIG_MACRO_DIR([dir1]) >> AC_CONFIG_MACRO_DIR([dir2]) >> >> And should we mention that the first directo

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Stefano Lattarini
On 07/04/2012 01:03 PM, Eric Blake wrote: > On 07/04/2012 04:55 AM, Stefano Lattarini wrote: >> This will allow projects to use several m4 macro local dirs. This is >> especially important for projects that are used as nested subpackages >> of larger projects. >> >> See also: >>

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Eric Blake
On 07/04/2012 05:03 AM, Eric Blake wrote: > Would it also make sense to allow multiple calls to AC_CONFIG_MACRO_DIR > to stack, as in: > > AC_CONFIG_MACRO_DIR([dir1]) > AC_CONFIG_MACRO_DIR([dir2]) > > And should we mention that the first directory listed has special > significance to other tools

Re: [PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Eric Blake
On 07/04/2012 04:55 AM, Stefano Lattarini wrote: > This will allow projects to use several m4 macro local dirs. This is > especially important for projects that are used as nested subpackages > of larger projects. > > See also: >

[PATCH 1/2] AC_CONFIG_MACRO_DIR: accept more than one argument

2012-07-04 Thread Stefano Lattarini
This will allow projects to use several m4 macro local dirs. This is especially important for projects that are used as nested subpackages of larger projects. See also:

[PATCH 2/2] docs: ACLOCAL_AMFLAGS will become obsolescent in Automake 1.13

2012-07-04 Thread Stefano Lattarini
See commit 'v1.12.1-165-gcd1a9cc' "aclocal: deprecate ACLOCAL_AMFLAGS, trace AC_CONFIG_MACRO_DIR instead" in the Automake git repository. See also follow-up discussion at: * doc/autoconf.texi (@node "Input" @defmac "AC_CON

Re: [PATCH] {master} aclocal: deprecate ACLOCAL_AMFLAGS, trace AC_CONFIG_MACRO_DIR instead

2012-07-04 Thread Stefano Lattarini
[Adding bug-autoconf] Reference: On 07/04/2012 11:23 AM, Stefano Lattarini wrote: > On 07/04/2012 11:03 AM, Ralf Corsepius wrote: >> >> Please revert this change. >> > No. Have you read my plan above? If the use case of

Re: [PATCH] {master} aclocal: deprecate ACLOCAL_AMFLAGS, trace AC_CONFIG_MACRO_DIR instead

2012-07-04 Thread Stefano Lattarini
On 07/04/2012 11:03 AM, Ralf Corsepius wrote: > On 07/03/2012 10:54 PM, Stefano Lattarini wrote: >> On 07/02/2012 02:57 PM, Stefano Lattarini wrote: >>> On 07/02/2012 02:26 PM, Ralf Corsepius wrote: ACLOCAL_AMFLAGS is able to take sequences of "-I's". Will your AC_CONFIG_MACRO_DIR b

Re: [PATCH] {master} aclocal: deprecate ACLOCAL_AMFLAGS, trace AC_CONFIG_MACRO_DIR instead

2012-07-04 Thread Ralf Corsepius
On 07/03/2012 10:54 PM, Stefano Lattarini wrote: On 07/02/2012 02:57 PM, Stefano Lattarini wrote: On 07/02/2012 02:26 PM, Ralf Corsepius wrote: On 06/30/2012 08:30 PM, Stefano Lattarini wrote: Maintaining ACLOCAL_AMFLAGS in the Makefile.am to pass extra flags to aclocal is (and have always bee