[OE-core] [PATCH V2] connman: enable connman client

Iorga, Cristian cristian.iorga at intel.com
Tue Feb 26 10:38:32 UTC 2013


Hello all,

So, what is the general agreement here?

I placed connmanctl in the same package as connman because it is a very valuable tool, which should go in by default if connman is present also.
It should not belong to connman-test, because it is used to control connman, not test it.
If you want it to be in a connman-ctl package, just let me know and I will do the changes, re-test, and re-send the patch.

Regards,
Cristian

-----Original Message-----
From: openembedded-core-bounces at lists.openembedded.org [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf Of Jack Mitchell
Sent: Tuesday, February 26, 2013 11:25 AM
To: openembedded-core at lists.openembedded.org
Subject: Re: [OE-core] [PATCH V2] connman: enable connman client

On 25/02/13 23:53, Saul Wold wrote:
> 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.

Yes, that was my suggestion. I'm blocked at the moment by a binutils failure I can't shift, so currently unable to contribute. I would suggest putting it in a new connman-client package which is a 2 minute change and test.

>
> Thanks
>     Sau!
>
>> I know Jack M. mentioned looking into this also, so let's be sure to 
>> coordinate this. -----Original Message-----
>> From: Iorga, Cristian
>> Sent: Monday, February 25, 2013 10:49 AM
>> To: openembedded-core at 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.iorga at 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 at lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


-- 

   Jack Mitchell (jack at embed.me.uk)
   Embedded Systems Engineer
   http://www.embed.me.uk

--


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




More information about the Openembedded-core mailing list