Re: [PATCH][for-danny] kernel bbclass: return to original directory in do_deploy

[Denys Dmytriyenko <denis@denix.org>]

On Thu, Apr 11, 2013 at 12:43:30PM +0000, Maupin, Chase wrote:
> > -----Original Message-----
> > From: Denys Dmytriyenko [mailto:denis@denix.org]
> > Sent: Wednesday, April 10, 2013 6:02 PM
> > To: Maupin, Chase
> > Cc: openembedded-core@lists.openembedded.org
> > Subject: Re: [OE-core] [PATCH][for-danny] kernel bbclass: return
> > to original directory in do_deploy
> > 
> > Ping.
> > 
> > Ross, is it in your queue for your next danny merge?
> 
> I see that one as already being commited to danny

Hmm, where? I don't see it here:

http://cgit.openembedded.org/openembedded-core/log/?h=danny


> > On Mon, Mar 18, 2013 at 12:39:19PM -0500, Chase Maupin wrote:
> > > * During the base kernel_do_deploy function the directory is
> > >   changed to DEPLOYDIR in order to do some cleanup and
> > symlinking.
> > >   However, the directory is not changed back to the original
> > >   starting directory ${S} at the end.  For append functions
> > this
> > >   means that the starting directory is not ${S} as expected but
> > >   instead ${DEPLOYDIR}.
> > >
> > >   For functions like the do_deploy_append in
> > >   recipes-kernel/linux/linux-dtb.inc there is an assumption
> > that
> > >   you are still in the source directory and not the DEPLOYDIR.
> > >   Without this change the .dtb files are not copied because the
> > >   check for the existence of ${DTS_FILE} which is a relative
> > >   path from the ${S} directory fails.  This means that the .dtb
> > >   files are not copied into the deploy directory and
> > subsequently
> > >   the deploy/images directory.
> > >
> > >   In the log.do_deploy file you will see lines like:
> > >
> > >     Warning: arch/arm/boot/dts/xxxxx.dts is not available!
> > >
> > > Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> > > ---
> > >  meta/classes/kernel.bbclass |    1 +
> > >  1 files changed, 1 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/meta/classes/kernel.bbclass
> > b/meta/classes/kernel.bbclass
> > > index 88ce561..9691c25 100644
> > > --- a/meta/classes/kernel.bbclass
> > > +++ b/meta/classes/kernel.bbclass
> > > @@ -529,6 +529,7 @@ kernel_do_deploy() {
> > >  	ln -sf ${KERNEL_IMAGE_BASE_NAME}.bin ${KERNEL_IMAGETYPE}
> > >
> > >  	cp ${COREBASE}/meta/files/deploydir_readme.txt
> > ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
> > > +	cd -
> > >  }
> > >  do_deploy[dirs] = "${DEPLOYDIR} ${B}"
> > >
> > > --
> > > 1.7.0.4
> > >
> > >
> > > _______________________________________________
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.linuxtogo.org/cgi-
> > bin/mailman/listinfo/openembedded-core
>