Re: HEAD: Makefile woes (2/n)

[Ralf Wildenhues]

* Ralf Wildenhues wrote on Sun, Feb 05, 2006 at 06:26:11PM CET:
> 
> The most correct solution would be to have `tests/testsuite' depend on
> Makefile.am.  This is heavy in many unnecessary cases, however.

Applied like this, also fixing the "bogus output left after interrupting
make".

Cheers,
Ralf

        * Makefile.am ($(TESTSUITE)): Also depend on `Makefile.am'.
        Write to temporary file and rename.

Index: Makefile.am
===================================================================
RCS file: /cvsroot/libtool/libtool/Makefile.am,v
retrieving revision 1.191
diff -u -r1.191 Makefile.am
--- Makefile.am 5 Feb 2006 17:12:52 -0000       1.191
+++ Makefile.am 8 Feb 2006 22:25:58 -0000
@@ -427,9 +427,10 @@
 
 # Use `$(srcdir)' for the benefit of non-GNU makes: this is
 # how `testsuite' appears in our dependencies.
-$(srcdir)/$(TESTSUITE): $(srcdir)/tests/package.m4 $(TESTSUITE_AT)
+$(srcdir)/$(TESTSUITE): $(srcdir)/tests/package.m4 $(TESTSUITE_AT) Makefile.am
        cd $(srcdir)/tests && \
-       $(AUTOTEST) `echo $(TESTSUITE_AT) | sed 's,tests/,,g'` -o testsuite
+       $(AUTOTEST) `echo $(TESTSUITE_AT) | sed 's,tests/,,g'` -o testsuite.tmp 
&& \
+       mv -f testsuite.tmp testsuite
 
 $(srcdir)/tests/package.m4: $(srcdir)/configure.ac
        { \