[ptxdist] Missing systemd-hwdb dependency?

[ptxdist] Missing systemd-hwdb dependency?

[Ladislav Michl]

While updating BSP to current git, build stops there:

----------------------------
target: systemd-hwdb.install
----------------------------

Removing files from sysroot...

install: creating directory '{BSP}/platform-armv7a/packages/systemd-hwdb-1/usr/lib/udev'
/usr/bin/bash: line 1: {BSP}/platform-armv7a/sysroot-host/bin/systemd-hwdb: No such file or directory
make: *** [{PTXDIST}/rules/systemd-hwdb.make:26: {BSP}/platform-armv7a/state/systemd-hwdb.install] Error 127

The systemd-hwdb.install rule needs host-systemd, but that is not selected as
select SYSTEMD_HWDB     if SYSTEMD_UDEV_HWDB && RUNTIME
is used in rules/systemd.in.
Removing 'RUNTIME' obviously fixes problem.

How is that intended to be working?

Thank you,
	ladis

Re: [ptxdist] Missing systemd-hwdb dependency?

[Ladislav Michl]

On Fri, Sep 30, 2022 at 03:55:47PM +0200, Ladislav Michl wrote:
> While updating BSP to current git, build stops there:
> 
> ----------------------------
> target: systemd-hwdb.install
> ----------------------------
> 
> Removing files from sysroot...
> 
> install: creating directory '{BSP}/platform-armv7a/packages/systemd-hwdb-1/usr/lib/udev'
> /usr/bin/bash: line 1: {BSP}/platform-armv7a/sysroot-host/bin/systemd-hwdb: No such file or directory
> make: *** [{PTXDIST}/rules/systemd-hwdb.make:26: {BSP}/platform-armv7a/state/systemd-hwdb.install] Error 127
> 
> The systemd-hwdb.install rule needs host-systemd, but that is not selected as
> select SYSTEMD_HWDB     if SYSTEMD_UDEV_HWDB && RUNTIME
> is used in rules/systemd.in.
> Removing 'RUNTIME' obviously fixes problem.

and in fact, it was done this way until 8b17816376e3
"systemd: move dependencies to the toplevel option"
changed that. Michael?

> How is that intended to be working?
> 
> Thank you,
> 	ladis

Re: [ptxdist] Missing systemd-hwdb dependency?

[Michael Olbrich]

On Fri, Sep 30, 2022 at 05:12:07PM +0200, Ladislav Michl wrote:
> On Fri, Sep 30, 2022 at 03:55:47PM +0200, Ladislav Michl wrote:
> > While updating BSP to current git, build stops there:
> > 
> > ----------------------------
> > target: systemd-hwdb.install
> > ----------------------------
> > 
> > Removing files from sysroot...
> > 
> > install: creating directory '{BSP}/platform-armv7a/packages/systemd-hwdb-1/usr/lib/udev'
> > /usr/bin/bash: line 1: {BSP}/platform-armv7a/sysroot-host/bin/systemd-hwdb: No such file or directory
> > make: *** [{PTXDIST}/rules/systemd-hwdb.make:26: {BSP}/platform-armv7a/state/systemd-hwdb.install] Error 127
> > 
> > The systemd-hwdb.install rule needs host-systemd, but that is not selected as
> > select SYSTEMD_HWDB     if SYSTEMD_UDEV_HWDB && RUNTIME
> > is used in rules/systemd.in.
> > Removing 'RUNTIME' obviously fixes problem.
> 
> and in fact, it was done this way until 8b17816376e3
> "systemd: move dependencies to the toplevel option"
> changed that. Michael?
> 
> > How is that intended to be working?

The intention was that systemd only needs systemd-hwdb at runtime and not
at build-time. So we have the 'if RUNTIME' here. And systemd-hwdb needs
host-systemd, so it selects it.
The problem is in the way 'if RUNTIME' is implemented:
kconfig runs with RUNTIME. Since systemd-hwdb has no prompt, it is disabled
entirely so the build-time dependency systemd-hwdb -> host-systemd is not
added :-/.
I'll think about this some more, but I think removing the 'if RUNTIME' is
the best solution. I'll fix it.

Michael

-- 
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 |