Re: [PATCH 1/2] Fix filespath for dylan

[Denys Dmytriyenko <denys@ti.com>]

On Mon, Aug 12, 2013 at 12:03:38AM -0400, Ian Geiser wrote:
> I can fix and resubmit. I found it while porting our 8148 build to Dylan. 
> Thanks!

Actually, never mind, I think I'm wrong on this one - looks like I'm not 
thinking straight at this late hour... :)


> > -----Original Message-----
> > From: Denys Dmytriyenko [mailto:denys@ti.com]
> > Sent: Sunday, August 11, 2013 11:30 PM
> > To: Ian Geiser
> > Cc: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [PATCH 1/2] Fix filespath for dylan
> > 
> > On Sun, Aug 11, 2013 at 07:41:34PM -0400, igeiser@devonit.com wrote:
> > > From: Ian Reinhart Geiser <igeiser@devonit.com>
> > >
> > > The old way of setting THISDIR in a bbappend clobbers THISDIR for
> > > other layers.  This change makes it use the new way that is done in
> > > dylan.
> > 
> > Thanks! I've been meaning to fix those for quite some time, but there
> > was always something more important... :)
> > 
> > 
> > > Signed-off-by: Ian Reinhart Geiser <igeiser@devonit.com>
> > > ---
> > >  recipes-bsp/formfactor/formfactor_0.0.bbappend |    5 ++---
> > >  1 file changed, 2 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/recipes-bsp/formfactor/formfactor_0.0.bbappend
> > b/recipes-bsp/formfactor/formfactor_0.0.bbappend
> > > index 4b8d0e6..491d403 100644
> > > --- a/recipes-bsp/formfactor/formfactor_0.0.bbappend
> > > +++ b/recipes-bsp/formfactor/formfactor_0.0.bbappend
> > > @@ -1,3 +1,2 @@
> > > -THISDIR := "${@os.path.dirname(bb.data.getVar('FILE', d, True))}"
> > > -FILESPATH =. "${@base_set_filespath(["${THISDIR}/${PN}"], d)}:"
> > > -
> > > +PRINC := "${@int(PRINC) + 1}"
> > > +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > 
> > The above should be _append since that is what =. operand does and also
> > since
> > the colon is at the end.
> > 
> > --
> > Denys
>