Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <rpurdie@rpsys.net>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: SDK and external toolchain
Date: Thu, 26 Jul 2012 20:58:50 +0100	[thread overview]
Message-ID: <1343332730.9574.4.camel@ted> (raw)
In-Reply-To: <50118D35.6070105@windriver.com>

On Thu, 2012-07-26 at 13:32 -0500, Mark Hatle wrote:
> On 7/26/12 1:14 PM, Chris Larson wrote:
> > On Thu, Jul 26, 2012 at 9:37 AM, Mark Hatle <mark.hatle@windriver.com> wrote:
> >>> index 44284c3..f5fd4d7 100644
> >>> --- a/meta/classes/toolchain-scripts.bbclass
> >>> +++ b/meta/classes/toolchain-scripts.bbclass
> >>> @@ -136,7 +136,7 @@ toolchain_create_sdk_env_script_for_installer () {
> >>>    #we get the cached site config in the runtime
> >>>    TOOLCHAIN_CONFIGSITE_NOCACHE = "${@siteinfo_get_files(d, True)}"
> >>>    TOOLCHAIN_CONFIGSITE_SYSROOTCACHE =
> >>> "${STAGING_DATADIR}/${TARGET_SYS}_config_site.d"
> >>> -TOOLCHAIN_NEED_CONFIGSITE_CACHE = "${TCLIBC} ncurses"
> >>> +TOOLCHAIN_NEED_CONFIGSITE_CACHE = "ncurses"
> >>
> >>
> >> That is incorrect.. the CONFIGSITE_CACHE should be generated for the TCLIBC.
> >> If you don't do that, then you will be running the same configure steps --
> >> looking for basic glibc information over and over and over, causing a fairly
> >> expensive performance penalty.
> >
> > No, he's right, this is a bug in toolchain-scripts.bbclass. We could
> > work around it in the recipe via RPROVIDES_${PN}, but
> > TOOLCHAIN_NEED_CONFIGSITE_CACHE feeds directly into task 'depends', so
> > it's pulling in ${TCLIBC} explicitly rather than the more accurate
> > 'virtual/libc'.
> >
> 
> The config site file though isn't called "virtual/libc".  My understanding what 
> that all that variable did was set the list of config site files, but didn't 
> directly affect the dependency mapping.  If it does, then there is a definite 
> issue w/ dependency vs file mappings.

The trouble as I understand it is we need to know both the dependency
name and the config/site cache name.

We'll probably have to change siteconfig.bbclass to generate libc_config
instead of ${PN}_config (create an intermediate variable the libc
recipes override).

We can then specify something like virtual/libc:libc in
TOOLCHAIN_NEED_CONFIGSITE_CACHE. Of course that probably still won't
work if you actually try and build meta-toolchain, we might just have to
skip things that don't exist...

Cheers,

Richard 




  reply	other threads:[~2012-07-26 20:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-26  7:14 SDK and external toolchain Matthieu CRAPET
2012-07-26 16:37 ` Mark Hatle
2012-07-26 18:14   ` Chris Larson
2012-07-26 18:32     ` Mark Hatle
2012-07-26 19:58       ` Richard Purdie [this message]
2013-03-19  9:03         ` Marcin Juszkiewicz
2013-03-19 12:26           ` Marcin Juszkiewicz
2012-07-27 13:24   ` Matthieu CRAPET

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=1343332730.9574.4.camel@ted \
    --to=rpurdie@rpsys.net \
    --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