diff options
author | Christopher Larson <chris_larson@mentor.com> | 2013-06-10 13:47:07 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-06-11 15:55:59 +0100 |
commit | d4a7820eee932c9796920e9453be68f9b7eb8ade (patch) | |
tree | 981df9379cae762f8d53fc5e9e213594da239b39 /meta/classes | |
parent | 9db09a66579087fda9c30df994c06e740cc15a20 (diff) | |
download | poky-d4a7820eee932c9796920e9453be68f9b7eb8ade.tar.gz |
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.
(From OE-Core rev: 8f5675e6d3eb8473f367de72d2d3d8a702011a47)
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes')
-rw-r--r-- | meta/classes/insane.bbclass | 6 |
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): | |||
143 | bb.error("QA Issue: %s" % error_msg) | 143 | bb.error("QA Issue: %s" % error_msg) |
144 | d.setVar("QA_SANE", False) | 144 | d.setVar("QA_SANE", False) |
145 | return False | 145 | return False |
146 | else: | 146 | elif error_class in (d.getVar("WARN_QA", True) or "").split(): |
147 | bb.warn("QA Issue: %s" % error_msg) | 147 | bb.warn("QA Issue: %s" % error_msg) |
148 | return True | 148 | else: |
149 | bb.note("QA Issue: %s" % error_msg) | ||
150 | return True | ||
149 | 151 | ||
150 | QAPATHTEST[libexec] = "package_qa_check_libexec" | 152 | QAPATHTEST[libexec] = "package_qa_check_libexec" |
151 | def package_qa_check_libexec(path,name, d, elf, messages): | 153 | def package_qa_check_libexec(path,name, d, elf, messages): |