Re: [PATCH] Fix "make distcheck".

[Akim Demaille]

Le 4 août 09 à 22:31, Joel E. Denny a écrit :

> I pushed this to branch-2.4.2, branch-2.5, and master.

I also needed the following.  I don't understand why you didn't: there  
was no reason for the stamp file to be shipped, so it should be  
created during the "make" step of distcheck, and it should fail since  
it tries to write in the srcdir, which is readonly at that moment.

From 67af719840616ed07ca120b8255a998f4533ce17 Mon Sep 17 00:00:00 2001
From: Akim Demaille <address@hidden>
Date: Wed, 1 Jul 2009 12:19:18 +0200
Subject: [PATCH] distcheck: fix.

        * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp.
---
 ChangeLog                   |    6 ++++++
 examples/calc++/Makefile.am |    5 ++++-
 2 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 25bd09c..73fa402 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-08-12  Akim Demaille  <address@hidden>
+
+       distcheck: fix.
+
+        * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp.
+
 2009-08-10  Joel E. Denny  <address@hidden>

        * tests/local.mk (TESTSUITE_AT): Add named-refs.at.
diff --git a/examples/calc++/Makefile.am b/examples/calc++/Makefile.am
index 5e800c2..d99a7dc 100644
--- a/examples/calc++/Makefile.am
+++ b/examples/calc++/Makefile.am
@@ -54,7 +54,10 @@ MAINTAINERCLEANFILES = $(srcdir)/*.stamp $ 
(BUILT_SOURCES)
 # Compile the parser and save cycles.
 # This code comes from "Handling Tools that Produce Many Outputs",
 # from the Automake documentation.
-EXTRA_DIST = $(srcdir)/calc++-parser.stamp $(srcdir)/calc++-parser.yy
+EXTRA_DIST =                                   \
+  $(srcdir)/calc++-parser.stamp                        \
+  $(srcdir)/calc++-parser.yy                   \
+  $(srcdir)/calc.stamp
 # Don't depend on $(BISON) otherwise we would rebuild these files
 # in srcdir, including during distcheck, which is forbidden.
 $(srcdir)/calc++-parser.stamp: $(srcdir)/calc++-parser.yy $(BISON_IN)
--
1.6.4