Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH v2] connman: add iptables to RDEPENDS
@ 2013-05-13  2:58 Jonathan Liu
  2013-05-13  9:05 ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Liu @ 2013-05-13  2:58 UTC (permalink / raw)
  To: openembedded-core

The connmand daemon requires libxtables.so from iptables to start up.

Signed-off-by: Jonathan Liu <net147@gmail.com>
---
 meta/recipes-connectivity/connman/connman.inc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index afc361c..44d9d6e 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -15,7 +15,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
 
 DEPENDS  = "dbus glib-2.0 ppp iptables gnutls"
 
-INC_PR = "r20"
+INC_PR = "r21"
 
 EXTRA_OECONF += "\
     ac_cv_path_WPASUPPLICANT=${sbindir}/wpa_supplicant \
@@ -99,6 +99,7 @@ RPROVIDES_${PN} = "\
 
 RDEPENDS_${PN} = "\
 	dbus \
+	iptables \
 	${@base_contains('PACKAGECONFIG', 'bluetooth', 'bluez4', '', d)} \
 	${@base_contains('PACKAGECONFIG', 'wifi','wpa-supplicant', '', d)} \
 	${@base_contains('PACKAGECONFIG', '3g','ofono', '', d)} \
-- 
1.8.2.3




^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] connman: add iptables to RDEPENDS
  2013-05-13  2:58 [PATCH v2] connman: add iptables to RDEPENDS Jonathan Liu
@ 2013-05-13  9:05 ` Burton, Ross
  2013-05-13  9:23   ` Jonathan Liu
  0 siblings, 1 reply; 6+ messages in thread
From: Burton, Ross @ 2013-05-13  9:05 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: openembedded-core

On 13 May 2013 03:58, Jonathan Liu <net147@gmail.com> wrote:
> The connmand daemon requires libxtables.so from iptables to start up.

And the automatic dependencies-from-linkage code seems to be working just fine:

$ dpkg -I connman_1.13-r20.0.2_core2.ipk
...
 Depends: iptables (>= 1.4.17), dbus-1, ofono, libdbus-1-3 (>= 1.6.8),
bluez4, libgnutls26 (>= 2.12.21), libc6 (>= 2.17), libglib-2.0-0 (>=
2.36.0), shadow, base-passwd, wpa-supplicant

So the question is can you verify that this dependency isn't happening
for you, and how do we fix *that* problem.

Ross



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] connman: add iptables to RDEPENDS
  2013-05-13  9:05 ` Burton, Ross
@ 2013-05-13  9:23   ` Jonathan Liu
  2013-05-13 10:03     ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Liu @ 2013-05-13  9:23 UTC (permalink / raw)
  To: Burton, Ross; +Cc: openembedded-core@lists.openembedded.org

I did a git pull which updated connman to 1.13 and rebuilt image from
scratch clearing all temp files and work. The iptables dependency was
missing when I checked with opkg-cl on the target.

Regards,
Jonathan

On 13/05/2013, at 7:05 PM, "Burton, Ross" <ross.burton@intel.com> wrote:

> On 13 May 2013 03:58, Jonathan Liu <net147@gmail.com> wrote:
>> The connmand daemon requires libxtables.so from iptables to start up.
>
> And the automatic dependencies-from-linkage code seems to be working just fine:
>
> $ dpkg -I connman_1.13-r20.0.2_core2.ipk
> ...
> Depends: iptables (>= 1.4.17), dbus-1, ofono, libdbus-1-3 (>= 1.6.8),
> bluez4, libgnutls26 (>= 2.12.21), libc6 (>= 2.17), libglib-2.0-0 (>=
> 2.36.0), shadow, base-passwd, wpa-supplicant
>
> So the question is can you verify that this dependency isn't happening
> for you, and how do we fix *that* problem.
>
> Ross



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] connman: add iptables to RDEPENDS
  2013-05-13  9:23   ` Jonathan Liu
@ 2013-05-13 10:03     ` Burton, Ross
  2013-05-13 11:29       ` Jonathan Liu
  0 siblings, 1 reply; 6+ messages in thread
From: Burton, Ross @ 2013-05-13 10:03 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: openembedded-core@lists.openembedded.org

On 13 May 2013 10:23, Jonathan Liu <net147@gmail.com> wrote:
> I did a git pull which updated connman to 1.13 and rebuilt image from
> scratch clearing all temp files and work. The iptables dependency was
> missing when I checked with opkg-cl on the target.

I really can't replicate this - my connman packages just built with
latest git have a versioned iptables dependency from the link.  Check
your do_package logs, you should find lines like this:

DEBUG: connman: Dependency libxtables.so.9 requires package iptables

Ross



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] connman: add iptables to RDEPENDS
  2013-05-13 10:03     ` Burton, Ross
@ 2013-05-13 11:29       ` Jonathan Liu
  2013-05-13 14:21         ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Liu @ 2013-05-13 11:29 UTC (permalink / raw)
  To: Burton, Ross; +Cc: openembedded-core@lists.openembedded.org

On 13/05/2013 8:03 PM, Burton, Ross wrote:
> On 13 May 2013 10:23, Jonathan Liu <net147@gmail.com> wrote:
>> I did a git pull which updated connman to 1.13 and rebuilt image from
>> scratch clearing all temp files and work. The iptables dependency was
>> missing when I checked with opkg-cl on the target.
> I really can't replicate this - my connman packages just built with
> latest git have a versioned iptables dependency from the link.  Check
> your do_package logs, you should find lines like this:
>
> DEBUG: connman: Dependency libxtables.so.9 requires package iptables
>
> Ross
The connmand daemon was complaining about libxtables.so.9 missing, but 
it's libxtables.so.10 now.
Maybe I just had to force a more complete rebuild.

Regards,
Jonathan



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] connman: add iptables to RDEPENDS
  2013-05-13 11:29       ` Jonathan Liu
@ 2013-05-13 14:21         ` Burton, Ross
  0 siblings, 0 replies; 6+ messages in thread
From: Burton, Ross @ 2013-05-13 14:21 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: openembedded-core@lists.openembedded.org

On 13 May 2013 12:29, Jonathan Liu <net147@gmail.com> wrote:
> The connmand daemon was complaining about libxtables.so.9 missing, but it's
> libxtables.so.10 now.
> Maybe I just had to force a more complete rebuild.

iptables upgrading should have caused connman to rebuild against the
new iptables and pick up the new dependency, so it's possible that
your machine is pending a connman upgrade if you've been using a feed
to upgrade.  If it's a fresh image, then something went rather wrong.

Ross



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-05-13 14:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-13  2:58 [PATCH v2] connman: add iptables to RDEPENDS Jonathan Liu
2013-05-13  9:05 ` Burton, Ross
2013-05-13  9:23   ` Jonathan Liu
2013-05-13 10:03     ` Burton, Ross
2013-05-13 11:29       ` Jonathan Liu
2013-05-13 14:21         ` Burton, Ross

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox