Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Ross Burton <ross.burton@intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [RFC][PATCH 0/2] WORKDIR and insane tweaks
Date: Tue, 13 Aug 2013 11:10:58 +0100	[thread overview]
Message-ID: <cover.1376388241.git.ross.burton@intel.com> (raw)

Hi,

Last night I was attempting a build with WORKDIR in a tmpfs to see what impact
it made on build time (the results were fairly interesting, I'll write them up
shortly).

I noticed that to move WORKDIR I had to replicate the full WORKDIR path logic,
which was a bit irritating but not that serious.  I then noticed that the
"rpaths" sanity test checks that WORKDIR is in fact TMPDIR/work, and does a
bb.fatal() if it isn't.  My best understanding here is that the check needs to
know the base of all possible WORKDIRS so that it can check for linkage directly
into another package's work directory.

Assuming that understanding is correct then both problems can be resolved with a
BASE_WORKDIR variable, which this series adds to bitbake.conf and then uses in
insane.bbclass.

Ross

The following changes since commit 16d522bcd1f1b7741577fa31fab7e2129da0cae9:

  maintainers.inc: reassign maintainers, remove obsolete recipes (2013-08-12 13:13:05 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ross/workdir

for you to fetch changes up to e91cbd17131101d1c470669acce66682ae06960a:

  insane: don't abort if workdir is not TMPDIR/work (2013-08-13 10:36:53 +0100)

----------------------------------------------------------------
Ross Burton (2):
      bitbake.conf: define WORKDIR in terms of BASE_WORKDIR
      insane: don't abort if workdir is not TMPDIR/work

 meta/classes/insane.bbclass |    5 +----
 meta/conf/bitbake.conf      |    3 ++-
 2 files changed, 3 insertions(+), 5 deletions(-)

Ross Burton (2):
  bitbake.conf: define WORKDIR in terms of BASE_WORKDIR
  insane: don't abort if workdir is not TMPDIR/work

 meta/classes/insane.bbclass |    5 +----
 meta/conf/bitbake.conf      |    3 ++-
 2 files changed, 3 insertions(+), 5 deletions(-)

-- 
1.7.10.4



             reply	other threads:[~2013-08-13 10:11 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-13 10:10 Ross Burton [this message]
2013-08-13 10:10 ` [PATCH 1/2] bitbake.conf: define WORKDIR in terms of BASE_WORKDIR Ross Burton
2013-08-13 10:11 ` [PATCH 2/2] insane: don't abort if workdir is not TMPDIR/work Ross Burton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1376388241.git.ross.burton@intel.com \
    --to=ross.burton@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox