[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: diffutils test failure on nixos/hydra's solaris build
From: |
Ed Maste |
Subject: |
Re: diffutils test failure on nixos/hydra's solaris build |
Date: |
Mon, 1 Oct 2012 15:03:32 -0400 |
On 1 October 2012 13:48, Jim Meyering <address@hidden> wrote:
> Ed Maste wrote:
>
>> On 30 September 2012 17:39, Bruno Haible <address@hidden> wrote:
>>> Jim Meyering wrote on 2012-08-28:
>>>> FAIL: test-localeconv (exit: 262)
>>>> =================================
>>>>
>>>> test-localeconv.c:41: assertion failed
>>>>
>>>> which corresponds to this line:
>>>>
>>>> $ cat -n tests/test-localeconv.c|grep -B6 41
>>>> 35 {
>>>> 36 struct lconv *l = localeconv ();
>>>> 37
>>>> 38 ASSERT (STREQ (l->decimal_point, "."));
>>>> 39 ASSERT (STREQ (l->thousands_sep, ""));
>>>> 40 #if !defined __FreeBSD__
>>>> 41 ASSERT (STREQ (l->grouping, ""));
>>>>
>>>
>>> It's easy to work around the failure. Since that particular test
>>> is already exempted on FreeBSD, it's not a big deal to also disable
>>> it on Solaris 11.
>>
>> For FreeBSD specifically I'd rather conclude either that this is a bug
>> in our localeconv and fix it, or that more than one value for
>> l->grouping is permissible and change the test. I've raised a FreeBSD
>> pr for this: http://www.freebsd.org/cgi/query-pr.cgi?pr=172215 since
>> it looks to me like the most recent standards suggest grouping should
>> be "" rather than "\177".
>>
>> That said, the test passes for me with the following change:
>
> Thanks for the patch!
> Dare I ask? ;-) Did you try it on Solaris 11?
> If not, I will get to it.
Sorry, I meant to mention that - no, I did not test it on Solaris.
-Ed