Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCHv2] sanity.bbclass: add more information to error message about TUNE_PKGARCH missing in PACKAGE_ARCHS
Date: Fri, 18 Dec 2015 14:15:25 +0100	[thread overview]
Message-ID: <1450444525-12405-1-git-send-email-Martin.Jansa@gmail.com> (raw)
In-Reply-To: <FEE7EF40-0110-4F2B-9A24-FF9C48E6DFFC@gmail.com>

* sometimes it's hard to see what's wrong here
* error message:
  Error, the PACKAGE_ARCHS variable does not contain TUNE_PKGARCH (cortexa7t2hfhf-vfp-vfpv4-neon).
  doesn't help much to understand what exactly went wrong and where

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/classes/sanity.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index ad9dda8..9f35558 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -329,6 +329,7 @@ def check_sanity_validmachine(sanity_data):
     # Check that we don't have duplicate entries in PACKAGE_ARCHS & that TUNE_PKGARCH is in PACKAGE_ARCHS
     pkgarchs = sanity_data.getVar('PACKAGE_ARCHS', True)
     tunepkg = sanity_data.getVar('TUNE_PKGARCH', True)
+    defaulttune = sanity_data.getVar('DEFAULTTUNE', True)
     tunefound = False
     seen = {}
     dups = []
@@ -345,7 +346,7 @@ def check_sanity_validmachine(sanity_data):
         messages = messages + "Error, the PACKAGE_ARCHS variable contains duplicates. The following archs are listed more than once: %s" % " ".join(dups)
 
     if tunefound == False:
-        messages = messages + "Error, the PACKAGE_ARCHS variable does not contain TUNE_PKGARCH (%s)." % tunepkg
+        messages = messages + "Error, the PACKAGE_ARCHS variable (%s) for DEFAULTTUNE (%s) does not contain TUNE_PKGARCH (%s)." % (pkgarchs, defaulttune, tunepkg)
 
     return messages
 
-- 
2.6.4



  reply	other threads:[~2015-12-18 13:12 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-10 13:48 [PATCH][master][RESEND 00/15] ARM tune fixes and improvements Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 01/15] sanity.bbclass: add more information to error message about TUNE_PKGARCH missing in PACKAGE_ARCHS Martin Jansa
2015-12-10 21:11   ` Khem Raj
2015-12-18 13:15     ` Martin Jansa [this message]
2015-12-10 13:48 ` [PATCH][master][RESEND 02/15] tune-cortexr4.inc: fix PACKAGE_EXTRA_ARCHS Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 03/15] arch-armv7a: add missing space before ?= Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 04/15] arch-armv7a, tune-cortexa*: improve comment VFP -> HF Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 05/15] arch-armv7a, tune-cortexa*: improve indentation Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 06/15] arm/arch-arm*, tune-cortexa*, tune-thunderx.inc, powerpac/arch-powerpc64.inc: Use normal assignment Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 07/15] arch-armv[456]*.inc: improve indentation like armv7a Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 08/15] arch-armv5.inc: drop duplicate ARMPKGSFX_DSP and PACKAGE_EXTRA_ARCHS_tune-armv5tehf-vfp Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 09/15] arch-armv7a.inc: Fix PACKAGE_EXTRA_ARCHS for tune-armv7atb-vfpv3, tune-armv7atb-vfpv3d16, cortexa7thf-neon-vfpv4 Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 10/15] arch-armv7a.inc: add vfpv4 support also to softfp and big endiand tunes Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 11/15] feature-arm-{neon, vfp}.inc: refactor and fix issues Martin Jansa
2015-12-10 21:14   ` Khem Raj
2015-12-18 13:10     ` Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 12/15] feature-arm-vfp.inc: Further simplify with TUNE_CCARGS_MFLOAT Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 13/15] cortexa{7, 15, 17}: add VFPv4 tunes Martin Jansa
2015-12-10 13:48 ` [PATCH][master][RESEND 14/15] arch-armv7ve: add tune include for armv7ve and use it from cortexa7 and cortexa15 Martin Jansa
2015-12-18  3:13   ` Andre McCurdy
2015-12-18 13:13     ` Martin Jansa
2015-12-18 20:08       ` Andre McCurdy
2015-12-18 20:13         ` Martin Jansa
2015-12-18 20:29           ` Andre McCurdy
2015-12-18 20:45             ` Martin Jansa
2015-12-18 20:50               ` Andre McCurdy
2015-12-18 20:57                 ` Martin Jansa
2015-12-18 21:18                   ` Andre McCurdy
2015-12-10 13:48 ` [PATCH][master][RESEND 15/15] tune-*: use mcpu instead of mtune for ARM tunes Martin Jansa
2015-12-23 10:10 ` [PATCH][master][RESEND 00/15] ARM tune fixes and improvements Martin Jansa
2016-01-04 16:58   ` Burton, Ross

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=1450444525-12405-1-git-send-email-Martin.Jansa@gmail.com \
    --to=martin.jansa@gmail.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