Re: [ptxdist] Missing systemd-hwdb dependency?

2022-09-30 Thread 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- |



Re: [ptxdist] Missing systemd-hwdb dependency?

2022-09-30 Thread 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