What's the point of PROVIDES += dbus-x11, it build fine without it (I'm
not questioning the correctness of this, just want to understand the
need for it).
Radu
On 07/26/2012 11:43 AM, Koen Kooi wrote:
Op 26 jul. 2012, om 10:42 heeft Paul Eggleton <[email protected]>
het volgende geschreven:
On Thursday 26 July 2012 10:08:49 Koen Kooi wrote:
Op 26 jul. 2012, om 08:29 heeft Radu Moisan <[email protected]> het
volgende geschreven:
it does not build, it complains about nothing providing dbus-x11
Radu
On 07/26/2012 09:17 AM, Radu Moisan wrote:
Followed suggestions from Bugz 2261:
1) remove the --with-x/--without-x configure arguments. If you want to
force no-discovery for native builds the correct argument is
--disable-x11-autolaunch. This ensures that DBus looks at the build
environment to determine whether to enable X11 bus discovery or not.
2) make the virtual/libx11 DEPENDS conditional based on the x11 distro
feature. This makes the build dependencies reflect the feature list.
3) remove dbus-x11, meaning that dbus-launch with its potential X11
dependency is now back in dbus where is belongs.
4) Potentially make dbus provide dbus-x11, for compatibility.
Fixes [Yocto #2261]
Signed-off-by: Radu Moisan <[email protected]>
---
meta/recipes-core/dbus/dbus.inc | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/meta/recipes-core/dbus/dbus.inc
b/meta/recipes-core/dbus/dbus.inc index a75583d..9559f6f 100644
--- a/meta/recipes-core/dbus/dbus.inc
+++ b/meta/recipes-core/dbus/dbus.inc
@@ -31,10 +31,10 @@ CONFFILES_${PN} = "${sysconfdir}/dbus-1/system.conf
${sysconfdir}/dbus-1/session>>
DEBIANNAME_${PN} = "dbus-1"
-PACKAGES =+ "${PN}-lib ${PN}-systemd
${@base_contains('DISTRO_FEATURES', 'x11', '${PN}-x11', '', d)}">>
+PACKAGES =+ "${PN}-lib ${PN}-systemd"
-FILES_${PN}-x11 = "${bindir}/dbus-launch"
-RDEPENDS_${PN}-x11 = "${PN}"
+# for compatibility
PROVIDES += "dbus-x11"
+RREPLACES_${PN} += "dbus-x11"
RPROVIDES_${PN} += "dbus-x11"
For the sake of clarity, you mean both of these, not RPROVIDES instead of
adding RREPLACES - right?
Correct, you need both (or rather all 3 if you include PROVIDES).
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
