<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>4752</bug_id>
          
          <creation_ts>2013-06-19 16:01:18 +0000</creation_ts>
          <short_desc>Some QA errors no longer enabled</short_desc>
          <delta_ts>2013-10-01 14:17:36 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>configuration</component>
          <version>1.5</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>VERIFIED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>High</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>1.5 M2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Paul Eggleton">bluelightning</reporter>
          <assigned_to name="Paul Eggleton">bluelightning</assigned_to>
          <cc>alexandrux.palalau</cc>
    
    <cc>jessica.zhang</cc>
    
    <cc>kergoth</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>Regression (Used to work)</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>33982</commentid>
    <comment_count>0</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-06-19 16:01:18 +0000</bug_when>
    <thetext>A recent change in master disabled QA errors for checks not mentioned in ERROR_QA or WARN_QA:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=d4a7820eee932c9796920e9453be68f9b7eb8ade

Unfortunately this has caused regressions with the default OE-Core and Poky values for these variables, e.g. &quot;installed-vs-shipped&quot; is not in ERROR_QA so this error will no longer be shown.

To fix this, we need to:

1) Determine if any QA checks other than &quot;installed-vs-shipped&quot; have also been turned off as a result of this change

2) Fix the ERROR_QA value in OE-Core

3) Fix the ERROR_QA value in Poky</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>33983</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-06-19 16:02:16 +0000</bug_when>
    <thetext>We&apos;ll also need to mention this in the migration notes for 1.5 as users may have set their own values for these variables.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34226</commentid>
    <comment_count>2</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-06-28 15:21:26 +0000</bug_when>
    <thetext>It turns out that OE-Core&apos;s default ERROR_QA and WARN_QA (from insane.bbclass) were fine here, it was just poky.conf that didn&apos;t mention a number of the checks in its values. A patch has been sent for review to add these back additionally with a lean towards making them errors rather than warnings.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34227</commentid>
    <comment_count>3</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-06-28 15:25:57 +0000</bug_when>
    <thetext>For reference, the list of QA checks that were affected by this issue:

already-stripped
compile-host-path
dep-cmp
files-invalid
incompatible-license
installed-vs-shipped
install-host-path
libdir
packages-list
perm-config
perm-line
perm-link
pkgv-undefined
pn-overrides
split-strip
var-undefined
xorg-driver-abi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34228</commentid>
    <comment_count>4</comment_count>
    <who name="Christopher Larson">kergoth</who>
    <bug_when>2013-06-28 15:28:33 +0000</bug_when>
    <thetext>That&apos;s a relief, thanks for looking into this, and sorry about the hassle.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34230</commentid>
    <comment_count>5</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-06-28 15:44:42 +0000</bug_when>
    <thetext>No problem. The silver lining is that some of these were defined as errors in OE-Core but were being treated as warnings in Poky so we had an undiscovered inconsistency already; also, I found that a bunch of the checks aren&apos;t documented in the reference manual section on insane.bbclass (I have added bug 4788 to fix the latter).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34272</commentid>
    <comment_count>6</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2013-07-02 09:31:02 +0000</bug_when>
    <thetext>Fix merged to master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=c1df5188bb2851bcc5a3f483b0a0b4c4f884e3c7</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37414</commentid>
    <comment_count>7</comment_count>
    <who name="Alexandru Palalau">alexandrux.palalau</who>
    <bug_when>2013-10-01 14:17:36 +0000</bug_when>
    <thetext>Swithing to verified.
Commit ID: a1d9b6df86b0829d591cae57ae351f6d96e63557</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>