Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
@ 2012-09-05 14:27 Richard Purdie
  2012-09-05 22:21 ` Chris Larson
  0 siblings, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2012-09-05 14:27 UTC (permalink / raw)
  To: openembedded-core

If we don't do this, users can get extremely confused errors since the sanity tests
happen too late (after parsing) and don't see the warnings.

Also cleanup messages about merging configuration file changes to give better
hints at where the updated files may be.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 40d8211..8f42fca 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -333,20 +333,20 @@ def check_sanity(sanity_data):
     conf_version =  sanity_data.getVar('LOCALCONF_VERSION', True)
 
     if current_conf != conf_version:
-        messages = messages + "Your version of local.conf was generated from an older version of local.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/local.conf conf/local.conf.sample\" is a good way to visualise the changes.\n"
+        messages = messages + "Your version of local.conf was generated from an older version of local.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/local.conf ${COREBASE}/meta*/conf/local.conf.sample\" is a good way to visualise the changes.\n"
 
     # Check bblayers.conf is valid
     current_lconf = sanity_data.getVar('LCONF_VERSION', True)
     lconf_version = sanity_data.getVar('LAYER_CONF_VERSION', True)
     if current_lconf != lconf_version:
-        messages = messages + "Your version of bblayers.conf was generated from an older version of bblayers.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/bblayers.conf conf/bblayers.conf.sample\" is a good way to visualise the changes.\n"
+        messages = messages + "Your version of bblayers.conf was generated from an older version of bblayers.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/bblayers.conf ${COREBASE}/meta*/conf/bblayers.conf.sample\" is a good way to visualise the changes.\n"
 
     # If we have a site.conf, check it's valid
     if check_conf_exists("conf/site.conf", sanity_data):
         current_sconf = sanity_data.getVar('SCONF_VERSION', True)
         sconf_version = sanity_data.getVar('SITE_CONF_VERSION', True)
         if current_sconf != sconf_version:
-            messages = messages + "Your version of site.conf was generated from an older version of site.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/site.conf conf/site.conf.sample\" is a good way to visualise the changes.\n"
+            messages = messages + "Your version of site.conf was generated from an older version of site.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/site.conf ${COREBASE}/meta*/conf/site.conf.sample\" is a good way to visualise the changes.\n"
 
     assume_provided = sanity_data.getVar('ASSUME_PROVIDED', True).split()
     # Check user doesn't have ASSUME_PROVIDED = instead of += in local.conf
@@ -556,7 +556,7 @@ def check_sanity(sanity_data):
         messages = messages + "Error, you have a space in your COREBASE directory path. Please move the installation to a directory which doesn't include a space."
 
     if messages != "":
-        raise_sanity_error(messages, sanity_data)
+        raise_sanity_error(sanity_data.expand(messages), sanity_data)
 
 # Create a copy of the datastore and finalise it to ensure appends and 
 # overrides are set - the datastore has yet to be finalised at ConfigParsed
@@ -567,7 +567,7 @@ def copy_data(e):
 
 addhandler check_sanity_eventhandler
 python check_sanity_eventhandler() {
-    if bb.event.getName(e) == "BuildStarted" and e.data.getVar("BB_WORKERCONTEXT", True) != "1" and e.data.getVar("DISABLE_SANITY_CHECKS", True) != "1":
+    if bb.event.getName(e) == "ConfigParsed" and e.data.getVar("BB_WORKERCONTEXT", True) != "1" and e.data.getVar("DISABLE_SANITY_CHECKS", True) != "1":
         sanity_data = copy_data(e)
         check_sanity(sanity_data)
     elif bb.event.getName(e) == "SanityCheck":





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
  2012-09-05 14:27 [PATCH] sanity.bbclass: Move back to running at ConfigParsed time Richard Purdie
@ 2012-09-05 22:21 ` Chris Larson
  2012-09-05 22:42   ` Richard Purdie
  0 siblings, 1 reply; 6+ messages in thread
From: Chris Larson @ 2012-09-05 22:21 UTC (permalink / raw)
  To: Richard Purdie; +Cc: openembedded-core

On Wed, Sep 5, 2012 at 7:27 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> If we don't do this, users can get extremely confused errors since the sanity tests
> happen too late (after parsing) and don't see the warnings.

Can you elaborate on this? This commit message is extremely unclear.
If there's an open bug in bugzilla or something that could be referred
to here, that'd be helpful.
-- 
Christopher Larson



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
  2012-09-05 22:21 ` Chris Larson
@ 2012-09-05 22:42   ` Richard Purdie
  2012-09-05 22:49     ` Chris Larson
  2012-09-11  6:39     ` Martin Jansa
  0 siblings, 2 replies; 6+ messages in thread
From: Richard Purdie @ 2012-09-05 22:42 UTC (permalink / raw)
  To: Chris Larson; +Cc: openembedded-core

On Wed, 2012-09-05 at 15:21 -0700, Chris Larson wrote:
> On Wed, Sep 5, 2012 at 7:27 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > If we don't do this, users can get extremely confused errors since the sanity tests
> > happen too late (after parsing) and don't see the warnings.
> 
> Can you elaborate on this? This commit message is extremely unclear.
> If there's an open bug in bugzilla or something that could be referred
> to here, that'd be helpful.

Sorry, I should have elaborated.

Set an invalid MACHINE, try and build and you set all kinds of nasty
warnings and no sensible message about what is wrong.

Change LCONF_VERSION and you don't get the warning unless parsing
completes which it may or may not depending on the kind of change.

and so on.

The sanity.bbclass code is meant to be helpful in these scenarios and
when its most needed, it wasn't working.

FWIW, I think we do need to fix the way sanity is being triggered and
improve this (massively). Unfortunately I didn't have time to do that
right this second but lets file an enhancement request in the bugzilla
and lets see if we can improve things soon?

Cheers,

Richard





^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
  2012-09-05 22:42   ` Richard Purdie
@ 2012-09-05 22:49     ` Chris Larson
  2012-09-11  6:39     ` Martin Jansa
  1 sibling, 0 replies; 6+ messages in thread
From: Chris Larson @ 2012-09-05 22:49 UTC (permalink / raw)
  To: Richard Purdie; +Cc: openembedded-core

On Wed, Sep 5, 2012 at 3:42 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Wed, 2012-09-05 at 15:21 -0700, Chris Larson wrote:
>> On Wed, Sep 5, 2012 at 7:27 AM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>> > If we don't do this, users can get extremely confused errors since the sanity tests
>> > happen too late (after parsing) and don't see the warnings.
>>
>> Can you elaborate on this? This commit message is extremely unclear.
>> If there's an open bug in bugzilla or something that could be referred
>> to here, that'd be helpful.
>
> Sorry, I should have elaborated.
>
> Set an invalid MACHINE, try and build and you set all kinds of nasty
> warnings and no sensible message about what is wrong.
>
> Change LCONF_VERSION and you don't get the warning unless parsing
> completes which it may or may not depending on the kind of change.

That makes sense, thanks for the clarification.
-- 
Christopher Larson



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
  2012-09-05 22:42   ` Richard Purdie
  2012-09-05 22:49     ` Chris Larson
@ 2012-09-11  6:39     ` Martin Jansa
  2012-09-11  9:43       ` Paul Eggleton
  1 sibling, 1 reply; 6+ messages in thread
From: Martin Jansa @ 2012-09-11  6:39 UTC (permalink / raw)
  To: Richard Purdie

[-- Attachment #1: Type: text/plain, Size: 996 bytes --]

On Thu, Sep 6, 2012 at 12:42 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Wed, 2012-09-05 at 15:21 -0700, Chris Larson wrote:
>> On Wed, Sep 5, 2012 at 7:27 AM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>> > If we don't do this, users can get extremely confused errors since the sanity tests
>> > happen too late (after parsing) and don't see the warnings.
>>
>> Can you elaborate on this? This commit message is extremely unclear.
>> If there's an open bug in bugzilla or something that could be referred
>> to here, that'd be helpful.
>
> Sorry, I should have elaborated.
>
> Set an invalid MACHINE, try and build and you set all kinds of nasty
> warnings and no sensible message about what is wrong.

Today with latest oe-core and bitbake I've set MACHINE=pitz (instead
of spitz) and the output doesn't look very tidy. Shows couple of DEBUG
entries even without debug enabled and then actuall ERROR, see
attached log.

Cheers,

[-- Attachment #2: bitbake.log --]
[-- Type: application/octet-stream, Size: 14168 bytes --]

OE pitz@shr ~/shr-core $ bitbake -c cleansstate kmod
DEBUG: Removed the following variables from the environment: PSEUDO_BINDIR, PSEUDO_LOCALSTATEDIR, LESS, LESSOPEN, PSEUDO_LIBDIR, HG, LANG, INFOPATH, SHLVL, CONFIG_PROTECT_MASK, WINDOW, LD_LIBRARY_PATH, EDITOR, MANPATH, BBFETCH2, LD_PRELOAD, PYTHONPATH, GCC_SPECS, _, STY, PSEUDO_OPTS, TERMCAP, BUILDDIR, PSEUDO_PREFIX, LS_COLORS, PAGER, OPENGL_PROFILE
DEBUG: UI 'knotty' does not require extra cache!
DEBUG: Found bblayers.conf (/OE/shr-core/conf/bblayers.conf)
DEBUG: LOAD /OE/shr-core/conf/bblayers.conf
DEBUG: Adding layer /OE/shr-core/meta-jama
DEBUG: LOAD /OE/shr-core/meta-jama/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-shr
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-shr/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-aurora
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-aurora/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-fso
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-fso/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-android
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-android/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-oe
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-oe/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-efl
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-efl/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-gnome
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-gnome/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-gpe
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-gpe/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-initramfs
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-initramfs/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-openembedded/meta-systemd
DEBUG: LOAD /OE/shr-core/meta-openembedded/meta-systemd/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-osmocombb
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-osmocombb/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-nokia
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-nokia/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-htc
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-htc/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-palm
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-palm/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-openmoko
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-openmoko/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-smartphone/meta-samsung
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-samsung/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-browser
DEBUG: LOAD /OE/shr-core/meta-browser/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-handheld
DEBUG: LOAD /OE/shr-core/meta-handheld/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/meta-webos-shr
DEBUG: LOAD /OE/shr-core/meta-webos-shr/conf/layer.conf
DEBUG: Adding layer /OE/shr-core/openembedded-core/meta
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/layer.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/bitbake.conf
DEBUG: CONF conf/bitbake.conf:672: including conf/abi_version.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/abi_version.conf
DEBUG: CONF conf/bitbake.conf:673: including conf/site.conf
DEBUG: LOAD /OE/shr-core/conf/site.conf
DEBUG: CONF conf/bitbake.conf:674: including conf/auto.conf
DEBUG: LOAD /OE/shr-core/conf/auto.conf
DEBUG: CONF conf/bitbake.conf:675: including conf/local.conf
DEBUG: LOAD /OE/shr-core/conf/local.conf
DEBUG: CONF /OE/shr-core/conf/local.conf:18: including local-builds.inc
DEBUG: BB /OE/shr-core/conf/local-builds.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/conf/local-builds.inc
DEBUG: CONF conf/bitbake.conf:676: including conf/build/x86_64-linux.conf
DEBUG: CONF file 'conf/build/x86_64-linux.conf' not found
DEBUG: CONF conf/bitbake.conf:677: including conf/target/INVALID-oe-linux.conf
DEBUG: CONF file 'conf/target/INVALID-oe-linux.conf' not found
DEBUG: CONF conf/bitbake.conf:678: including conf/machine/pitz.conf
DEBUG: CONF file 'conf/machine/pitz.conf' not found
DEBUG: CONF conf/bitbake.conf:679: including conf/machine-sdk/x86_64.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/machine-sdk/x86_64.conf
DEBUG: CONF conf/bitbake.conf:680: including conf/distro/shr.conf
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-shr/conf/distro/shr.conf
DEBUG: CONF /OE/shr-core/meta-smartphone/meta-shr/conf/distro/shr.conf:9: including conf/distro/include/preferred-shr-versions.inc
DEBUG: BB /OE/shr-core/meta-smartphone/meta-shr/conf/distro/include/preferred-shr-versions.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-shr/conf/distro/include/preferred-shr-versions.inc
DEBUG: CONF conf/bitbake.conf:681: including conf/distro/defaultsetup.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:1: including conf/distro/include/default-providers.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/default-providers.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/default-providers.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:2: including conf/distro/include/default-versions.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/default-versions.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/default-versions.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:3: including conf/distro/include/default-distrovars.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/default-distrovars.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/default-distrovars.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:4: including conf/distro/include/world-broken.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/world-broken.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/world-broken.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:7: including conf/distro/include/tcmode-default.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/tcmode-default.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/tcmode-default.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/include/tcmode-default.inc:59: including conf/distro/include/as-needed.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/as-needed.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/as-needed.inc
DEBUG: CONF /OE/shr-core/openembedded-core/meta/conf/distro/defaultsetup.conf:10: including conf/distro/include/tclibc-eglibc.inc
DEBUG: BB /OE/shr-core/openembedded-core/meta/conf/distro/include/tclibc-eglibc.inc: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/distro/include/tclibc-eglibc.inc
DEBUG: CONF conf/bitbake.conf:682: including conf/documentation.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/documentation.conf
DEBUG: CONF conf/bitbake.conf:683: including conf/sanity.conf
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/conf/sanity.conf
DEBUG: BB configuration INHERITs:0: inheriting classes/base.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/base.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:4: inheriting classes/patch.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/patch.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/patch.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/patch.bbclass:8: inheriting classes/terminal.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/terminal.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/terminal.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:5: inheriting classes/staging.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/staging.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/staging.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:7: inheriting classes/mirrors.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/mirrors.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/mirrors.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:8: inheriting classes/utils.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/utils.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/utils.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:9: inheriting classes/utility-tasks.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/utility-tasks.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/utility-tasks.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:10: inheriting classes/metadata_scm.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/metadata_scm.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/metadata_scm.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/base.bbclass:11: inheriting classes/logging.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/logging.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/logging.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/buildhistory.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/buildhistory.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/buildhistory.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/buildhistory.bbclass:19: inheriting classes/package.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/package.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/package.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/package.bbclass:41: inheriting classes/packagedata.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/packagedata.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/packagedata.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/package.bbclass:42: inheriting classes/prserv.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/prserv.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/prserv.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/package.bbclass:43: inheriting classes/chrpath.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/chrpath.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/chrpath.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/blacklist.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/blacklist.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/blacklist.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/debian.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/debian.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/debian.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/shr-mirrors.bbclass
DEBUG: BB /OE/shr-core/meta-smartphone/meta-shr/classes/shr-mirrors.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/meta-smartphone/meta-shr/classes/shr-mirrors.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/package_ipk.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/package_ipk.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/package_ipk.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/insane.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/insane.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/insane.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/devshell.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/devshell.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/devshell.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/sstate.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/sstate.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/sstate.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/license.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/license.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/license.bbclass
DEBUG: BB configuration INHERITs:0: inheriting classes/sanity.bbclass
DEBUG: BB /OE/shr-core/openembedded-core/meta/classes/sanity.bbclass: handle(data, include)
DEBUG: LOAD /OE/shr-core/openembedded-core/meta/classes/sanity.bbclass
DEBUG: Clearing SRCREV cache due to cache policy of: clear
DEBUG: Using cache in '/OE/shr-core/tmp-eglibc/cache/local_file_checksum_cache.dat'
DEBUG: Using cache in '/OE/shr-core/tmp-eglibc/cache/bb_codeparser.dat'
ERROR:  OE-core's config sanity checker detected a potential misconfiguration.
    Either fix the cause of this error or at your own risk disable the checker (see sanity.conf).
    Following is the list of potential problems / advisories:

    Please set a valid MACHINE in your local.conf or environment
Toolchain tunings invalid:
No tuning found for default multilib.

ERROR: Execution of event handler 'check_sanity_eventhandler' failed

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] sanity.bbclass: Move back to running at ConfigParsed time
  2012-09-11  6:39     ` Martin Jansa
@ 2012-09-11  9:43       ` Paul Eggleton
  0 siblings, 0 replies; 6+ messages in thread
From: Paul Eggleton @ 2012-09-11  9:43 UTC (permalink / raw)
  To: openembedded-core; +Cc: Martin Jansa

On Tuesday 11 September 2012 08:39:39 Martin Jansa wrote:
> Today with latest oe-core and bitbake I've set MACHINE=pitz (instead
> of spitz) and the output doesn't look very tidy. Shows couple of DEBUG
> entries even without debug enabled and then actuall ERROR, see
> attached log.

I've just sent a patch to the bitbake list that fixes this, as well as a patch 
to the OE-Core list that gets rid of the not particularly useful tune check 
failures when MACHINE is invalid.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-09-11  9:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-05 14:27 [PATCH] sanity.bbclass: Move back to running at ConfigParsed time Richard Purdie
2012-09-05 22:21 ` Chris Larson
2012-09-05 22:42   ` Richard Purdie
2012-09-05 22:49     ` Chris Larson
2012-09-11  6:39     ` Martin Jansa
2012-09-11  9:43       ` Paul Eggleton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox