summaryrefslogtreecommitdiff
path: root/meta/classes/insane.bbclass
diff options
context:
space:
mode:
authorChristopher Larson <chris_larson@mentor.com>2013-06-10 13:47:07 -0700
committerRichard Purdie <richard.purdie@linuxfoundation.org>2013-06-11 15:45:24 +0100
commit8f5675e6d3eb8473f367de72d2d3d8a702011a47 (patch)
treef59cce56a4a315bd0a56d5b588416a3688a57156 /meta/classes/insane.bbclass
parent4ec66dc5f38913056942e1288c0b01a00ea0551f (diff)
downloadopenembedded-core-8f5675e6d3eb8473f367de72d2d3d8a702011a47.tar.gz
openembedded-core-8f5675e6d3eb8473f367de72d2d3d8a702011a47.tar.bz2
openembedded-core-8f5675e6d3eb8473f367de72d2d3d8a702011a47.zip
insane.bbclass: quiet package errors not in *_QA
Currently, if a package QA type from package.bbclass is not listed in ERROR_QA, it gets shown, regardless of whether it's in WARN_QA. This differs from the behavior of the rest of the QA handling (which doesn't call package_qa_handle_error at all if it's in neither _QA variable), and is nonintuitive. Change this to use bb.note() if it's listed neither in ERROR_QA nor WARN_QA, so it ends up in the task logs, but doesn't clutter up the user's output. Signed-off-by: Christopher Larson <chris_larson@mentor.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/insane.bbclass')
-rw-r--r--meta/classes/insane.bbclass6
1 files changed, 4 insertions, 2 deletions
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index c0910057f7..2b305386c6 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -143,9 +143,11 @@ def package_qa_handle_error(error_class, error_msg, d):
bb.error("QA Issue: %s" % error_msg)
d.setVar("QA_SANE", False)
return False
- else:
+ elif error_class in (d.getVar("WARN_QA", True) or "").split():
bb.warn("QA Issue: %s" % error_msg)
- return True
+ else:
+ bb.note("QA Issue: %s" % error_msg)
+ return True
QAPATHTEST[libexec] = "package_qa_check_libexec"
def package_qa_check_libexec(path,name, d, elf, messages):