<div dir="ltr">Just notice typo in subject s/thei/they/g, will resend soon, in the meantime is there some feedback on this?<div><br></div><div>Alternatively we can write them with NOTE: WARN: ERROR: prefix like with the logger, so that the qa.log can be filtered by severity of the issue.</div><div><br></div><div>TL;DR; version:</div><div>In my jenkins builds I&#39;m printing content of qa.log at the end of usually very long build log, so that they are easier to spot in one place and because I get a lot of &quot;version-going-backwards&quot; warnings which I&#39;m not interested in anymore at all, I&#39;ve removed them from ERROR_QA - but then I was wondering why I still see very long list of &quot;version-going-backwards&quot; warnings, but those were from qa.log. It would work for me fine to just &quot;grep -v ^NOTE: qa.log&quot; at the end of jenkins build if the alternative solution is more acceptable.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 21, 2017 at 8:56 AM, Martin Jansa <span dir="ltr">&lt;<a href="mailto:martin.jansa@gmail.com" target="_blank">martin.jansa@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">* QA check which aren&#39;t included in WARN_QA and ERROR_QA are shown<br>
  during the build only as NOTE message (not shown at all with default<br>
  knotty setting), so it might be surprising to see them later in qa.log<br>
  file<br>
<br>
Signed-off-by: Martin Jansa &lt;<a href="mailto:Martin.Jansa@gmail.com">Martin.Jansa@gmail.com</a>&gt;<br>
---<br>
 meta/classes/insane.bbclass | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass<br>
index b7177c9b32..44e4a01c92 100644<br>
--- a/meta/classes/insane.bbclass<br>
+++ b/meta/classes/insane.bbclass<br>
@@ -184,12 +184,13 @@ def package_qa_write_error(type, error, d):<br>
             f.write(&quot;%s: %s [%s]\n&quot; % (p, error, type))<br>
<br>
 def package_qa_handle_error(error_<wbr>class, error_msg, d):<br>
-    package_qa_write_error(error_<wbr>class, error_msg, d)<br>
     if error_class in (d.getVar(&quot;ERROR_QA&quot;) or &quot;&quot;).split():<br>
+        package_qa_write_error(error_<wbr>class, error_msg, d)<br>
         bb.error(&quot;QA Issue: %s [%s]&quot; % (error_msg, error_class))<br>
         d.setVar(&quot;QA_SANE&quot;, False)<br>
         return False<br>
     elif error_class in (d.getVar(&quot;WARN_QA&quot;) or &quot;&quot;).split():<br>
+        package_qa_write_error(error_<wbr>class, error_msg, d)<br>
         bb.warn(&quot;QA Issue: %s [%s]&quot; % (error_msg, error_class))<br>
     else:<br>
         bb.note(&quot;QA Issue: %s [%s]&quot; % (error_msg, error_class))<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.14.0<br>
<br>
</font></span></blockquote></div><br></div>