Re: [GNU Bison 3.0.4] Problem in testsuite 191

[Ferdinand Thiessen]

Hi,
Sorry for late response.
The patch works, but now some working tests are skipped:

187: Prologue syncline                               skipped
(synclines.at:175)
188: %union syncline                                 skipped
(synclines.at:193)
189: %union name syncline                            skipped
(synclines.at:233)
190: Postprologue syncline                           skipped
(synclines.at:243)
191: Action syncline                                 skipped
(synclines.at:267)
192: Epilogue syncline                               skipped
(synclines.at:286)
193: %code top syncline                              skipped
(synclines.at:303)

----------------------
463 tests were successful.
30 tests were skipped.


But I do not know a better way for fixing this, unless the posix issue
#663 is fixed.

Am 12.02.2015 um 10:44 schrieb Akim Demaille:
> Hi Ferdinand,
>
>> Le 10 févr. 2015 à 11:19, Akim Demaille <address@hidden> a écrit :
>>
>> Hi Paul, hi Ferdinand,
>>
>> FWIW, the testsuite does enforce LC_ALL=C.
>>
>> In the present case, I'm unsure about the suggested patch,
>> since it uses UTF-8, which might behave unexpectedly on other
>> platforms I guess.
>>
>> So I'd rather go for something like the following patch:
>>
>> commit 8b35712ae3e56e2f1094eb2afb29b0b4dacdb230
>> Author: Akim Demaille <address@hidden>
>> Date:   Tue Feb 10 11:12:56 2015 +0100
>>
>>    tests: be robust to platforms that support UTF-8 even with LC_ALL=C
>>
>>    Because musl supports UTF-8 with LC_ALL=C, gcc produces:
>>
>>      input.y: In function ‘yyparse’:
>>
>>    instead of:
>>
>>      input.y: In function 'yyparse':
>>
>>    Reported by Ferdinand Thiessen.
>>    http://lists.gnu.org/archive/html/bug-bison/2015-02/msg00001.html
>>
>>    * tests/synclines.at (AT_SYNCLINES_COMPILE): Skip syncline tests when
>>    we can't trust error messages issued about a function body.
> Could you tell me if the patch works for you?  You may also try this
> tarball if that's easier for you:
>
> http://www.lrde.epita.fr/~akim/private/bison-3.0.4.4-a668.tar.gz
>
>