[PATCH] connman: add alias for connman

[PATCH] connman: add alias for connman

[Zhenbo Gao]

connman get conflicts with networkmanager when building
the project.

here introduce alias virtual/networkmanager for these
two recipes, so setting PREFERRED_PROVIDER to the proper
one can solve the conflicts.

this patch is for connman recipe.

Signed-off-by: Zhenbo Gao <zhenbo.gao@windriver.com>
---
 meta/recipes-connectivity/connman/connman.inc               | 1 +
 meta/recipes-core/packagegroups/packagegroup-self-hosted.bb | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index c375251..b5fc572 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -94,6 +94,7 @@ do_install_append() {
 
 # These used to be plugins, but now they are core
 RPROVIDES_${PN} = "\
+	virtual/networkmanager \
 	connman-plugin-loopback \
 	connman-plugin-ethernet \
 	${@bb.utils.contains('PACKAGECONFIG', 'bluetooth','connman-plugin-bluetooth', '', d)} \
diff --git a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
index aa88cf9..6d7d6af 100644
--- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
@@ -28,7 +28,7 @@ RDEPENDS_packagegroup-self-hosted = "\
     "
 
 RDEPENDS_packagegroup-self-hosted-host-tools = "\
-    connman \
+    virtual/networkmanager \
     connman-plugin-ethernet \
     dhcp-client \
     e2fsprogs \
-- 
1.9.1

Re: [PATCH] connman: add alias for connman

[Andreas Oberritter]

On 25.05.2016 09:11, Zhenbo Gao wrote:
> connman get conflicts with networkmanager when building
> the project.
> 
> here introduce alias virtual/networkmanager for these
> two recipes, so setting PREFERRED_PROVIDER to the proper
> one can solve the conflicts.
> 
> this patch is for connman recipe.
> 
> Signed-off-by: Zhenbo Gao <zhenbo.gao@windriver.com>
> ---
>  meta/recipes-connectivity/connman/connman.inc               | 1 +
>  meta/recipes-core/packagegroups/packagegroup-self-hosted.bb | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
> index c375251..b5fc572 100644
> --- a/meta/recipes-connectivity/connman/connman.inc
> +++ b/meta/recipes-connectivity/connman/connman.inc
> @@ -94,6 +94,7 @@ do_install_append() {
>  
>  # These used to be plugins, but now they are core
>  RPROVIDES_${PN} = "\
> +	virtual/networkmanager \
>  	connman-plugin-loopback \
>  	connman-plugin-ethernet \
>  	${@bb.utils.contains('PACKAGECONFIG', 'bluetooth','connman-plugin-bluetooth', '', d)} \
> diff --git a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> index aa88cf9..6d7d6af 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> @@ -28,7 +28,7 @@ RDEPENDS_packagegroup-self-hosted = "\
>      "
>  
>  RDEPENDS_packagegroup-self-hosted-host-tools = "\
> -    connman \
> +    virtual/networkmanager \
>      connman-plugin-ethernet \
       ^
This will create a conflict whenever virtual/networkmanager is not
provided by connman. It's probably safe to remove this line, though,
because the ethernet plugin package is virtual, too (see connman.inc above).

Best regards,
Andreas

>      dhcp-client \
>      e2fsprogs \
> 

Re: [PATCH] connman: add alias for connman

[Richard Purdie]

On Wed, 2016-05-25 at 15:11 +0800, Zhenbo Gao wrote:
> connman get conflicts with networkmanager when building
> the project.
> 
> here introduce alias virtual/networkmanager for these
> two recipes, so setting PREFERRED_PROVIDER to the proper
> one can solve the conflicts.
> 
> this patch is for connman recipe.

We do not use the virtual/* namespace in package dependencies, only in
the build time DEPENDS/PROVIDES space. This is because there is no way
to indicate how the package manager should resolve such things amongst
many other reasons.

Cheers,

Richard

Re: [PATCH] connman: add alias for connman

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 249 bytes --]

On 25 May 2016 at 08:11, Zhenbo Gao <zhenbo.gao@windriver.com> wrote:

> connman get conflicts with networkmanager when building
> the project.
>

Do you mean installing connman and networkmanager at the same time breaks
networking?

Ross

[-- Attachment #2: Type: text/html, Size: 660 bytes --]

Re: [PATCH] connman: add alias for connman

[zhenbo]

[-- Attachment #1: Type: text/plain, Size: 943 bytes --]



On 2016年05月25日 16:30, Burton, Ross wrote:
>
> On 25 May 2016 at 08:11, Zhenbo Gao <zhenbo.gao@windriver.com 
> <mailto:zhenbo.gao@windriver.com>> wrote:
>
>     connman get conflicts with networkmanager when building
>     the project.
>
>
> Do you mean installing connman and networkmanager at the same time 
> breaks networking?
>
Nope, if my project include connman and networkmanager at the same time, 
the compile step will fail:
...
Computing transaction...error: Can't install 
xfce4-power-manager-1.4.4-r0.0@corei7_64: unable to install provider for 
networkmanager:
error:     networkmanager-1.0.4-r0.0@corei7_64 conflicts with 
connman-1.30-r0.0@corei7_64
...
The dependence relationship is:
xfce4-power-manager --> networkmanager
packagegroup-self-hosted --> connman

Anyway, the patch is not suitable, Koen Kooi has suggested me another 
way to solve my problem.

Thanks,
Zhenbo

> Ross


[-- Attachment #2: Type: text/html, Size: 2388 bytes --]

Re: [PATCH] connman: add alias for connman

[zhenbo]

On 2016年05月25日 16:05, Richard Purdie wrote:
> On Wed, 2016-05-25 at 15:11 +0800, Zhenbo Gao wrote:
>> connman get conflicts with networkmanager when building
>> the project.
>>
>> here introduce alias virtual/networkmanager for these
>> two recipes, so setting PREFERRED_PROVIDER to the proper
>> one can solve the conflicts.
>>
>> this patch is for connman recipe.
> We do not use the virtual/* namespace in package dependencies, only in
> the build time DEPENDS/PROVIDES space. This is because there is no way
> to indicate how the package manager should resolve such things amongst
> many other reasons.
Hi Richard & Andreas,

Thanks for replying this mail.

The situation on my side is: if my project include connman and 
networkmanager at the same time, the compile step will fail:
...
Computing transaction...error: Can't install 
xfce4-power-manager-1.4.4-r0.0@corei7_64: unable to install provider for 
networkmanager:
error:     networkmanager-1.0.4-r0.0@corei7_64 conflicts with 
connman-1.30-r0.0@corei7_64
...
The dependence relationship is:
xfce4-power-manager --> networkmanager
packagegroup-self-hosted --> connman

Anyway, the patch is not suitable, Koen Kooi(koen@dominion.thruhere.net) 
has suggested me to fix packagegroup-self-hosted to not specify a 
connection manager.
When i am trying to fix my issue, i was wondering why self-hosted needs 
connman ? if self-hosted really don't need connman, the way provided by 
koen is the best solution for this issue.

The draft patch looks like:
---
--- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
@@ -28,8 +28,6 @@ RDEPENDS_packagegroup-self-hosted = "\
      "

  RDEPENDS_packagegroup-self-hosted-host-tools = "\
-    connman \
-    connman-plugin-ethernet \
      dhcp-client \
      e2fsprogs \
      e2fsprogs-e2fsck \

What do you think? is that ok? if so, i will send an official patch for it.

Thanks,
Zhenbo

> Cheers,
>
> Richard
>

Re: [PATCH] connman: add alias for connman

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 803 bytes --]

On 25 May 2016 at 10:41, zhenbo <zhenbo.gao@windriver.com> wrote:

> The draft patch looks like:
> ---
> --- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> @@ -28,8 +28,6 @@ RDEPENDS_packagegroup-self-hosted = "\
>      "
>
>  RDEPENDS_packagegroup-self-hosted-host-tools = "\
> -    connman \
> -    connman-plugin-ethernet \
>      dhcp-client \
>      e2fsprogs \
>      e2fsprogs-e2fsck \
>
> What do you think? is that ok? if so, i will send an official patch for it.
>

I don't think self-hosted should include a specific network manager, and
you can also remove dhcp-client whilst you're there.  The image that pulls
in this packagegroup should handle any networking requirements.

Ross

[-- Attachment #2: Type: text/html, Size: 1443 bytes --]