automake-patches
[Top][All Lists]
Advanced

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

Re: [PATCH] {maint} Tests: fix requirement of m4 files


From: Stefano Lattarini
Subject: Re: [PATCH] {maint} Tests: fix requirement of m4 files
Date: Sun, 26 Sep 2010 12:46:56 +0200
User-agent: KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; )

On Sunday 26 September 2010, Ralf Wildenhues wrote:
> Hello Stefano,
> 
> * Stefano Lattarini wrote on Fri, Sep 24, 2010 at 01:00:02AM CEST:
> > While testing my recent patches, I found a nasty weakness in the
> > testsuite (a real weakness this time, not a theoretical one ;-).
> 
> I'm sorry, but I have to disagree with you on this one.
> 
> When you install Automake below some prefix, you have to take care
> yourself that it finds the third-party macros that you want it to
> find. Just like you have to take care that third-party tools can
> be found (by way of $PATH) etc.
True, but the testsuite might be run *before* the installation, when
a $prefix/share/aclocal doesn't still exist.
 
> We cannot take this burden off the user, we can only document it
> better. (Otherwise, it would be impossible to test, e.g., one of
> several Libtool installations, with its separate macro files and
> libtoolize programs.)
OK.
 
> Likewise, having the Automake testsuite search for third-party
> macros in $prefix/share/aclocal is only done because the "make
> install"ed tools would do the same: search there.  If they are not
> found there, then we simply don't run those tests.
> 
> "make distcheck" is very much in line with that, by intention.
But the current behaviour breaks "make distcheck"; and while the 
problem "no macros in $prefix/share/aclocal" can be easily worked 
around with the creation of a proper `dirlist' file, I see no such
workaround for "make distcheck".
 
> > Incidentally, the same problem can also happen to a user or
> > developer having working libtool and automake installations,
> > when he tests a new automake version configured with ${prefix}
> > != from the prefix of the pre-existing automake installation.
> 
> Yes.  Documenting what she needs to do then is the right way to go.
> 
> I don't want to burden the testsuite with having to know more about
> the setup details of third-party tools, or which macro files
> really are needed.  For example, just invoking the macros (without
> running libtoolize) may not be sufficient at all.
> That's a flawed approach IMVHO.
What do you suggest then that could fix the "make distcheck" breakage?
Maybe new configure options --with-{libtool,gettext}-m4-dir?  That has
been my first approach BTW, but I quickly dropped it to favour the 
more "automatized" behaviour of the patch currently under discussion.

Regards,
   Stefano



reply via email to

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