Wouldn’t the easiest solution be to define two variables:
modulesloaddir ??= "${sysconfdir}/modules-load.d "
modprobedir ??= "${sysconfdir}/modprobe.d "
then in init-manager-systemd.inc you could change them to:
modulesloaddir ?= "${nonarch_libdir}/modules-load.d"
modprobedir ?= "${nonarch_base_libdir}/modprobe.d"
Using a single prefix variable doesn’t really work since they need different
paths in the latter case.
//Peter
From: openembedded-core@lists.openembedded.org
<openembedded-core@lists.openembedded.org> On Behalf Of Jose Quaresma
Sent: den 16 februari 2023 12:08
To: Ola x Nilsson <ola.x.nils...@axis.com>
Cc: Otavio Salvador <otavio.salva...@ossystems.com.br>; Jose Quaresma
<jose.quare...@foundries.io>; openembedded-core@lists.openembedded.org
Subject: Re: [OE-core][PATCH 1/2] kernel-module-split: make autoload and
probeconf distribution specific
Ola x Nilsson <ola.x.nils...@axis.com<mailto:ola.x.nils...@axis.com>> escreveu
no dia quinta, 16/02/2023 à(s) 08:22:
On Wed, Feb 15 2023, Jose Quaresma wrote:
> Hi Octavio,
>
> Otavio Salvador
> <otavio.salva...@ossystems.com.br<mailto:otavio.salva...@ossystems.com.br><mailto:otavio.salva...@ossystems.com.br<mailto:otavio.salva...@ossystems.com.br>>>
> escreveu no dia quarta, 15/02/2023 à(s) 13:25:
> Em qua., 15 de fev. de 2023 às 10:01, Jose Quaresma
> <quaresma.j...@gmail.com<mailto:quaresma.j...@gmail.com><mailto:quaresma.j...@gmail.com<mailto:quaresma.j...@gmail.com>>>
> escreveu:
>>
>> The modules-load.d [1] - Configure kernel modules to load at boot
>> should install their configuration files in /usr/lib/modules-load.d.
>>
>> The modprobe.d [2] - Configuration directory for modprobe
>> should install their configuration files in /lib/modprobe.d
>>
>> [1] https://www.freedesktop.org/software/systemd/man/modules-load.d.html
>> [2]
>> https://www.man7.org/linux/man-pages//man5/modprobe.d.5.html<https://www.man7.org/linux/man-pages/man5/modprobe.d.5.html>
>>
>> Signed-off-by: Jose Quaresma
>> <jose.quare...@foundries.io<mailto:jose.quare...@foundries.io><mailto:jose.quare...@foundries.io<mailto:jose.quare...@foundries.io>>>
>
>
> Did you validate this using busybox only? Using sysv and busybox need
> to be validated as well.
>
> Please drop this patch series because it doesn work with busybox.
>
> Looks like busybox don't support other path than /etc:
> https://git.busybox.net/busybox/tree/modutils/modprobe.c?id=669c40ed8ebf480c95ce36135104e474e361a7e6#n658
>
> So I need to implement this in another way, maybe relocating the files from
> /etc only when systemd is enabled.
>
> Thanks for raising the issue.
>
> Jose
>
Hi,
Please consider https://bugzilla.yoctoproject.org/show_bug.cgi?id=12212
as well.
--
Ola x Nilsson
Hi Ola,
We use usermerge too but I think the weird issue here is we need to have this
distribution specified only for systemd
excluding busybox and sysint as pointed out by Octavio.
The other solution is adding support on busybox and sysint if needed.
My idea is add a prefix variable with path "/etc" by default but conditioned by
systemd DISTRO_FEATURES
Thanks for the bug hint.
--
Best regards,
José Quaresma
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#177308):
https://lists.openembedded.org/g/openembedded-core/message/177308
Mute This Topic: https://lists.openembedded.org/mt/96981851/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
