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

[Iorga, Cristian]

Also, please note that tools-test and tools-wispr use also the ${S} dir in install, so my code is in accordance to previous code (prior artwork :-) ).

-----Original Message-----
From: Iorga, Cristian 
Sent: Monday, February 25, 2013 9:19 AM
To: Burton, Ross
Cc: openembedded-core at lists.openembedded.org
Subject: RE: [OE-core] [PATCH] connman: enable connman client

Hello Ross,

1. As far as I have seen, building the connmanctl is the default, there is no --enable-client.
	Maybe I am wrong, I have extracted this info from the documentation.
2. Like you said, at the moment, the client executable is produced directly in the source directory, hence my install command.
	If I understand correctly, will be the same thing if I change into build directory?

Regards,
Cristian

-----Original Message-----
From: Ross Burton [mailto:ross.burton at intel.com] 
Sent: Saturday, February 23, 2013 10:34 AM
To: Iorga, Cristian
Cc: openembedded-core at lists.openembedded.org
Subject: Re: [OE-core] [PATCH] connman: enable connman client

Hi Cristian,

On Friday, 22 February 2013 at 20:10, Cristian Iorga wrote: 
> - --disable-client \

I'd prefer to see an explicit --enable-client here to make it clear what we're turning on and off.
 
> + install -m 0755 ${S}/client/connmanctl ${D}${bindir}

${S} (source) should be ${B} (build), although at the moment they are the same directory I've been slowly working on changing that.

Also the fact that you have to do this is depressing.  I wonder why upstream doesn't want to install any of the tools?

Ross