Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 0/2] Handle the hossttools directory when restoring from the sstate cache
Date: Fri, 28 Apr 2017 17:01:00 +0200	[thread overview]
Message-ID: <cover.1493391620.git.pkj@axis.com> (raw)

After the introduction of copying host tools to the build directory
and cleaning out $PATH, we got a problem with one of our tools. It
turned out that its configure.ac uses AC_PATH_PROG(PERL, perl) to
locate the perl interpreter, and uses that on the shebang line of the
installed tool. Previously it found /usr/bin/perl and used that, but
now it will find ${TMPDIR}/hosttools/perl and use that instead, which
means that if the tool is restored from the sstate cache in another
build directory than where it originated from, the path will be wrong.

These two patches adds a new variable (HOSTTOOLS_DIR) for the
${TMPDIR}/hosttools directory, and then makes sure it is handled by
the staging code so that any references to its value are properly
corrected when restoring from the sstate cache.

//Peter

The following changes since commit 7a0e795373653886452a7a2992ced10080711c26:

  build-appliance-image: Update to master head revision (2017-04-21 08:22:18 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib pkj/hosttools_dir
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=pkj/hosttools_dir

Peter Kjellerstedt (2):
  bitbake.conf: Add HOSTTOOLS_DIR for ${TMPDIR}/hosttools
  sstate.bbclass, staging.bbclass: Handle HOSTTOOLS_DIR when restoring
    state

 meta/classes/base.bbclass                           | 4 ++--
 meta/classes/sstate.bbclass                         | 2 +-
 meta/classes/staging.bbclass                        | 2 +-
 meta/conf/bitbake.conf                              | 3 +++
 meta/conf/layer.conf                                | 2 +-
 meta/recipes-kernel/kmod/depmodwrapper-cross_1.0.bb | 2 +-
 6 files changed, 9 insertions(+), 6 deletions(-)

-- 
2.12.0



             reply	other threads:[~2017-04-28 15:01 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-28 15:01 Peter Kjellerstedt [this message]
2017-04-28 15:01 ` [PATCH 1/2] bitbake.conf: Add HOSTTOOLS_DIR for ${TMPDIR}/hosttools Peter Kjellerstedt
2017-04-28 15:01 ` [PATCH 2/2] sstate.bbclass, staging.bbclass: Handle HOSTTOOLS_DIR when restoring state Peter Kjellerstedt
2017-04-28 15:06   ` Burton, Ross
2017-04-28 20:24     ` Peter Kjellerstedt
2017-04-29  8:52 ` [PATCH 0/2] Handle the hossttools directory when restoring from the sstate cache Richard Purdie
2017-04-29 18:07   ` Peter Kjellerstedt
2017-05-01  8:14     ` Richard Purdie
2017-05-03 21:12       ` Peter Kjellerstedt

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.1493391620.git.pkj@axis.com \
    --to=peter.kjellerstedt@axis.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