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: Rafal Luzynski <digitalfreak at lingonborough dot com>
- To: Pravin Satpute <psatpute at redhat dot com>, libc-alpha at sourceware dot org
- Date: Fri, 14 Apr 2017 02:08:23 +0200 (CEST)
- Subject: Re: [PATCH] localedata: Remove trailing spaces [BZ #20275]
- Authentication-results: sourceware.org; auth=none
- References: <410737595.37436.1488415167839@poczta.nazwa.pl> <2092611179.445981.1491608103329@poczta.nazwa.pl> <679567977.26602106.1492079105322.JavaMail.zimbra@redhat.com>
- Reply-to: Rafal Luzynski <digitalfreak at lingonborough dot com>
13.04.2017 12:25 Pravin Satpute <psatpute@redhat.com> wrote:
> >----- 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
Thank you, Pravin. Your review is exactly what I needed.
So, taking into account that:
- now we have *_IN, sq_AL, and quz_PE locales verified,
- it may be difficult to reach the native speakers of
the_NP, ne_NP, ln_CD, ht_HT, mt_MT, and so_*,
- all changes seem to be reasonable,
- in case of so_* this is only the change of the space character
encoding, there is no change in the output,
can anybody please commit this patch? Links above.
Best regards,
Rafal