All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] host-localedef: Compile against glibc-2.29
Date: Wed, 19 Jun 2019 13:48:12 +0200	[thread overview]
Message-ID: <20190619134812.4bd672e7@windsurf> (raw)
In-Reply-To: <20190619020707.3798-1-sam@mendozajonas.com>

Hello,

On Wed, 19 Jun 2019 12:07:07 +1000
Samuel Mendoza-Jonas <sam@mendozajonas.com> wrote:

> In glibc 2.27 the following change occurred:
> "Statically compiled applications attempting to load locales compiled
> for the GNU C Library version 2.27 will fail and fall back to the
> builtin C/POSIX locale."
> 
> This impacts us since upstream buildroot uses a localdef built against
> an older eglibc release [0].
> 
> This is a combination of my patch to move to glibc and Peter Seiderer's
> patch to avoid building all of glibc just for localedef.
> 
>  [0] https://bugs.busybox.net/show_bug.cgi?id=11096
> 
> Signed-off-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>
> [localedef build & fixups:]
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> This is an update to a previous very similar patch, but updated for
> glibc 2.29 now that Buildroot has moved to it.

So, since it's been a problem pending for way too long, I applied your
patch to master. I however did a change: make sure we re-use the glibc
tarball if possible. To do this:

 - I defined LOCALEDEF_SOURCE = glibc-$(LOCALEDEF_VERSION).tar.gz and
   HOST_LOCALEDEF_DL_SUBDIR = glibc

 - I changed the hash file because the tarball is now named
   glibc-<version>.tar.gz

I also wondered about making localedef.hash a symlink to glibc.hash,
but that would require updating the symlink everything the glibc
version is updated, because glibc.hash is in a version-specific folder.

Overall, I am wondering if we shouldn't give up on this host-localedef
package and in fact move this to a host-glibc package. Indeed that's
really what we're doing here: build a host-glibc package.

The only issue with doing this is the "HACK" patch. Indeed, how to make
sure this patch will apply to all glibc versions we support ?

But since the locale problem has been around for a long time, I
preferred to apply your approach now, we can always improve things
later if we think it's useful.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2019-06-19 11:48 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-19  2:07 [Buildroot] [PATCH] host-localedef: Compile against glibc-2.29 Samuel Mendoza-Jonas
2019-06-19 11:48 ` Thomas Petazzoni [this message]
2019-06-24 18:52   ` Peter Korsgaard
2019-06-25  0:38     ` Samuel Mendoza-Jonas
2019-06-25  6:16       ` Peter Korsgaard
2019-08-07  7:12         ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190619134812.4bd672e7@windsurf \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.