Skip to contentSkip to conversation

connman: fix missing dbus configuration files

PATCH New Mike Williams 2 participants 3 messages Oct 15, 2015 – Oct 17, 2015

lore
Share & tools
lore

Mike Williams

author
#0 lore raw

PATCH 1/2 connman: fix missing dbus configuration files

Without connman.conf in /etc/dbus-1/system.d/, connmand cannot take its name on the dbus system bus and will fail to start with a permission denied error.

Signed-off-by Mike Williams <[email protected]>
---
 package/connman/connman.mk | 1 +
 1 file changed, 1 insertion(+)
package/connman/connman.mk +1 -0
index 87d81ce..3b14a80 100644
@@ -12,6 +12,7 @@ CONNMAN_INSTALL_STAGING = YES
 CONNMAN_LICENSE = GPLv2
 CONNMAN_LICENSE_FILES = COPYING
 CONNMAN_CONF_OPTS += \
+	--with-dbusconfdir=/etc \
 	$(if $(BR2_PACKAGE_CONNMAN_DEBUG),--enable-debug,--disable-debug)		\
 	$(if $(BR2_PACKAGE_CONNMAN_ETHERNET),--enable-ethernet,--disable-ethernet)	\
 	$(if $(BR2_PACKAGE_CONNMAN_WIFI),--enable-wifi,--disable-wifi)			\
Signature · 1 line
2.1.4
author
#1 lore raw

PATCH 2/2 ofono: fix missing systemd and dbus files

Ofono's service file was not being installed when systemd was selected as the init system.

Without ofono.conf in /etc/dbus-1/system.d, ofonod cannot take its dbus name on the system bus and will fail to start with a permission denied error.

Signed-off-by Mike Williams <[email protected]>
---
 package/ofono/ofono.mk | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
package/ofono/ofono.mk +3 -1
index f1a63e2..43d16fd 100644
@@ -16,7 +16,9 @@ OFONO_DEPENDENCIES = \
 	libcap-ng \
 	mobile-broadband-provider-info
 
-OFONO_CONF_OPTS = --disable-test
+OFONO_CONF_OPTS = --disable-test \
+	--with-dbusconfdir=/etc \
+	$(if $(BR2_INIT_SYSTEMD),--with-systemdunitdir=/usr/lib/systemd/system)
 
 # N.B. Qualcomm QMI modem support requires O_CLOEXEC; so
 # make sure that it is defined.
Signature · 1 line
2.1.4
#2 lore raw

PATCH 1/2 connman: fix missing dbus configuration files

In reply to Mike Williams

Dear Mike Williams,

Quoted 8 lines · Mike Williams

Without connman.conf in /etc/dbus-1/system.d/, connmand cannot take its name on the dbus system bus and will fail to start with a permission denied error.

Signed-off-by Mike Williams <[email protected]>
---
 package/connman/connman.mk | 1 +
 1 file changed, 1 insertion(+)

Both patches applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com