Re: Move the preselections

[Ralf Wildenhues]

Hello Stepan,

* Stepan Kasal wrote on Fri, Sep 08, 2006 at 02:38:41PM CEST:
> On Thu, Sep 07, 2006 at 10:58:45AM -0700, Paul Eggert wrote:
> > Stepan Kasal <address@hidden> writes:
> > 
> > >   * lib/autom4te.in (Autoconf-without-aclocal-m4): Move the
> > >   preselections ...
> > >   (Autoconf): ... here, and drop the Autoscan-preselections.
> > 
> > That looks OK to me; please apply.  Thanks.
> 
> since there were no protests,

Well, not a protest, but I still don't know why the first version of
your patch effectively made my test I wrote for Automake unreliable:
http://lists.gnu.org/archive/html/autoconf-patches/2006-09/msg00029.html

I would be very happy if you could explain that to me; I gather you've
understood this autom4te caching business much better than I have.

> I also deleted Autoscan-preselections

I don't think anybody OKed that.  I don't want to say that it's not OK,
nor that you should revert it -- I simply don't know whether it is or
not.

What I do think and have tested (with the coreutils package) is that,
while your patch makes one of the traces somewhat shorter, speedup (for
one autoreconf run) seems to be negligible if there is any at all.  But
presumably that's not what you went after(?).  (After all, if your
primary goal was optimization, you would have measured...  ;-)

Cheers,
Ralf

> 2006-09-08  Stepan Kasal  <address@hidden>
> 
>       * lib/autom4te.in (Autoconf-without-aclocal-m4): Move the
>       preselections ...
>       (Autoconf): ... here.
>       (Autoscan-preselections): Delete.