Re: [PATCH] localedata: Convert French language locales (fr_*) to UTF_8

[Frederic Berat <fberat@redhat.com>]

Perfect then, thanks.

If it's worth anything:

Reviewed-by: Frédéric Bérat <fberat@redhat.com>

On Wed, Aug 17, 2022 at 11:06 AM Florian Weimer <fweimer@redhat.com> wrote:
>
> * Frederic Berat:
>
> > Although I'm not an expert for CA and BE writing convention, I'd
> > recommend to double check the following:
> >
> > For CA:
> > thousand_sep
> >
> > I note that the "mon_thousand_sep" was "<U202F>" which becomes " " as
> > expected, but there is no visible change on "thousand_sep" in this
> > patch. That may be expected, but is worth verifying.
>
> fr_CA copies the data from fr_FR, so it inherits the change there:
>
> | LC_NUMERIC
> | copy "fr_FR"
> | END LC_NUMERIC
>
> > For BE:
> > mon_thousand_sep
> > thousand_sep
> >
> > None of these separators seem to appear in this patch, that may need a
> > double check.
>
> fr_BE uses ".", which is ASCII; no change required.
>
> Thanks,
> Florian
>