Re: [PATCH 0/3] ylwrap: handle header inclusion properly

[Akim Demaille]

Le 12 juil. 2012 à 17:38, Stefano Lattarini a écrit :

> Hi Akim.
> 
> On 07/12/2012 03:51 PM, Akim Demaille wrote:
>> The following patches address a bug in ylwrap that cause it
>> to be unable to handle Bison glr parsers, but also prevents
>> future Bison releases from also using header inclusion in yacc mode.
>> 
> Thanks!  A review of your patches will follow shortly.
> 
> Apart from that, on which systems can you test your series?  I think
> it should be tested at least on Solaris and a BSD system in addition
> to GNU/Linux.

I don't have these machines available, I just have my machine.
Don't you have a build farm somewhere in charge of this?

>> A related, but different, matter consists in using the -o
>> option when supported by yacc.
>> 
> I'm not going to work on that anytime soon (and anyway, I'm waiting
> to hear back from the Autoconf guys after this message):
> <http://lists.gnu.org/archive/html/autoconf/2012-06/msg00051.html>
> So feel free to give it a shot if you want to ;-)

I was not pointing at anyone here, just making clear that
even if we want to avoid using ylwrap, it does have to be
fixed.

>> BTW, there are plenty of files that should be git ignored.
>> Should I add them?
> In automake, we've renamed our autom4te cache to .autom4te.cache:
> 
>  $ cat .autom4te.cfg
>  ## autom4te.cfg for the Automake package.
>  ##
>  begin-language: "Autoconf-without-aclocal-m4"
>  args: --cache=.autom4te.cache
>  end-language: "Autoconf-without-aclocal-m4"

Funny, thanks :)