[gnome-continuous-yocto/gnomeostree-3.28-rocko: 7599/8267] insane.bbclass: write QA issues to log file only when they are in ERROR_QA or WARN_QA

[Emmanuele Bassi <ebassi src gnome org>]

commit 4a9d1836b96fde88b8584bcf799a493cc93f58b4
Author: Martin Jansa <martin jansa gmail com>
Date:   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
    
    (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>

 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 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))