Hi Richard,

On 18.03.2013 16:09, Richard Purdie wrote:
On Mon, 2013-03-18 at 15:38 +0200, Jukka Rissanen wrote:
The /etc/connman is not suitable for the setup script.
There are other connman related scripts in /usr/lib/connman
so moving the wired setup script there.

Signed-off-by: Jukka Rissanen <jukka.rissa...@linux.intel.com>
---
  meta/recipes-connectivity/connman/connman-conf.bb | 8 +++++---
  1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-connectivity/connman/connman-conf.bb 
b/meta/recipes-connectivity/connman/connman-conf.bb
index 7b99953..6ecfbfb 100644
--- a/meta/recipes-connectivity/connman/connman-conf.bb
+++ b/meta/recipes-connectivity/connman/connman-conf.bb
@@ -6,16 +6,18 @@ LIC_FILES_CHKSUM = 
"file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=80
  SRC_URI_append_qemuall = "file://wired.config \
                            file://wired-setup \
                           "
-PR = "r1"
+PR = "r2"

  PACKAGE_ARCH = "${MACHINE_ARCH}"

+FILES_${PN} = "${localstatedir}/* ${libdir}/*"
+
  do_install() {
      #Configure Wired network interface in case of qemu* machines
      if test -e ${WORKDIR}/wired.config && test -e ${WORKDIR}/wired-setup; then
          install -d ${D}${localstatedir}/lib/connman
          install -m 0644 ${WORKDIR}/wired.config 
${D}${localstatedir}/lib/connman
-        install -d ${D}${sysconfdir}/connman
-        install -m 0755 ${WORKDIR}/wired-setup ${D}${sysconfdir}/connman
+        install -d ${D}${libdir}/connman
+        install -m 0755 ${WORKDIR}/wired-setup ${D}${libdir}/connman
      fi
  }

You've moved the files, fine, but what uses them and do we need to
update the caller?

I sent a separate patch "[PATCH] connman-conf: Initial systemd support" for systemd support. I did not found any file that would have init.d support (I might have missed that one :)


Cheers,
Jukka


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

Reply via email to