Re: [PATCH 0/6] api.value.type support

[Akim Demaille]

Le 25 févr. 2013 à 14:26, Akim Demaille <address@hidden> a écrit :

> Actually I just found what could arguable called a bug in
> Bison:
> 
>> $ ls f.y
>> f.y
>> $ bison -o y.tab.c -d -v f.y
>> $ ls y.*
>> y.output     y.tab.c         y.tab.h
> 
> where I would have expected "y.tab.output".  Fortunately this
> is also what Automake/Autoconf expect.  So, if instead of -y
> Autoconf were using '-o y.tab.c', everybody would be happy.
> 
> And then --yacc = '-Werror=yacc -o y.tab.c'.  I'd be happy
> to avoid making --fixed-output-file public again, it's
> useless.
> 
> WDYT?

I'm installing this.  I will suggest a modification of AC_PROG_YACC,
and also suggest in Bison's NEWS that users of AC_PROG_YACC actually
define YACC='bison -o y.tab.c' in their configure.ac until they have
an updated Autoconf.

commit f1526426cddd0aaef7f54c583b5a8651f2dad482
Author: Akim Demaille <address@hidden>
Date:   Wed Feb 27 15:41:55 2013 +0100

    build: stop using bison -y
    
    * Makefile.am (YACC): Pass -o y.tab.c, so that ylwrap is happy, and
    yet we don't pass --yacc to bison.
    (AM_YFLAGS): Disable Yacc warnings.

diff --git a/Makefile.am b/Makefile.am
index 34edfc7..7a752d7 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -32,8 +32,8 @@ EXTRA_DIST = .prev-version .version \
 ## version of Bison nearby...
 BISON = $(top_builddir)/tests/bison
 BISON_IN = $(top_srcdir)/tests/bison.in
-YACC = $(BISON) -y
-AM_YFLAGS = -d -v --warnings=all,error --report=all
+YACC = $(BISON) -o y.tab.c
+AM_YFLAGS = -d -v -Werror -Wall -Wno-yacc --report=all
 
 # Initialization before completion by local.mk's.
 AM_CFLAGS = $(WARN_CFLAGS)