Avoid a race condition that would make parallel "distclean" fail.

* tests/Makefile.am (distclean-generic): Replace the default,
automake-provided rule with an identical one, but with an additional
dependency on distclean-local.  Simply adding the dependency would
cause automake not to emit the rule at all.
* BUGS: Building with -jN works, now.

diff --git a/BUGS b/BUGS
index 747de331ad62ee8946357ebbf6b45f5e23afe6d6..1978dd19c7fa1c6f96224710fdf23539d9bfecbf 100644 (file)
--- a/BUGS
+++ b/BUGS
@@ -3,7 +3,7 @@
 This file lists the bugs you must be aware of.  Be sure to check this
 file before using Autoconf, and especially CVS versions of Autoconf.
 
-       Copyright (C) 2000, 2001, 2002, 2006 Free Software Foundation, Inc.
+       Copyright (C) 2000-2002, 2006-2007 Free Software Foundation, Inc.
 
        This program is free software: you can redistribute it and/or modify
        it under the terms of the GNU General Public License as published by
@@ -29,7 +29,3 @@ and use with caution an Autoconf with ``Important bugs''.
                     `--------------------------*/
 
 * No known serious or important bugs.
-
-* Minor Problems
-
-Parallel builds via `make -jN' do not work.

diff --git a/ChangeLog b/ChangeLog
index f77a97d07ea422447aa205460bdb06748144a5f3..8988e8d4283fbe7daf8bedd3c6820a660a61b6b5 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2007-11-10  Jim Meyering  <meyering@redhat.com>
 
+       Avoid a race condition that would make parallel "distclean" fail.
+       * tests/Makefile.am (distclean-generic): Replace the default,
+       automake-provided rule with an identical one, but with an additional
+       dependency on distclean-local.  Simply adding the dependency would
+       cause automake not to emit the rule at all.
+       * BUGS: Building with -jN works, now.
+
        Distribute git-version-gen.
        * Makefile.am (EXTRA_DIST): Add build-aux/git-version-gen,
        since GNUmakefile is distributed, and requires it for dist* rules.

diff --git a/tests/Makefile.am b/tests/Makefile.am
index ed129fd84f117bbdbae8fad877b4142bed4e5b36..e069a0a431c7c9d5504084276e53803611fc6a75 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -120,6 +120,15 @@ $(TESTSUITE): $(srcdir)/package.m4 \
 atconfig: $(top_builddir)/config.status
        cd $(top_builddir) && ./config.status tests/$@
 
+# Avoid a race condition that would make parallel "distclean" fail.
+# The rule in clean-local tests for existence of $(TESTSUITE), and
+# if found, attempts to run it.  But the distclean-generic rule may
+# be running in parallel, and it removes $(DISTCLEANFILES) which
+# includes $(TESTSUITE).
+distclean-generic: clean-local
+       -test -z "$(CONFIG_CLEAN_FILES)" || rm -f $(CONFIG_CLEAN_FILES)
+       -test -z "$(DISTCLEANFILES)" || rm -f $(DISTCLEANFILES)
+
 clean-local:
        test ! -f $(TESTSUITE) || $(SHELL) $(TESTSUITE) --clean
        rm -f *.tmp