[Buildroot] [PATCH] connman: add missing dbus system bus configuration

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Oct 13 21:27:13 UTC 2015


Dear Mike Williams,

On Tue, 13 Oct 2015 10:45:38 -0400, Mike Williams wrote:
> Signed-off-by: Mike Williams <mike at mikebwilliams.com>
> ---
>  package/connman/connman.mk | 7 +++++++
>  1 file changed, 7 insertions(+)

Thanks for the patch. However, I believe it's not the proper way of
solving the problem. Can you try the below patch instead:

diff --git a/package/connman/connman.mk b/package/connman/connman.mk
index 87d81ce..3b14a80 100644
--- a/package/connman/connman.mk
+++ b/package/connman/connman.mk
@@ -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)                  \

This will make Connman automatically install its D-Bus configuration
file in /etc/dbus-1/system.d/.

If it works for you, can you send a new version of the patch? For this
new version, please include the explanation *inside* the commit log,
and not in the cover letter.

Thanks a lot!

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


More information about the buildroot mailing list