Re: [PATCH] shadow: Create recipe nativesdk-shadow

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Mon, 2013-09-23 at 22:14 +0100, Richard Purdie wrote:
> On Mon, 2013-09-23 at 21:01 +0200, David Nystrom wrote:
> > 
> > On Sep 23, 2013 8:01 PM, "Mark Hatle" <mark.hatle@windriver.com>
> > wrote:
> > >
> > > This looks to be the same as the 'native' package, but I don't think
> > you can just add a BBCLASSEXTEND = "nativesdk"
> > 
> > It mostly is and I cant, your right.
> 
> Why not? Just the SRC_URI or other issues?
> 
> > > Perhaps (in 1.6) someone should look at merging the three back
> > together.
> > 
> > Agree, the merged recipe will also get a bit messy, but probably
> > easier to maintain than in its current state.
> > 
> > > If this is the same as the native version, I'd suggest adding a
> > comment stating as much so we can hopefully keep them in sync.
> > 
> > Will do, will return with v2
> > 
> Well, I'm not happy with the idea of duplicating all this code.
> shadow-native shouldn't exist and this makes things worse. With a few
> minutes, I could write this patch:
> 
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=rpurdie/t2&id=0730423fcda165a12112182b093d2f6df2f9c5eb
> 
> which at least makes the situation a bit less ugly. I guess I did spend
> years doing these kinds of patches back when we had 50 versions of
> everything and BBCLASSEXTEND didn't yet exist. With a few more minutes
> perhaps I can get rid of the separate -native, now the differences are
> clear. meld is a nice too for this kind of work btw.

Better still, a conversion to BBCLASSEXTEND:

http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=rpurdie/t2&id=142d11b9b61bf18567cdb0527b7e63683c5afa1a

Cheers,

Richard