[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