bug#10852: VPATH builds cannot recover from missing parser header

2020-05-28 Thread Karl Berry
Back on this bug from 2012, https://debbugs.gnu.org/cgi/bugreport.cgi?bug=10852 ak> Also, why two "if"? sl> For the sake of "make -n": * doc/automake.texi (Multiple Outputs): Split commands than reinvoke $(MAKE) to avoid file removals during dry runs. I pushed Akim's patch. Reg

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-24 Thread Stefano Lattarini
Hi Akim. On 02/24/2012 01:55 PM, Akim Demaille wrote: > > Le 20 févr. 2012 à 14:58, Stefano Lattarini a écrit : > >>> Also, why two "if"? >>> >> For the sake of "make -n": at least GNU make and Solaris make execute >> recipes containing the $(MAKE) string even when they are running in dry >> mod

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-24 Thread Akim Demaille
Le 20 févr. 2012 à 14:58, Stefano Lattarini a écrit : >> Also, why two "if"? >> > For the sake of "make -n": at least GNU make and Solaris make execute > recipes containing the $(MAKE) string even when they are running in dry > mode; so if we didn't break the recipe above in two invocations, the

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-23 Thread Akim Demaille
Le 23 févr. 2012 à 10:43, Stefano Lattarini a écrit : >> src/parse-gram.h: src/parse-gram.c >> test -f $@ || rm -f src/parse-gram.c >> test -f $@ || $(MAKE) $(AM_MAKEFLAGS) src/parse-gram.c >> > This seems nicer. Care to write a patch to implement this simplification > (here and for o

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-23 Thread Stefano Lattarini
On 02/22/2012 03:54 PM, Akim Demaille wrote: > hi Stefano, Hello World!\n > >> On 02/20/2012 02:24 PM, Akim Demaille wrote: src/parse-gram.h: src/parse-gram.c @if test ! -f $@; then rm -f src/parse-gram.c; else :; fi @if test ! -f $@; then $(MAKE) $(AM_MAKEFLAGS) src/pars

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-22 Thread Akim Demaille
hi Stefano, Hello World!\n Le 20 févr. 2012 à 14:58, Stefano Lattarini a écrit : > Hi Akim. > > On 02/20/2012 02:24 PM, Akim Demaille wrote: >>> src/parse-gram.h: src/parse-gram.c >>> @if test ! -f $@; then rm -f src/parse-gram.c; else :; fi >>> @if test ! -f $@; then $(MAKE) $(AM_MAKEFL

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-21 Thread Stefano Lattarini
On 02/21/2012 11:25 AM, Akim Demaille wrote: > > Le 20 févr. 2012 à 15:23, Akim Demaille a écrit : > >>> But the test is wrong, because it checks that the Yacc-generated .h and .c >>> files are placed in the $srcdir, while we expect them to be placed in the >>> $builddir. Do the tests added by y

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-21 Thread Akim Demaille
Le 20 févr. 2012 à 15:23, Akim Demaille a écrit : >> But the test is wrong, because it checks that the Yacc-generated .h and .c >> files are placed in the $srcdir, while we expect them to be placed in the >> $builddir. Do the tests added by your patch work for you? They don't work >> for me (as

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-20 Thread Akim Demaille
Le 20 févr. 2012 à 14:58, Stefano Lattarini a écrit : > Hi Akim. Hi Stefano, Thanks for the quick answer! > The following patch extends a test which is aimed at checking >> >> this, but does it in a non-vpath build :) >> > But the test is wrong, because it checks that the Yacc-generated .h a

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-20 Thread Stefano Lattarini
Hi Akim. On 02/20/2012 02:24 PM, Akim Demaille wrote: > I am having problems in Bison (current master) to recover > from a lost parse-gram.h, generated from parse-gram.y > with regular Automake (1.11.3) handling: > >> AM_YFLAGS = -d -v --warnings=all,error --report=all >> >> src_bison_SOURCES =

bug#10852: VPATH builds cannot recover from missing parser header

2012-02-20 Thread Akim Demaille
I am having problems in Bison (current master) to recover from a lost parse-gram.h, generated from parse-gram.y with regular Automake (1.11.3) handling: > AM_YFLAGS = -d -v --warnings=all,error --report=all > > src_bison_SOURCES = \ > ... > src/output.h