[OE-core] [PATCH 1/2] systemd: change default locale from C.UTF-8 to C

[Andreas Müller]

On Wed, Apr 17, 2019 at 3:06 AM Changqing Li <changqing.li at windriver.com> wrote:
>
>
> On 4/16/19 6:22 PM, Richard Purdie wrote:
> > On Tue, 2019-04-16 at 17:49 +0800, changqing.li at windriver.com wrote:
> >> From: Changqing Li <changqing.li at windriver.com>
> >>
> >> current default locale is set to C.UTF-8, but glibc not support
> >> locale C.UTF-8. so set to the default locale C.
> >>
> >> [snip]
> >>          if not meson.is_cross_build()
> >>                  choose_default_locale_sh =
> >> find_program('tools/choose-default-locale.sh')
> >>                  default_locale =
> >> run_command(choose_default_locale_sh).stdout().strip()
> >>          else
> >>                  default_locale = 'C.UTF-8'
> >>          endif
> >>
> >> if default locale set to C.UTF-8, it will cause libpcre ptest fail:
> >>
> >>    re> //8+L
> >> ** Failed to set locale ""
> > Doesn't your second patch mean we don't need this one?
>
> 1.  No ,  we need this one.
>
> because image maybe don't  install any locale except glibc default C
> locale, like below:
>
> root at qemux86:~# locale -a
> C
> POSIX
>
> if systemd set to C.UTF-8,   any image need to install C.UTF-8 locale by
> default, but only C is default locale.
>
> with this patch,  sysVinit and systemd both set to default C locale.
>
>
> 2. patch 2/2 glibc: Add a C.UTF-8 locale
>
> This patch is not necessary, it is just for add support of C.UTF-8
> locale.  if any package depend on it in the future,  it can install
>
> this locale.
>
>
> Add this patch just because I noticed most of major distro have support
> this locale. not for fix  "** Failed to set locale "".
>
> >
> > Cheers,
> >
> > Richard
> >
> >
> --
> BRs
>
> Sandy(Li Changqing)
>
Can we make default locale overridable please so distros have a chance
to choose different?

Andreas