All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [RFC v1 PATCH 00/16] populate perl-native into its own	directory
Date: Thu, 09 Jun 2011 09:04:38 +0100	[thread overview]
Message-ID: <1307606678.15712.130.camel@rex> (raw)
In-Reply-To: <4DE7C09F.9000104@mentor.com>

On Thu, 2011-06-02 at 09:55 -0700, Tom Rini wrote:
> On 06/02/2011 09:35 AM, Richard Purdie wrote:
> > On Thu, 2011-06-02 at 09:28 -0700, Tom Rini wrote:
> >>   Even if we're using the sstate
> >> cache from /foo/oecore/tmp over in /bar/oecore/tmp (and /foo/oecore/tmp
> >> is rm -rf'd) ?  Since we've got a create_wrapper around perl and
> >> perl${PV} it should be I suppose (or is easily added there), but I'd
> >> feel a lot better with some testing of the above case (and the updates
> >> to cpan*bbclass).
> > 
> > I only took the perl-native DEPENDS patch on the condition this gets
> > fixed properly. The patches that are there look to do that, at least for
> > OE-Core. If there are further issues we're going to have to take them as
> > they arise as I have an objection to crippling the build dependencies
> > because perl is broken. Really this could use some TLC from someone with
> > experience in the area...
> 
> Well, I guess I'd boil down what I said above into a request like this
> for v3:
> - Modify cpan.bbclass to NOT set PERL_INC / PERL_LIB / PERL_ARCHLIB /
> PERLHOSTLIB.

The first three of these are all about the *target* perl location and I
think we still need them due the mess that perl's build system is. With
the patch series in question they won't actually point at perl-native in
the target case and they are only really used for cross compiling
purposes.

PERLHOSTLIB is used by the target perl when cross compiling to find
native .so files. perl-native will always be present at this point and
again, it seems like a valid use case.

Summary is that I don't think perl-native is broken in any way but we do
need those variables.

> - In /scratch/oecore/tmp0 build the images that were built for v1
> - In /scratch/oecore/tmp1 build perl-native's full sstate cache.
> - Keep the sstate cache from tmp1 and otherwise rm -rf tmp1.
> - In /scratch/oecore/tmp2 using the sstate from tmp1, build the same
> images that were built for tmp0.

I'm confused by this test cycle. What do you mean by "build
perl-native's full sstate cache"?

I suspect you've asking for some partial sstate cache to be shared
between two builds?

Put simpler, you probably want:

in tmp0 "bitbake perl-native"
in tmp1, different location to tmp0, "bitbake core-image-sato" but
sharing the same sstate cache

Is that what you're thinking?

FWIW, I've been running locally with the patch series and I think I'd
like to merge it. If there are issues we can address them as and when
they're identified...

Cheers,

Richard




  reply	other threads:[~2011-06-09  8:08 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-01 13:18 [RFC v1 PATCH 00/16] populate perl-native into its own directory Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 01/16] native.bbclass: allow a native package to be populated into its own dir Dexuan Cui
2011-06-01 13:40   ` Phil Blundell
2011-06-02  6:04     ` Cui, Dexuan
2011-06-02 13:50       ` Cui, Dexuan
2011-06-01 13:18 ` [RFC v1 PATCH 02/16] perl-native: populate " Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 03/16] perlnative.bbclass: add the file Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 04/16] gnu-config-native: should depend on perl-native-runtime rather than perl-native Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 05/16] libcap: " Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 06/16] openssl: " Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 07/16] git: " Dexuan Cui
2011-06-10  0:10   ` Joshua Lock
2011-06-10  7:28     ` Cui, Dexuan
2011-06-01 13:18 ` [RFC v1 PATCH 08/16] coreutils: remove unnecessary dependency on perl Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 09/16] dpkg: should depend on perl-native-runtime rather than perl-native Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 10/16] webkit-gtk: " Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 11/16] perl: inherit perlnative Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 12/16] cpan.bbclass, cpan-base.bbclas: update them for the perlnative change Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 13/16] libxml-parser-perl: inherit perlnative Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 14/16] libconvert-asn1-perl: fix EXTRA_PERLFLAGS due to the perl-native change Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 15/16] libxml-simple-perl: fix EXTRA_PERLFLAGS due the the perlnative change Dexuan Cui
2011-06-01 13:18 ` [RFC v1 PATCH 16/16] icon-naming-utils-native: inherit perlnative Dexuan Cui
2011-06-01 13:38 ` [RFC v1 PATCH 00/16] populate perl-native into its own directory Richard Purdie
2011-06-01 17:17 ` Tom Rini
2011-06-01 17:56   ` Richard Purdie
2011-06-01 19:39     ` Tom Rini
2011-06-01 20:05       ` Phil Blundell
2011-06-01 20:42         ` Tom Rini
2011-06-01 20:45           ` Phil Blundell
2011-06-01 20:59             ` Tom Rini
2011-06-02 14:06               ` Richard Purdie
2011-06-02 14:25                 ` Tom Rini
2011-06-02 14:37                   ` Phil Blundell
2011-06-02 16:28                     ` Tom Rini
2011-06-02 16:35                       ` Richard Purdie
2011-06-02 16:55                         ` Tom Rini
2011-06-09  8:04                           ` Richard Purdie [this message]
2011-06-09  8:08                             ` Richard Purdie
2011-06-09 13:51                               ` Cui, Dexuan
2011-06-09 13:56                                 ` Richard Purdie
2011-06-10  6:13                                   ` Cui, Dexuan
2011-06-10 14:23                               ` Tom Rini
2011-06-10 14:26                             ` Tom Rini
2011-06-02 13:46           ` Cui, Dexuan

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=1307606678.15712.130.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.