This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PING^2][PATCH] localedata: Remove trailing spaces [BZ #20275]


+1from me, FWIW

cjl

On Wed, Jun 7, 2017 at 5:40 AM, Rafal Luzynski
<digitalfreak@lingonborough.com> wrote:
> 30.05.2017 11:43 Rafal Luzynski <digitalfreak@lingonborough.com> wrote:
>>
>>
>> 02.03.2017 01:39 Rafal Luzynski <digitalfreak@lingonborough.com> wrote:
>> >
>> >
>> > Please find more details at:
>> > https://sourceware.org/bugzilla/show_bug.cgi?id=20275#c3
>> >
>> > Again, I'd like to hear some feedback from native speakers,
>> > for at least some of those patches.
>> >
>> > Regards,
>> >
>> > Rafal
>>
>> PING
>>
>> This patch is pending review.
>>
>> It changes 21 locale data files and so far we have confirmation
>> from the actual users of few of them that they are correct. It may
>> be difficult to get confirmation from other ones.
>>
>> On the other hand, they seem to look reasonable and in case of
>> so_* (Somali language, 4 variants) the changes are not visible
>> to the user, only fixing the source code format.
>>
>> If you prefer, you may consider accepting the patch for only
>> some locales while leaving other waiting.
>>
>> As always, the patch is applied to the binaries in my copr repo:
>> https://copr.fedorainfracloud.org/coprs/rluzynski/genitive/
>>
>> Regards,
>>
>> Rafal
>
>
> PING^2
>
> I'd like to remind that it is confirmed that hi_IN, quz_PE, and
> sq_AL are confirmed that they are correct, so_* do not introduce
> visible changes, and other just look reasonable.
>
> What else do we need to accept this patch?
>
> Do you prefer to accept it for few languages and leave open for
> others?
>
> Regards,
>
> Rafal


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]