From: Don Armstrong <don@donarmstrong.com>
Date: Tue, 31 Dec 2013 00:41:06 +0000 (-0800)
Subject: Fix bison build failures with 42cb7d44
X-Git-Tag: debian/2.16.2-3~2
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=3a96e9fa2307c3359f51c02ada7da900d9d1c491;p=lilypond.git

Fix bison build failures with 42cb7d44
---

diff --git a/debian/changelog b/debian/changelog
index 88f069e580..eb7c0d61de 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,7 @@ lilypond (2.16.2-3) UNRELEASED; urgency=low
   * Breaks/Replaces lilypond-doc in html packages to stop problems on
     wheezy upgrade. (Closes: #718013)
   * Fix freetype path by applying patch from upstream (Closes: #733399)
+  * Fix bison build failures with 42cb7d44
 
  -- Don Armstrong <don@debian.org>  Mon, 11 Nov 2013 09:57:06 -0800
 
diff --git a/debian/patches/fix_bison_parser_generation_rules b/debian/patches/fix_bison_parser_generation_rules
new file mode 100644
index 0000000000..dc0c63edab
--- /dev/null
+++ b/debian/patches/fix_bison_parser_generation_rules
@@ -0,0 +1,63 @@
+From 42cb7d446827dce32a42b5b243bda522518b3d46 Mon Sep 17 00:00:00 2001
+From: Julien Rioux <julien.rioux@gmail.com>
+Date: Mon, 2 Sep 2013 10:41:50 +0200
+Subject: [PATCH] Keep bison-generated files in sync.
+
+Use a single rule to build header and source files from
+the syntax file, thus avoiding any possibility to have
+`make' rebuild one but not the other, which might happen
+if the syntax and target files have almost the same but
+slightly differing timestamps.
+
+See http://www.mail-archive.com/bug-lilypond@gnu.org/msg37081.html
+
+Fixes issue 3528: Keep bison-generated files in sync.
+http://code.google.com/p/lilypond/issues/detail?id=3528
+---
+ stepmake/stepmake/c++-rules.make |  9 ++-------
+ stepmake/stepmake/c-rules.make   | 10 ++--------
+ 2 files changed, 4 insertions(+), 15 deletions(-)
+
+diff --git a/stepmake/stepmake/c++-rules.make b/stepmake/stepmake/c++-rules.make
+index a368f4f..8de00bc 100644
+--- a/stepmake/stepmake/c++-rules.make
++++ b/stepmake/stepmake/c++-rules.make
+@@ -12,13 +12,8 @@ $(outdir)/%.lo: %.cc
+ $(outdir)/%.lo: $(outdir)/%.cc
+ 	$(DO_LO_DEP) $(CXX) -c $(ALL_CXXFLAGS) $(PIC_FLAGS) -o $@ $<
+ 
+-$(outdir)/%.cc: %.yy
+-	$(BISON) -o $@  $<
+-
+-$(outdir)/%.hh: %.yy
+-	$(BISON) -o $(subst .hh,-tmp.cc,$@) -d  $<
+-	rm $(subst .hh,-tmp.cc,$@)
+-	mv $(subst .hh,-tmp.hh,$@) $@
++$(outdir)/%.cc $(outdir)/%.hh: %.yy
++	$(BISON) -d -o $(outdir)/$*.cc $<
+ 
+ $(outdir)/%.cc: %.ll
+ 	$(FLEX) -Cfe -p -p -o$@ $<
+diff --git a/stepmake/stepmake/c-rules.make b/stepmake/stepmake/c-rules.make
+index 620275b..954e7a3 100644
+--- a/stepmake/stepmake/c-rules.make
++++ b/stepmake/stepmake/c-rules.make
+@@ -12,14 +12,8 @@ $(outdir)/%.lo: %.c
+ $(outdir)/%.lo: %.c
+ 	$(DO_LO_DEP) $(CC) -c $(ALL_CFLAGS) $(PIC_FLAGS) -o $@ $<
+ 
+-$(outdir)/%.c: %.y
+-	$(BISON) $<
+-	mv $(*F).tab.c $@
+-
+-$(outdir)/%.h: %.y
+-	$(BISON) -d $<
+-	mv $(*F).tab.h $@
+-	rm -f $(*F).tab.c # if this happens in the wrong order it triggers recompile of the .cc file
++$(outdir)/%.c $(outdir)/%.h: %.y
++	$(BISON) -d -o $(outdir)/$*.c $<
+ 
+ $(outdir)/%.c: %.l
+ 	$(FLEX) -Cfe -p -p -o$@ $<
+-- 
+1.8.4.rc3
diff --git a/debian/patches/series b/debian/patches/series
index a329a4a0b9..9e37f9f0e1 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 fix_freetype_inclusions
+fix_bison_parser_generation_rules