[oe] [meta-xfce][PATCH] meta-xfce/layer: explicitly set networkmanager to NETWORK_MANAGER

[Hongxu Jia]

On 11/13/2015 06:19 PM, Martin Jansa wrote:

> Why not use VIRTUAL-RUNTIME variable like in other places?

If use it, I will rename NETWORK_MANAGER with VIRTUAL-RUNTIME_networkmanager
in packagegroup-core-x11-sato, to see oe-core whether accept it or not.

> And if it's
> needed by xfce4-power-manager, then why don't you add it to RDEPENDS there?

It is reasonable, add the variable(NETWORK_MANAGER or
VIRTUAL-RUNTIME_networkmanager) to xfce4-power-manager'S RDEPENDS

//Hongxu

> On Fri, Nov 13, 2015 at 11:13 AM, Andreas Müller <
> schnitzeltony at googlemail.com> wrote:
>
>> On Fri, Nov 13, 2015 at 11:04 AM, Hongxu Jia <hongxu.jia at windriver.com>
>> wrote:
>>> The NETWORK_MANAGER is connman-gnome in oe-core by default,
>>> which conflicts with networkmanager. And networkmanager is
>>> required by xfce4-power-manager in meta-xfce. So we explicitly
>>> set networkmanager to NETWORK_MANAGER for meta-xfce.
>>>
>>> Signed-off-by: Hongxu Jia <hongxu.jia at windriver.com>
>>> ---
>>>   meta-xfce/conf/layer.conf | 3 +++
>>>   1 file changed, 3 insertions(+)
>>>
>>> diff --git a/meta-xfce/conf/layer.conf b/meta-xfce/conf/layer.conf
>>> index 55bcef6..e38d85d 100644
>>> --- a/meta-xfce/conf/layer.conf
>>> +++ b/meta-xfce/conf/layer.conf
>>> @@ -17,3 +17,6 @@ LAYERDEPENDS_xfce-layer += "openembedded-layer"
>>>   LAYERDEPENDS_xfce-layer += "gnome-layer"
>>>   LAYERDEPENDS_xfce-layer += "openembedded-layer"
>>>   LAYERDEPENDS_xfce-layer += "meta-python"
>>> +
>>> +# It is required by xfce4-power-manager
>>> +NETWORK_MANAGER = "networkmanager"
>>> --
>> I was wondering why I never needed such kind of patch and checked: In
>> my layers It is only used in packagegroup-core-x11-sato.
>>
>> So I am fine with this patch but could you mention
>> packagegroup-core-x11-sato in the commit message please.
>>
>> Andreas
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>