Re: [OE-core] [PATCH v4] systemd: Build the systemctl executable

[Mikko Rapeli <mikko.rapeli@linaro.org>]

Hi,

On Mon, Feb 10, 2025 at 09:07:57AM +0100, Vyacheslav Yurkov via lists.openembedded.org wrote:
> On 08.02.2025 19:34, Richard Purdie wrote:
> > On Fri, 2025-02-07 at 08:10 +0100, Vyacheslav Yurkov via lists.openembedded.org wrote:
> > > From: Vyacheslav Yurkov <uvv.mail@gmail.com>
> > > 
> > > Instead of the python re-implementation build the actual systemctl from
> > > the systemd source tree. The python script was used when systemd didn't
> > > provide an option to build individual executables. It is possible in the
> > > meantime, so instead of always adapting the script when there's a new
> > > functionality, we simply use upstream implementation.
> > > 
> > > License-Update: Base recipe is used
> > > 
> > > Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
> > > ---
> > >  �.../systemd/systemd-systemctl-native.bb������ |� 23 +-
> > >  �...-meson-add-install-tag-for-systemctl.patch |� 25 ++
> > >  �...2-meson-Bypass-certain-config-checks.patch |� 64 +++
> > >  �.../systemd/systemd-systemctl/systemctl������ | 366 ------------------
> > >  �4 files changed, 100 insertions(+), 378 deletions(-)
> > >  �create mode 100644 meta/recipes-core/systemd/systemd-systemctl/0001-meson-add-install-tag-for-systemctl.patch
> > >  �create mode 100644 meta/recipes-core/systemd/systemd-systemctl/0002-meson-Bypass-certain-config-checks.patch
> > >  �delete mode 100755 meta/recipes-core/systemd/systemd-systemctl/systemctl
> > Unfortunately this showed circular dependency errors in testing:
> > 
> > https://autobuilder.yoctoproject.org/valkyrie/#/builders/76/builds/933
> > 
> > Cheers,
> > 
> > Richard
> 
> I believe the failure comes from util-linux. Any idea how to "inherit"
> conditionally only for target builds? (without creating a separate native
> recipe).

Set a variable conditionally and use that with "inherit".

FOO = ""
FOO:class-target = "bar"

inherit ${FOO}

Cheers,

-Mikko