diff options
author | Martin Jansa <martin.jansa@gmail.com> | 2017-08-21 22:56:24 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-09-11 17:30:28 +0100 |
commit | 4a9d1836b96fde88b8584bcf799a493cc93f58b4 (patch) | |
tree | 1f2ecdccf4d9bb7357d793714491282ac2143396 /meta/classes/insane.bbclass | |
parent | aa0c24f3514e2ecdec2250aa6bcd37d5e2b573b8 (diff) | |
download | poky-4a9d1836b96fde88b8584bcf799a493cc93f58b4.tar.gz |
insane.bbclass: write QA issues to log file only when they are in ERROR_QA or WARN_QA
* QA check which aren't included in WARN_QA and ERROR_QA are shown
during the build only as NOTE message (not shown at all with default
knotty setting), so it might be surprising to see them later in qa.log
file
(From OE-Core rev: 35ab2c7b08359f22f74106339841f8134123adf4)
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/insane.bbclass')
-rw-r--r-- | meta/classes/insane.bbclass | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass index aa304f108e..78b41caf99 100644 --- a/meta/classes/insane.bbclass +++ b/meta/classes/insane.bbclass | |||
@@ -184,12 +184,13 @@ def package_qa_write_error(type, error, d): | |||
184 | f.write("%s: %s [%s]\n" % (p, error, type)) | 184 | f.write("%s: %s [%s]\n" % (p, error, type)) |
185 | 185 | ||
186 | def package_qa_handle_error(error_class, error_msg, d): | 186 | def package_qa_handle_error(error_class, error_msg, d): |
187 | package_qa_write_error(error_class, error_msg, d) | ||
188 | if error_class in (d.getVar("ERROR_QA") or "").split(): | 187 | if error_class in (d.getVar("ERROR_QA") or "").split(): |
188 | package_qa_write_error(error_class, error_msg, d) | ||
189 | bb.error("QA Issue: %s [%s]" % (error_msg, error_class)) | 189 | bb.error("QA Issue: %s [%s]" % (error_msg, error_class)) |
190 | d.setVar("QA_SANE", False) | 190 | d.setVar("QA_SANE", False) |
191 | return False | 191 | return False |
192 | elif error_class in (d.getVar("WARN_QA") or "").split(): | 192 | elif error_class in (d.getVar("WARN_QA") or "").split(): |
193 | package_qa_write_error(error_class, error_msg, d) | ||
193 | bb.warn("QA Issue: %s [%s]" % (error_msg, error_class)) | 194 | bb.warn("QA Issue: %s [%s]" % (error_msg, error_class)) |
194 | else: | 195 | else: |
195 | bb.note("QA Issue: %s [%s]" % (error_msg, error_class)) | 196 | bb.note("QA Issue: %s [%s]" % (error_msg, error_class)) |