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 s

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

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'): > > >

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-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 > >> > >>

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 after

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). >

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, s

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 > 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. > > > > Because, if I

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Paolo Bonzini
On Mon, Sep 19, 2011 at 18:39, Stefano Lattarini 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. > > Because, if I'm not reading it wrong (which I might be doing in fact, since > I'v

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

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: I really want the ACLOCAL_PAT

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, an

[PATCH] aclocal: handle ACLOCAL_PATH environment variable

2011-09-19 Thread Stefano Lattarini
ot; field of the git commit? Thanks, Stefano From e913815f1e074d358ee01ac774ea48f23f5c2133 Mon Sep 17 00:00:00 2001 Message-Id: From: Paolo Bonzini Date: Tue, 9 Nov 2010 20:14:38 +0100 Subject: [PATCH] aclocal: handle ACLOCAL_PATH environment variable * aclocal.in (parse_ACLOCAL_PATH): New funct

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-04 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Thu, Nov 04, 2010 at 10:03:27PM CET: > On Thursday 04 November 2010, Ralf Wildenhues wrote: > > * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 04:24:51PM CET: > > > Shouldn't we use `...@path_separator@' here instead of `:', for better > > > portability to windows? >

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{"ACLOCA

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-04 Thread Ralf Wildenhues
* 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"}) > > +{ > > + foreach my $dir (split /:/, $EN

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

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 bet

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

[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 pat