Re: _AC_COMPILER_EXEEXT cleanup

[Ralf Wildenhues]

Hello again,

* Ralf Wildenhues wrote on Thu, Sep 08, 2005 at 09:25:02AM CEST:
> 

That was me.  :-/

> * Paul Eggert wrote on Thu, Sep 08, 2005 at 01:09:43AM CEST:
> > Stepan Kasal <address@hidden> writes:
> > > 2005-08-31  Stepan Kasal  <address@hidden>
> > >
> > >   * lib/autoconf/lang.m4 (_AC_COMPILER_EXEEXT_DEFAULT): Don't try to
> > >     detect exeext, it will be done by _AC_COMPILER_EXEEXT_O; just set
> > >     ac_file top the name of the default output file.  Move the definition
> > >     of ac_files and the initial `rm' of the candidate files...
> > >   (_AC_COMPILER_EXEEXT): ... here and simplify them.  Moreover, use
> > >     the same list in subsequent `rm' calls, and for the temporary
> > >     redefinition of ac_clean_files.
> > >   (_AC_COMPILER_EXEEXT_O): Don't export ac_cv_exeext, it's not needed (or
> > >     no longer needed) by libtool.  Make it a cache check.
> > >   (_AC_COMPILER_EXEEXT_CROSS): Remove the comment, it was obviously
> > >     copied here by mistake.
> > 
> > I didn't see any problem with this, and nobody else has commented on it.

> Guess I (or whoever) will just speak up when I find something broken. :)

Well, I found something, after having this patch installed for a while.
As I am not 100% sure, I also copied bug-automake, in case.

With this patch in CVS Autoconf, CVS Automake fails the tests
  subdir5.test pr300-lib.test distname.test
(at least).  This is due to `configure' removing a.c, which looks like a 
bug in this patch to me.

Cheers,
Ralf