automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCHES] Bootstrap: Allow user overriding of $AUTOCONF and $PERL.


From: Ralf Wildenhues
Subject: Re: [PATCHES] Bootstrap: Allow user overriding of $AUTOCONF and $PERL.
Date: Sun, 8 Aug 2010 12:19:28 +0200
User-agent: Mutt/1.5.20 (2010-04-22)

* Stefano Lattarini wrote on Sun, Aug 08, 2010 at 12:09:52PM CEST:
> I squashed the following into the first patch:

> It's OK for you?

Sure.

> > > (BTW, if my understanding is correct, this would make a nice
> > > addition to HACKING, too).
> > 
> > Yep.
> Attached is a follow-up patch that does that.  Comments and criticism
> are welcome.

OK with typos fixed.

> P.S. I'll fix the dates in the ChangeLog entried before pushing.

Fine with me.  If you have all your patches in branches or otherwise
ordered, you can also just wait until I'm done reviewing and commit
or merge them in your preferred order.  Whatever seems easier for you.

Also, regenerating the tree with Autoconf 2.67 and committing that
separately is preapproved for maint.

Thanks,
Ralf

> Subject: [PATCH] Bootstrap: updated HACKING entry.
> 
> * HACKING ("Working with git"): Tell how to override the autoconf

I think  s/Tell/Explain/  is the right term.

> and autom4te used by the bootstrap process.

s/used/programs &/  ?

> --- a/HACKING
> +++ b/HACKING
> @@ -96,7 +96,12 @@
>    tree, so the resulting files (aclocal.m4 and Makefile.in) should
>    be the same as you would get if you install this version of
>    automake and use it to generate those files.  Be sure to have the
> -  latest stable version of Autoconf installed.
> +  latest stable version of Autoconf installed.  If such version is
> +  not installed as the default "autoconf" command, you'll have to

I think more common would be
  s/the default "autoconf" command/"autoconf"/
  s/you'll have to//

> +  pass it explicitly (along with the accompanying "autom4te") when
> +  calling `bootstrap' and `configure'.  For example:
> +    $ AUTOCONF=autoconf2.65 AUTOM4TE=autom4te2.65 ./boostrap

s/boo/&t/

> +    $ ./configure AUTOCONF=autoconf2.65 AUTOM4TE=autom4te2.65
>  
>  * Dependent files aclocal.m4, configure and Makefile.in in all
>    directories should be up to date in the git repository, so that




reply via email to

[Prev in Thread] Current Thread [Next in Thread]