Re: [PATCH] tests: fix broken quoting in the generated yacc grammar

[Stefano Lattarini]

On 02/18/2012 12:11 AM, Peter Rosin wrote:
> Stefano Lattarini skrev 2012-02-17 23:28:
>> On 02/17/2012 10:15 PM, Peter Rosin wrote:
>>> Fixes problem introduced by v1.11-1912-ge0906b7 "tests: avoid
>>> spurious failure with non-bison yacc" from 12-02-2012.
>>>
>>> * tests/suffix10.tap (Makefile.am): Avoid outer single quotes
>>> when we need to produce single quotes.
>>> ---
>>>  tests/suffix10.tap |    2 +-
>>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> Hi!
>>>
>>> I stumbled on what seems like a trivial problem.
>>>
>> Oops, my bad.  I've introduced this bug while recently tackling
>> a portability weakness in the test case (see commit e0906b7e).
> 
> It's already referenced in the commit message :-)
>
Oops, the late hour had its effects :-)

>>> But I'm not very fluent in yacc, so I thought I'd better ask
>>> if this is the right fix...  Is it?  Ok for master?
>>>
>> Your fix looks good.  ACK for master, and thanks.
> 
> Pushed, let me know if I shouldn't branch of from the "offending"
> commit in the future.
>
Usually that is recommended, but given the recent flurry of activity
and fixlets in master, I don't think it was really worth doing in this
case.

Thanks,
  Stefano