From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: uvv.mail@gmail.com, Openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH v2 1/3] bitbake.conf: Add conditional host tool getent
Date: Tue, 04 Feb 2025 11:03:42 +0000 [thread overview]
Message-ID: <e1e68f2bdd10981d996869d23fe36b34f15c0bf9.camel@linuxfoundation.org> (raw)
In-Reply-To: <20250204105315.84079-1-uvv.mail@gmail.com>
On Tue, 2025-02-04 at 11:53 +0100, Vyacheslav Yurkov via lists.openembedded.org wrote:
> From: Vyacheslav Yurkov <uvv.mail@gmail.com>
>
> In case of systemd init mananger, it's required for the native build.
> Even though we don't build systemd natively, we might build some of its
> binaries, which need to run on the host system.
>
> Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
> ---
> meta/conf/bitbake.conf | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index 8b607088c6..6b1bfaf64e 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -532,6 +532,9 @@ HOSTTOOLS += "${@'ip ping ps scp ssh stty' if (bb.utils.contains_any('IMAGE_CLAS
> # Used by archiver.bbclass when compression is xz
> HOSTTOOLS += "${@'xz' if (('archiver.bbclass' in (d.getVar('BBINCLUDED') or '')) and (d.getVarFlag('ARCHIVER_MODE', 'compression') == 'xz')) else ''}"
>
> +# Native build of systemd tools requires getent utility
> +HOSTTOOLS += "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'getent', '', d)}"
> +
> # Link to these if present
> HOSTTOOLS_NONFATAL += "aws gcc-ar gpg gpg-agent ld.bfd ld.gold nc pigz sftp socat ssh sudo"
I'm a bit nervous about relying on getent being present. It comes from
glibc and I'm a bit concerned about the host implications of depending
upon that. It certainly gives us a very specific host dependency
(unlike more generic things like coreutils).
Do we have any more information about where/why systemctl is using it
and whether we really do need it?
Cheers,
Richard
next prev parent reply other threads:[~2025-02-04 11:03 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-04 10:53 [PATCH v2 1/3] bitbake.conf: Add conditional host tool getent uvv.mail
2025-02-04 10:53 ` [PATCH v2 2/3] meson.bbclass: Add an option to specify install tags uvv.mail
2025-02-04 10:53 ` [PATCH v2 3/3] systemd: Build the systemctl executable uvv.mail
2025-02-04 11:02 ` Patchtest results for " patchtest
2025-02-04 13:18 ` [OE-core] " Mikko Rapeli
2025-02-05 7:06 ` Vyacheslav Yurkov
2025-02-05 8:30 ` Böszörményi Zoltán
2025-02-04 11:03 ` Richard Purdie [this message]
2025-02-04 11:33 ` [OE-core] [PATCH v2 1/3] bitbake.conf: Add conditional host tool getent Vyacheslav Yurkov
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=e1e68f2bdd10981d996869d23fe36b34f15c0bf9.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=Openembedded-core@lists.openembedded.org \
--cc=uvv.mail@gmail.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.