On 02/25/2013 12:51 AM, Iorga, Cristian wrote:
Changes from V1:
        - connman client is explicitly enabled
        - ${B} is used instead of ${S} (in this specific case are the same, it 
will be changed later on)

Please add these to a proper commit message.

Also, there was some discussion prior about putting the connmanctl in a separate package, that does not seem to have happened here.

It could go into the -tools packages that already exists or it's own package connman-ctl?

I know Jack M. mentioned looking into this also, so let's be sure to coordinate this.

Thanks
        Sau!

-----Original Message-----
From: Iorga, Cristian
Sent: Monday, February 25, 2013 10:49 AM
To: openembedded-core@lists.openembedded.org
Cc: Iorga, Cristian
Subject: [PATCH V2] connman: enable connman client

connmanctl is now included when connman is installed

Signed-off-by: Cristian Iorga <cristian.io...@intel.com>
---
  meta/recipes-connectivity/connman/connman.inc |    5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-connectivity/connman/connman.inc 
b/meta/recipes-connectivity/connman/connman.inc
index b61e2af..b0ede60 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -20,7 +20,7 @@ DEPENDS  = "dbus glib-2.0 ppp iptables gnutls \
              ${@base_contains('DISTRO_FEATURES', '3g','ofono', '', d)} \
              "

-INC_PR = "r17"
+INC_PR = "r18"

  TIST = "--enable-tist"
  TIST_powerpc = ""
@@ -38,7 +38,7 @@ EXTRA_OECONF += "\
      --enable-tools \
      --enable-test \
      --disable-polkit \
-    --disable-client \
+    --enable-client \
      --enable-fake \
      ${@base_contains('DISTRO_FEATURES', 'systemd', 
'--with-systemdunitdir=${systemd_unitdir}/system/', '', d)} \  "
@@ -73,6 +73,7 @@ do_install_append() {
        install -d ${D}${bindir}
        install -m 0755 ${S}/tools/*-test ${D}${bindir}
        install -m 0755 ${S}/tools/wispr ${D}${bindir}
+       install -m 0755 ${B}/client/connmanctl ${D}${bindir}

        # We don't need to package an empty directory
        rmdir ${D}${libdir}/connman/scripts
--
1.7.10.4


_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to