Am 06.02.22 um 17:43 schrieb Stefan Herbrechtsmeier:
Am 06.02.22 um 10:53 schrieb Richard Purdie:
On Sun, 2022-02-06 at 08:10 +0000, Richard Purdie via
lists.openembedded.org
wrote:
On Wed, 2022-02-02 at 08:35 +0100, Stefan Herbrechtsmeier wrote:
From: Stefan Herbrechtsmeier <stefan.herbrechtsme...@weidmueller.com>
Link udev shared with systemd helper to minimize the udev package size
if DISTRO_FEATURES contains systemd.
It is only usefull to link udev static with systemd helper if udev
should be installed without systemd.
Signed-off-by: Stefan Herbrechtsmeier
<stefan.herbrechtsme...@weidmueller.com>
---
(no changes since v1)
meta/recipes-core/systemd/systemd_249.7.bb | 1 +
1 file changed, 1 insertion(+)
This seems to break some of our tests:
https://autobuilder.yoctoproject.org/typhoon/#/builders/72/builds/4708
https://autobuilder.yoctoproject.org/typhoon/#/builders/79/builds/3121
https://autobuilder.yoctoproject.org/typhoon/#/builders/80/builds/3085
https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/3089
https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/3127
Presumably it causes extra dependencies to be pulled in causing the
file overlap
where there was previously none.
Thinking a bit more, I think this needs to be:
diff --git a/meta/recipes-core/systemd/systemd_250.3.bb
b/meta/recipes-core/systemd/systemd_250.3.bb
index 2b6bfcc2c25..7851c4c16a7 100644
--- a/meta/recipes-core/systemd/systemd_250.3.bb
+++ b/meta/recipes-core/systemd/systemd_250.3.bb
@@ -69,7 +69,7 @@ PACKAGECONFIG ??= " \
${@bb.utils.filter('DISTRO_FEATURES', 'acl audit efi ldconfig
pam selinux smack usrmerge polkit seccomp', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'wifi', 'rfkill', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'xkbcommon', '',
d)} \
- ${@bb.utils.contains('DISTRO_FEATURES', 'systemd',
'link-udev-shared', '', d)} \
+ ${@bb.utils.contains('DISTRO_FEATURES', 'sysvinit', '',
'link-udev-shared', d)} \
backlight \
binfmt \
gshadow \
which I'll test...
Maybe the systemd naming for the option is sub optimal. The
link-udev-shared option create a dependency between udev and systemd
package. It links udev against the libsystemd-shared library. Without it
udev is linked statically against this library. The option should only
be enabled if udev is always used together with systemd.
Sorry, I should read the patch careful.
I we can assume that udev from systemd is only used with sysvinit or
systemd, this is okay.
Regards
Stefan
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161404):
https://lists.openembedded.org/g/openembedded-core/message/161404
Mute This Topic: https://lists.openembedded.org/mt/88853822/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-