[oe] [PATCH] iw: build version 0.9.22 from git sources

[Reizer, Eyal]

Dear Paul,

> -----Original Message-----
> From: openembedded-devel-bounces at lists.openembedded.org
> [mailto:openembedded-devel-bounces at lists.openembedded.org] On Behalf Of
> Paul Menzel
> Sent: Monday, September 26, 2011 12:07 PM
> To: openembedded-devel at lists.openembedded.org
> Cc: Eyal Reizer
> Subject: Re: [oe] [PATCH] iw: build version 0.9.22 from git sources
> 
> Dear Eyal,
> 
> 
> Am Montag, den 26.09.2011, 11:27 +0300 schrieb Eyal Reizer:
> 
> how has this recipe been tested?

We have tested this newer iw version on am37x-evm and am180x-evm with a newer version of NLCP that supports wowlan (wake on wireless lan).
The support for this feature was added in this newer version.

> 
> > Signed-off-by: Eyal Reizer <eyalr at ti.com>
> > ---
> >  recipes/iw/iw_git.bb |   29 +++++++++++++++++++++++++++++
> >  1 files changed, 29 insertions(+), 0 deletions(-)  create mode
> 100644
> > recipes/iw/iw_git.bb
> >
> > diff --git a/recipes/iw/iw_git.bb b/recipes/iw/iw_git.bb new file
> mode
> > 100644 index 0000000..e4d0cd5
> > --- /dev/null
> > +++ b/recipes/iw/iw_git.bb
> > @@ -0,0 +1,29 @@
> > +DESCRIPTION = "nl80211 based CLI configuration utility for wireless
> devices"
> > +HOMEPAGE = "http://linuxwireless.org/en/users/Documentation/iw"
> > +SECTION = "base"
> > +PRIORITY = "optional"
> > +LICENSE = "BSD"
> 
> Could you create a inc file to not duplicate the header and the
> overrides below? Or can 0.9.20 be removed?
> 

This version is an evolution of the iw product and can actually can replace the 0.9.20 one unless there are users that prefer to lock to 0.9.20 public zip.
Not sure if it is worth creating an inc file as I do not believe we should hold multiple versions of this recipe.
We should maybe have just one that pulls from a zip and one that pulls from git for cases where the latest development version is needed.

> > +
> > +SRCREV = "0a236ef5f8e4ba7218aac7d0cdacf45673d5b35c"
> > +PR = "r1"
> 
> Please start with `r0`.
> 

Ok, I will submit a new version for that.

> > +PV = "0.9.22"
> > +PR_append = "+gitr${SRCPV}"
> > +
> > +DEPENDS = "libnl pkgconfig"
> > +
> > +SRC_URI = " \
> > +	git://git.sipsolutions.net/iw.git;protocol=git \
> > +	"
> > +CFLAGS += "-DCONFIG_LIBNL20"
> > +
> > +S = "${WORKDIR}/git/"
> > +
> > +do_compile() {
> > +	oe_runmake
> > +}
> > +
> > +do_install() {
> > +	oe_runmake DESTDIR=${D} install
> > +}
> > +
> > +
> 
> Please remove the new lines at the end.
> 


Ok, I will submit a new version for that

> 
> Thanks,
> 
> Paul