Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers

[Stefano Lattarini]

On 07/13/2012 03:19 PM, Akim Demaille wrote:
> 
> Le 13 juil. 2012 à 13:43, Stefano Lattarini a écrit :
> 
>>>>> * lib/ylwrap (rename_sed): New.
>>>>> (main loop): Use it the rename the dependencies to other files.
>>>>>
>>>> Oops, diving straight into the details, no rationale.  No good.
>>>
>>> Well, I believed the title to be clear enough, but ok.
>>>
>>>> As an outsider, it is not clear to me why this change is useful, nor
>>>> how it fits in the bigger picture of Bison development.
>>>
>>> It's not only Bison's future changes, it is that it is already
>>> wrong from all the other skeletons.
>>>
>> See?  Another thing I had got wrong, given my ignorance and the lack
>> of a proper explanation ;-)
> 
> I have an XPASS.
> 
> XPASS: t/yacc-bison-skeleton.sh
> 
> Guess what… It's about glr.c :)
> 
> I have many failures, I don't know if this is expected.
>
Not expected at all.  Could you post their list, and maybe also the
contents of 'test-suite.log'?

> I have pushed my updated patches to origin/yacc-work.
>
Thanks.  Until the issues are resolved, I suggest we consider this
branch as rewindable, so that we can fix its commits if the necessity
arise, instead of adding fixes on the top of them (thus muddying the
history).

Thanks,
  Stefano