mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] gettext: deterministically build with included libraries
Date: Fri, 14 Feb 2020 10:12:48 +0100	[thread overview]
Message-ID: <20200214091248.GC7958@pengutronix.de> (raw)
In-Reply-To: <20200213172208.22452-3-matthias.schiffer@ew.tq-group.com>

On Thu, Feb 13, 2020 at 06:22:05PM +0100, Matthias Schiffer wrote:
> gettext's configure will automatically used the libraries it finds, and
> fall back to included versions otherwise. Avoid this non-deterministic
> behaviour by explicitly choosing the included libraries.
> 
> In particular, this issue can break parallel builds with --force after
> config changes due to a race condition:
> 
> 1. gettext configure finds libraries from a previous build in sysroot,
>    decides not to use the included libraries
> 2. Installed libraries gets cleaned from sysroot due to --force
> 3. gettext build fails, as it attemps to use the now removed libraries

I'd prefer to select glib and libxml instead. Or are there any reasons not
to do this?

The only current user is parted. Do you know why it needs that on the
target anyways? Most packages only need gettext at runtime.

Michael

> Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>
> ---
>  rules/gettext.make | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/rules/gettext.make b/rules/gettext.make
> index dd1d22ffb..199616c87 100644
> --- a/rules/gettext.make
> +++ b/rules/gettext.make
> @@ -54,10 +54,16 @@ GETTEXT_AUTOCONF := \
>  	--disable-acl \
>  	--disable-openmp \
>  	--disable-curses \
> +	--with-included-gettext \
> +	--with-included-glib \
> +	--with-included-libcroco \
> +	--with-included-libunistring \
> +	--with-included-libxml \
>  	--without-emacs \
>  	--without-bzip2 \
>  	--without-xz
>  
> +
>  # ----------------------------------------------------------------------------
>  # Target-Install
>  # ----------------------------------------------------------------------------
> -- 
> 2.17.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2020-02-14  9:12 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-13 17:22 [ptxdist] [PATCH] kmscube: switch to https URL Matthias Schiffer
2020-02-13 17:22 ` [ptxdist] [PATCH] parted: update to 3.3 Matthias Schiffer
2020-02-14  8:49   ` Michael Olbrich
2020-02-14  9:07   ` [ptxdist] [PATCH v2] " Matthias Schiffer
2020-02-14 10:38   ` [ptxdist] [PATCH v3] " Matthias Schiffer
2020-02-15 10:05     ` Michael Olbrich
2020-02-17  8:53     ` [ptxdist] [PATCH v4] " Matthias Schiffer
2020-02-13 17:22 ` [ptxdist] [PATCH] gettext: deterministically build with included libraries Matthias Schiffer
2020-02-14  9:12   ` Michael Olbrich [this message]
2020-02-14  9:48     ` [ptxdist] (EXT) " Matthias Schiffer
2020-02-14 10:38       ` Matthias Schiffer
2020-02-15  9:27         ` Michael Olbrich
2020-02-13 17:22 ` [ptxdist] [PATCH] gst-plugins-good1: qmlglsink: fix build on EGL platform without X11 headers Matthias Schiffer
2020-02-14  9:18   ` Michael Olbrich
2020-02-13 17:22 ` [ptxdist] [PATCH] varoverlayfs: support kernels without metacopy Matthias Schiffer
2020-02-14  9:21   ` Michael Olbrich
2020-02-14 10:38   ` [ptxdist] [PATCH v2] " Matthias Schiffer
2020-02-13 17:22 ` [ptxdist] [PATCH] syslogng: update to 3.25.1 Matthias Schiffer
2020-02-17  9:38   ` Michael Olbrich
2020-02-17  9:59   ` Michael Olbrich
2020-02-20  8:39   ` [ptxdist] [PATCH v2] " Matthias Schiffer
2020-02-21  7:47     ` Michael Olbrich
2020-02-21  8:30       ` [ptxdist] (EXT) " Matthias Schiffer
2020-03-18 10:50         ` Michael Olbrich
2020-04-03  5:28           ` Michael Olbrich
2020-04-03  8:00             ` [ptxdist] (EXT) " Matthias Schiffer
2020-02-14  8:40 ` [ptxdist] [PATCH] kmscube: switch to https URL Michael Olbrich
2020-02-14  8:53   ` [ptxdist] (EXT) " Matthias Schiffer
2020-02-15  9:29     ` Michael Olbrich
2020-02-17  9:00 ` [ptxdist] [PATCH v2] kmscube: switch to https archive download Matthias Schiffer

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=20200214091248.GC7958@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox