New temporary, non-rewindable public branch `aclocal-search-path' (was: Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable)

2011-09-28 Thread Stefano Lattarini
On Saturday 24 September 2011, Stefano Lattarini wrote: [dropping bug-automake] The attached patch (for master) is the first step of the testsuite reorganization. I'd like to apply it to a new public branch `aclocal-search-path' (to be merged into master eventually). The patch will

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-24 Thread Stefano Lattarini
[dropping bug-automake] On Tuesday 20 September 2011, Stefano Lattarini wrote: On Tuesday 20 September 2011, Paolo Bonzini wrote: On 09/20/2011 02:09 PM, Stefano Lattarini wrote: Yeah, I think the problem is that in the normal search path (from `aclocal -I /foo -I /bar'): 1.

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-24 Thread Stefano Lattarini
On Tuesday 20 September 2011, Stefano Lattarini wrote: On Tuesday 20 September 2011, Stefano Lattarini wrote: We definitely want to clean this all up in 1.12 -- code, documentation and testsuite as well. For the moment, I'd like to push this follow-up patch to maint as well, to smooth

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Stefano Lattarini
On Monday 19 September 2011, Paolo Bonzini wrote: On Mon, Sep 19, 2011 at 18:39, Stefano Lattarini stefano.lattar...@gmail.com wrote: Just to be sure, you are not including http://lists.gnu.org/archive/html/automake-patches/2010-11/msg00091.html are you? No, not yet at least.

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Paolo Bonzini
On 09/20/2011 12:51 PM, Stefano Lattarini wrote: Yes (and that's on purpose: ACLOCAL_PATH is typically used by a user which has newer versions of libraries installed, so they need their directories to override everything). Yes, this might be desirable. But then, for consistency sake,

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Stefano Lattarini
On Tuesday 20 September 2011, Paolo Bonzini wrote: On 09/20/2011 12:51 PM, Stefano Lattarini wrote: Yes (and that's on purpose: ACLOCAL_PATH is typically used by a user which has newer versions of libraries installed, so they need their directories to override everything). Yes,

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Paolo Bonzini
On 09/20/2011 02:09 PM, Stefano Lattarini wrote: Yeah, I think the problem is that in the normal search path (from `aclocal -I /foo -I /bar'): 1. `/foo' 2. `/bar' 3. ACDIR-APIVERSION 4. ACDIR The directories from dirlist and ACLOCAL_PATH should go after (3), rather than

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Stefano Lattarini
On Tuesday 20 September 2011, Paolo Bonzini wrote: On 09/20/2011 02:09 PM, Stefano Lattarini wrote: Yeah, I think the problem is that in the normal search path (from `aclocal -I /foo -I /bar'): 1. `/foo' 2. `/bar' 3. ACDIR-APIVERSION 4. ACDIR The

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-20 Thread Stefano Lattarini
On Tuesday 20 September 2011, Stefano Lattarini wrote: We definitely want to clean this all up in 1.12 -- code, documentation and testsuite as well. For the moment, I'd like to push this follow-up patch to maint as well, to smooth out the transition a bit. I will push this patch and the

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Paolo Bonzini
On 09/19/2011 06:05 PM, Stefano Lattarini wrote: I'll push the attached patch in 72 hours if there is no review by then. Paolo, since it's you who have written the previous version of this patch, from which I've drawn heavily, are you ok to be listed as the main author in the ChangeLog entry,

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Paolo Bonzini
On 09/19/2011 06:05 PM, Stefano Lattarini wrote: Resurrecting the old thread Add ACLOCAL_PATH to aclocal; references: http://lists.gnu.org/archive/html/automake-patches/2010-11/msg00089.html http://thread.gmane.org/gmane.comp.sysutils.automake.patches/4972 I really want the ACLOCAL_PATH

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Stefano Lattarini
[CC:ing automake bug 9026, which I should have done right away] On Monday 19 September 2011, Paolo Bonzini wrote: On 09/19/2011 06:05 PM, Stefano Lattarini wrote: Resurrecting the old thread Add ACLOCAL_PATH to aclocal; references:

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-04 Thread Stefano Lattarini
On Thursday 04 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 04:24:51PM CET: On Tuesday 02 November 2010, Paolo Bonzini wrote: + # Add any directory listed in the `ACLOCAL_PATH' environment + # variable. + if (defined $ENV{ACLOCAL_PATH})

[PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Paolo Bonzini
Hi all, this patch provides a fourth scheme of adding directories to the search path. It is a simple colon-separated list of directories (without globbing). It is useful when you're using a global installation of Automake but you want to add directories from your account as well to the search

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Stefano Lattarini
On Tuesday 02 November 2010, Paolo Bonzini wrote: Hi all, this patch provides a fourth scheme of adding directories to the search path. It is a simple colon-separated list of directories (without globbing). It is useful when you're using a global installation of Automake but you want to

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Paolo Bonzini
On 11/03/2010 04:24 PM, Stefano Lattarini wrote: + # Add any directory listed in the `ACLOCAL_PATH' environment + # variable. + if (defined $ENV{ACLOCAL_PATH}) +{ + foreach my $dir (split /:/, $ENV{ACLOCAL_PATH}) Shouldn't we use `...@path_separator@' here instead of `:', for better

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-03 Thread Stefano Lattarini
On Wednesday 03 November 2010, Paolo Bonzini wrote: On 11/03/2010 04:24 PM, Stefano Lattarini wrote: + # Add any directory listed in the `ACLOCAL_PATH' environment + # variable. + if (defined $ENV{ACLOCAL_PATH}) +{ + foreach my $dir (split /:/, $ENV{ACLOCAL_PATH})