Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 699958

Summary: app-accessibility/at-spi2-core-2.34.0 : add optional support for dbus-broker
Product: Gentoo Linux Reporter: Chris Mayo <aklhfex>
Component: Current packagesAssignee: Gentoo Linux Gnome Desktop Team <gnome>
Status: UNCONFIRMED ---    
Severity: normal CC: ibuyandtrade0+bugs.gentoo.org
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Chris Mayo 2019-11-12 19:54:06 UTC
Currently at-spi2-core will start sys-apps/dbus's dbus-daemon but it can now optionally use dbus-broker:

https://gitlab.gnome.org/GNOME/at-spi2-core/commit/d7f47c99d093ba91eb18523e14b23bde1794f5bc

Configured with:

-Ddefault_bus=dbus-broker


Looks like sys-apps/dbus is still a build-time dependency:

registryd/registry.h:30:#include <dbus/dbus.h>
registryd/registry-main.c:32:#include <dbus/dbus.h>

with sys-apps/dbus-broker only needed at run time.

The patch below does work for me, although I don't know if [${MULTILIB_USEDEP}] is needed for an RDEPEND on an executable.

--- at-spi2-core-2.34.0.ebuild
+++ at-spi2-core-2.34.0-r1.ebuild
@@ -10,10 +10,10 @@
 
 LICENSE="LGPL-2+"
 SLOT="2"
-IUSE="X gtk-doc +introspection"
+IUSE="X dbus-broker gtk-doc +introspection"
 KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~mips ~ppc ~ppc64 ~s390 ~sparc ~x86 ~amd64-linux ~x86-linux ~x64-macos ~x86-macos"
 
-RDEPEND="
+DEPEND="
 	>=sys-apps/dbus-1.5[${MULTILIB_USEDEP}]
 	>=dev-libs/glib-2.36:2[${MULTILIB_USEDEP}]
 	introspection? ( >=dev-libs/gobject-introspection-1.54.0:= )
@@ -23,7 +23,9 @@
 		x11-libs/libXi[${MULTILIB_USEDEP}]
 	)
 "
-DEPEND="${RDEPEND}"
+RDEPEND="${DEPEND}
+	dbus-broker? ( sys-apps/dbus-broker )
+"
 BDEPEND="
 	dev-util/glib-utils
 	gtk-doc? (
@@ -41,6 +43,7 @@
 multilib_src_configure() {
 	local emesonargs=(
 		-Dsystemd_user_dir="$(systemd_get_userunitdir)"
+		-Ddefault_bus=$(usex dbus-broker dbus-broker dbus-daemon)
 		-Ddocs=$(multilib_native_usex gtk-doc true false)
 		-Dintrospection=$(multilib_native_usex introspection)
 		-Dx11=$(usex X)