From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Yegor Yefremov <yegorslists@googlemail.com>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2] package/gobject-introspection: fix host build when /bin/sh warns
Date: Wed, 21 Oct 2026 20:59:45 +0200 [thread overview]
Message-ID: <atkLNw8pfPRoTzzl@windsurf> (raw)
In-Reply-To: <20261020221309.4194290-1-yegorslists@googlemail.com>
On Wed, Oct 21, 2026 at 12:13:09AM +0200, Yegor Yefremov via buildroot wrote:
> host-gobject-introspection is built with LD_LIBRARY_PATH pointing to
> $(HOST_DIR)/lib. When g-ir-scanner calls pkg-config, which in Buildroot
> is a /bin/sh wrapper script, the shell inherits that LD_LIBRARY_PATH.
> On hosts where /bin/sh is bash linked against the system libreadline
> (e.g. Arch Linux), and host-ncurses has already been built, the
> dynamic loader picks up Buildroot's libncursesw, which has no symbol
> versioning, and warns on stderr:
I've applied your patch because it is the most obvious direct
solution, but I find the above explanation rather worrying. It really
isn't great that we end up executing the system-provided bash with a
LD_LIBRARY_PATH pointing to our libraries. It happens to work by
chance because our ncurses is probably not too different from the
system-provided ncurses, but that seems like to be more luck than
correctness.
Shouldn't we investigate a different way that avoids having to pass
LD_LIBRARY_PATH? I honestly don't recall why passing LD_LIBRARY_PATH
is needed.
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2026-10-21 19:00 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-20 21:22 [Buildroot] [PATCH] package/gobject-introspection: fix host build when /bin/sh warns Yegor Yefremov via buildroot
2026-10-20 22:13 ` [Buildroot] [PATCH v2] " Yegor Yefremov via buildroot
2026-10-21 18:59 ` Thomas Petazzoni via buildroot [this message]
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=atkLNw8pfPRoTzzl@windsurf \
--to=buildroot@buildroot.org \
--cc=thomas.petazzoni@bootlin.com \
--cc=yegorslists@googlemail.com \
/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.