11002 2017-02-01 10:37:05 +0000 dbus: user-session dbus.socket contains path from native sysroot 2017-03-20 17:33:16 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 2.4 x86 Multiple RESOLVED FIXED Medium+ normal 2.3 M3 1 andreas.kling ross.burton meta.mr.watcher meta.watcher --- --- Don't know oldest_to_newest 70324 0 3613 andreas.kling 2017-02-01 10:37:05 +0000 Created attachment 3613 replace systemctl path from native sysroot Preconditions/Environment ------------------------- image with systemd and PACKAGECONFIG_append_pn-dbus="user-session" Triggering Action/Cause ----------------------- :~# export XDG_RUNTIME_DIR=/run/user/0 :~# /lib/systemd/systemd --user :~# systemctl --user start dbus :~# systemctl --user show-environment Expectation ----------- DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/0/bus ... Actual Result ------------- missing variable What really happens ExecStartPost from dbus.socket tries to launch systemctl from PATH on build machine. Reproducibility --------------- 1/1 70664 1 ross.burton 2017-02-13 16:45:10 +0000 It's probably safer to always set the systemctl assignment in case any other bits of dbus use it in the future. Can you submit this patch on the mailing list please? (http://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded) 70682 2 andreas.kling 2017-02-14 14:02:36 +0000 (In reply to comment #1) > It's probably safer to always set the systemctl assignment in case any other > bits of dbus use it in the future. > > Can you submit this patch on the mailing list please? > > (http://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded) many thanks for your kind advice. since recipe specific sysroots are used the situation has changed. user dbus.socket contains /bin/systemctl for target and native builds even without this patch. dbus does not include sysroot components from systemd-systemctl-native yet, this means the value now refers to installation on the host system determined by $PATH. Adding systemd-systemctl-native to DEPENDS brings back the old situation with '.../1.10.14-r0/recipe-sysroot-native/usr/bin/systemctl' in user dbus.socket for target and native build. So I'm sorry to have some questions before submitting a patch to the mailing list. Even if they somehow sound philosophical cause fiddling with systemd user services for native seems weird. 1) should dbus contain a DEPENDS to systemd-systemctl-native 1a) and should it be conditional on DISTRO_FEATURES systemd? 2) should dbus override ac_cv_path_SYSTEMCTL 2a) and should override be done only for target with EXTRA_OECONF_append_class-target or with EXTRA_OECONF_append? (recipe explicitly sets EXTRA_OECONF_append_class-native = " --disable-selinux") many thanks in advance - Andy 70707 3 andreas.kling 2017-02-15 06:16:48 +0000 I gave the minimal change a shot on the mailing list. 71459 4 ross.burton 2017-03-20 17:33:16 +0000 Merged in oe-core e9471f8da3946439141ccdd8284200aa614df46c. 3613 2017-02-01 10:37:05 +0000 2017-02-01 10:37:05 +0000 replace systemctl path from native sysroot 0001-dbus-do-not-use-systemctl-from-PATH-for-configure.patch text/plain 1582 andreas.kling RnJvbSBkM2Y3N2U4OGRiOWM4ZTE3MGZmM2QzMWE0Yzg4ODRmM2I5MWQ2MGM4IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBBbmR5IEtsaW5nIDxhbmRyZWFzLmtsaW5nQHBlaWtlci1jZWUu ZGU+CkRhdGU6IFdlZCwgMSBGZWIgMjAxNyAxMTowNzo1MCArMDEwMApTdWJqZWN0OiBbUEFUQ0hd IGRidXM6IGRvIG5vdCB1c2Ugc3lzdGVtY3RsIGZyb20gJFBBVEggZm9yIGNvbmZpZ3VyZQoKICBp ZiBwYWNrYWdlIGNvbmZpZyB1c2VyLXNlc3Npb24gaXMgZW5hYmxlZCB1c2VyIGRidXMuc29ja2V0 IGNvbnRhaW5zCiAgcGF0aCB0byBzeXN0ZW1jdGwgZnJvbSBuYXRpdmUgc3lzcm9vdC4gc3lzdGVt Y3RsIGlzIHVzZWQgYXQgcnVudGltZQogIHRvIHNldCB1cCB0aGUgZGJ1cyBzZXNzaW9uIGVudmly b25tZW50LgoKICBPdmVycmlkZSBwYXRoIHVzaW5nIGFjX2N2X3BhdGhfU1lTVEVNQ1RMPSR7YmFz ZV9iaW5kaXJ9L3N5c3RlbWN0bAogIGZvciBwYWNrYWdlIGNvbmZpZyB1c2VyLXNlc3Npb24uCi0t LQogbWV0YS9yZWNpcGVzLWNvcmUvZGJ1cy9kYnVzXzEuMTAuMTQuYmIgfCAyICstCiAxIGZpbGUg Y2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkKCmRpZmYgLS1naXQgYS9tZXRh L3JlY2lwZXMtY29yZS9kYnVzL2RidXNfMS4xMC4xNC5iYiBiL21ldGEvcmVjaXBlcy1jb3JlL2Ri dXMvZGJ1c18xLjEwLjE0LmJiCmluZGV4IGU0ZjU0ZWM3MzQuLjYyMjExOTE3YzMgMTAwNjQ0Ci0t LSBhL21ldGEvcmVjaXBlcy1jb3JlL2RidXMvZGJ1c18xLjEwLjE0LmJiCisrKyBiL21ldGEvcmVj aXBlcy1jb3JlL2RidXMvZGJ1c18xLjEwLjE0LmJiCkBAIC0xMDgsNyArMTA4LDcgQEAgUEFDS0FH RUNPTkZJR19jbGFzcy1uYXRpdmVzZGsgPSAiIgogUEFDS0FHRUNPTkZJR1tzeXN0ZW1kXSA9ICIt LWVuYWJsZS1zeXN0ZW1kIC0td2l0aC1zeXN0ZW1kc3lzdGVtdW5pdGRpcj0ke3N5c3RlbWRfc3lz dGVtX3VuaXRkaXJ9LC0tZGlzYWJsZS1zeXN0ZW1kIC0td2l0aG91dC1zeXN0ZW1kc3lzdGVtdW5p dGRpcixzeXN0ZW1kIgogUEFDS0FHRUNPTkZJR1t4MTFdID0gIi0td2l0aC14IC0tZW5hYmxlLXgx MS1hdXRvbGF1bmNoLC0td2l0aG91dC14IC0tZGlzYWJsZS14MTEtYXV0b2xhdW5jaCwgdmlydHVh bC9saWJ4MTEgbGlic20iCiBQQUNLQUdFQ09ORklHW2xhcmdlZmlsZV0gPSAiLS1lbmFibGUtbGFy Z2VmaWxlLC0tZGlzYWJsZS1sYXJnZWZpbGUsLCIKLVBBQ0tBR0VDT05GSUdbdXNlci1zZXNzaW9u XSA9ICItLWVuYWJsZS11c2VyLXNlc3Npb24gLS13aXRoLXN5c3RlbWR1c2VydW5pdGRpcj0ke3N5 c3RlbWRfdXNlcl91bml0ZGlyfSwtLWRpc2FibGUtdXNlci1zZXNzaW9uIgorUEFDS0FHRUNPTkZJ R1t1c2VyLXNlc3Npb25dID0gIi0tZW5hYmxlLXVzZXItc2Vzc2lvbiAtLXdpdGgtc3lzdGVtZHVz ZXJ1bml0ZGlyPSR7c3lzdGVtZF91c2VyX3VuaXRkaXJ9IGFjX2N2X3BhdGhfU1lTVEVNQ1RMPSR7 YmFzZV9iaW5kaXJ9L3N5c3RlbWN0bCwtLWRpc2FibGUtdXNlci1zZXNzaW9uIgogCiBkb19pbnN0 YWxsKCkgewogCWF1dG90b29sc19kb19pbnN0YWxsCi0tIAoyLjExLjAKCg==