This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] localedata: Remove trailing spaces [BZ #20275]
- From: Pravin Satpute <psatpute at redhat dot com>
- To: Rafal Luzynski <digitalfreak at lingonborough dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Thu, 13 Apr 2017 06:25:05 -0400 (EDT)
- Subject: Re: [PATCH] localedata: Remove trailing spaces [BZ #20275]
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx10.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx10.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=psatpute at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 0D099624BE
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 0D099624BE
- References: <410737595.37436.1488415167839@poczta.nazwa.pl> <2092611179.445981.1491608103329@poczta.nazwa.pl>
>----- Original Message -----
>From: "Rafal Luzynski" <digitalfreak@lingonborough.com>
>To: libc-alpha@sourceware.org
>Sent: Saturday, April 8, 2017 5:05:03 AM
>Subject: Re: [PATCH] localedata: Remove trailing spaces [BZ #20275]
>
>Ping! Is there anybody able to review this patch?
>
>https://sourceware.org/ml/libc-alpha/2017-03/msg00029.html
Hi Rafal,
I have reviewed this patch today and it looks good to me. I have already provided comments on bugzilla [1]
Again adding it here.
- Testing installing copr repo, its working fine for Indian locales.
- Applied patch and generated all locales, its working fine.
- Verified patch, it only removes only <U0020> from locales and not creating any other typo kind mistake or unnecessary removal.
Best Regards,
Pravin Satpute
1. https://sourceware.org/bugzilla/show_bug.cgi?id=20275
>
>Regards,
>
>Rafal
>
>
>2.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