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:10 +0100 |
| commit | 35ab2c7b08359f22f74106339841f8134123adf4 (patch) | |
| tree | 54edc8d894724735271f23d2b197a699e98a41cd /meta/classes/insane.bbclass | |
| parent | 87110e971c548d9d3eb7296ea6772d18e0226e53 (diff) | |
| download | openembedded-core-contrib-35ab2c7b08359f22f74106339841f8134123adf4.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
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
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): f.write("%s: %s [%s]\n" % (p, error, type)) def package_qa_handle_error(error_class, error_msg, d): - package_qa_write_error(error_class, error_msg, d) if error_class in (d.getVar("ERROR_QA") or "").split(): + package_qa_write_error(error_class, error_msg, d) bb.error("QA Issue: %s [%s]" % (error_msg, error_class)) d.setVar("QA_SANE", False) return False elif error_class in (d.getVar("WARN_QA") or "").split(): + package_qa_write_error(error_class, error_msg, d) bb.warn("QA Issue: %s [%s]" % (error_msg, error_class)) else: bb.note("QA Issue: %s [%s]" % (error_msg, error_class)) |