Re: bootstrapping failure

[Joel E. Denny]

On Fri, 5 Mar 2010, Eric Blake wrote:

> On 02/25/2010 08:40 AM, Joel E. Denny wrote:
> > On Wed, 24 Feb 2010, Eric Blake wrote:
> > 
> >> touch src/parse-gram.[ch]
> >>
> >> so that they did not have the same timestamp as src/parse-gram.y; that
> >> way make no longer tried to regenerated them with a missing src/bison.
> >>
> >> Is this step something that should be folded into bootstrap?
> > 
> > That would be fine with me, but it would be nice if there were a feature 
> > of gnulib's bootstrap that we could configure with bison's bootstrap.conf. 
> 
> bootstrap.conf is allowed to redefine the shell function
> bootstrap_epilogue, which is a perfect place to stick in this particular
> functionality.

Thanks.  I pushed this to branch-2.4.3, branch-2.5, and master.

>From b733bcd00e6fc8db45c86dc289938a28c1bf72d8 Mon Sep 17 00:00:00 2001
From: Joel E. Denny <address@hidden>
Date: Sun, 25 Apr 2010 17:44:43 -0400
Subject: [PATCH] build: don't require src/bison during bootstrap.

Suggested by Eric Blake at
<http://lists.gnu.org/archive/html/bug-bison/2010-03/msg00003.html>.
* bootstrap.conf (bootstrap_epilogue): New function to make sure
src/parse-gram.[ch] are stamped later than src/parse-gram.y.
---
 ChangeLog      |    8 ++++++++
 bootstrap.conf |    7 +++++++
 2 files changed, 15 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 98d8eaf..18b0946 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2010-04-25  Joel E. Denny  <address@hidden>
+
+       build: don't require src/bison during bootstrap.
+       Suggested by Eric Blake at
+       <http://lists.gnu.org/archive/html/bug-bison/2010-03/msg00003.html>.
+       * bootstrap.conf (bootstrap_epilogue): New function to make sure
+       src/parse-gram.[ch] are stamped later than src/parse-gram.y.
+
 2010-04-11  Joel E. Denny  <address@hidden>
 
        * NEWS (2.4.3): Mention fix for Sun Studio C++.
diff --git a/bootstrap.conf b/bootstrap.conf
index fc5a0e9..c8993fe 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -65,3 +65,10 @@ excluded_files='
 # In directories like lib/, m4/, and po/, generate both these files because we
 # still try to support CVS emulation.
 vc_ignore='.cvsignore .gitignore'
+
+bootstrap_epilogue()
+{
+  # Make sure we don't need src/bison, which usually doesn't exist at
+  # the time of a bootstrap.
+  touch src/parse-gram.[ch]
+}
-- 
1.5.4.3