Patchwork [RFC] bitbake.conf: Change WORKDIR to split by PN and version into seperate directories

login
register
mail settings
Submitter Richard Purdie
Date Sept. 21, 2012, 2:24 p.m.
Message ID <1348237488.10108.83.camel@ted>
Download mbox | patch
Permalink /patch/36999/
State New
Headers show

Comments

Richard Purdie - Sept. 21, 2012, 2:24 p.m.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

As per the previous patch, this is really a request for comments, I can
see pros and cons...
Phil Blundell - Sept. 21, 2012, 2:27 p.m.
On Fri, 2012-09-21 at 15:24 +0100, Richard Purdie wrote:
> As per the previous patch, this is really a request for comments, I can
> see pros and cons...

I've been using this configuration locally for quite some time (it's
needed for rm_old_works in particular) and it seems to be an improvement
in general.

p.
Martin Jansa - Sept. 21, 2012, 3:18 p.m.
On Fri, Sep 21, 2012 at 03:27:48PM +0100, Phil Blundell wrote:
> On Fri, 2012-09-21 at 15:24 +0100, Richard Purdie wrote:
> > As per the previous patch, this is really a request for comments, I can
> > see pros and cons...
> 
> I've been using this configuration locally for quite some time (it's
> needed for rm_old_works in particular) and it seems to be an improvement
> in general.

Last time I was using rm_old_works without this change
https://github.com/shr-distribution/meta-smartphone/commit/0d4c17de1ba7343d82bb10c311754e29d573c485

I've seen a lot of /usr/src/debug/${EXTENDPE}${PV}-${PR}" on target from
-dbg packages, which doesn't look right, was this fixed in whatever packs 
source code to -dbg packages?

Cheers,
Richard Purdie - Sept. 22, 2012, 11:43 a.m.
On Fri, 2012-09-21 at 17:18 +0200, Martin Jansa wrote:
> On Fri, Sep 21, 2012 at 03:27:48PM +0100, Phil Blundell wrote:
> > On Fri, 2012-09-21 at 15:24 +0100, Richard Purdie wrote:
> > > As per the previous patch, this is really a request for comments, I can
> > > see pros and cons...
> > 
> > I've been using this configuration locally for quite some time (it's
> > needed for rm_old_works in particular) and it seems to be an improvement
> > in general.
> 
> Last time I was using rm_old_works without this change
> https://github.com/shr-distribution/meta-smartphone/commit/0d4c17de1ba7343d82bb10c311754e29d573c485
> 
> I've seen a lot of /usr/src/debug/${EXTENDPE}${PV}-${PR}" on target from
> -dbg packages, which doesn't look right, was this fixed in whatever packs 
> source code to -dbg packages?

Probably from the way splitfile2 works in package.bbclass. I think this
should be using ${PF} explicitly if that is what it wants...

Cheers,

Richard

Patch

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 4b6fdb7..bc870e7 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -334,7 +334,7 @@  HGDIR = "${CO_DIR}/hg"
 
 STAMP = "${TMPDIR}/stamps/${MULTIMACH_TARGET_SYS}/${PN}/${EXTENDPE}${PV}-${PR}"
 STAMPCLEAN = "${TMPDIR}/stamps/${MULTIMACH_TARGET_SYS}/${PN}/*-*"
-WORKDIR = "${TMPDIR}/work/${MULTIMACH_TARGET_SYS}/${PF}"
+WORKDIR = "${TMPDIR}/work/${MULTIMACH_TARGET_SYS}/${PN}/${EXTENDPE}${PV}-${PR}"
 T = "${WORKDIR}/temp"
 D = "${WORKDIR}/image"
 S = "${WORKDIR}/${BP}"