[oe-commits] [openembedded-core] 12/65: insane.bbclass: write QA issues to log file only when they are in ERROR_QA or WARN_QA

[git at git.openembedded.org]

This is an automated email from the git hooks/post-receive script.

rpurdie pushed a commit to branch master-next
in repository openembedded-core.

commit 21a3e2a437551e02422b1bf270fbd4c75b94d533
Author: Martin Jansa <martin.jansa at gmail.com>
AuthorDate: Mon Aug 21 22:56:24 2017 +0200

    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 at gmail.com>
    Signed-off-by: Ross Burton <ross.burton at intel.com>
---
 meta/classes/insane.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index aa304f1..78b41ca 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))

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.