[oe-commits] Richard Purdie : insane.bbclass: Improve ability to detect enabled tests

[git at git.openembedded.org]

Module: openembedded-core.git
Branch: master
Commit: 069992a502658e8e44b870601e2e189cd9596ec9
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=069992a502658e8e44b870601e2e189cd9596ec9

Author: Richard Purdie <richard.purdie at linuxfoundation.org>
Date:   Fri Sep 21 11:16:24 2012 +0100

insane.bbclass: Improve ability to detect enabled tests

Define an ALL_QA variable which can be used to determine which tests to
run. Improve the libdir test to work in the case it is set to raise an
error rather than a warning.

Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>

---

 meta/classes/insane.bbclass |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 425e93a..1fb8970 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -116,6 +116,8 @@ def package_qa_get_machine_dict():
 WARN_QA ?= "ldflags useless-rpaths rpaths unsafe-references-in-binaries unsafe-references-in-scripts staticdev libdir"
 ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch la2 pkgconfig la perms"
 
+ALL_QA = "${WARN_QA} ${ERROR_QA}"
+
 def package_qa_clean_path(path,d):
     """ Remove the common prefix from the path. In this case it is the TMPDIR"""
     return path.replace(d.getVar('TMPDIR',True),"")
@@ -722,7 +724,7 @@ python do_package_qa () {
             rdepends_sane = False
 
 
-    if 'libdir' in (d.getVar("WARN_QA", True) or "").split():
+    if 'libdir' in d.getVar("ALL_QA", True).split():
         package_qa_check_libdir(d)
 
     if not walk_sane or not rdepends_sane:
@@ -787,7 +789,7 @@ do_populate_sysroot[postfuncs] += "do_qa_staging "
 do_configure[postfuncs] += "do_qa_configure "
 
 python () {
-    tests = d.getVar('WARN_QA', True) + " " + d.getVar('ERROR_QA', True)
-    if tests.find("desktop") != -1:
+    tests = d.getVar('ALL_QA', True).split()
+    if "desktop" in tests:
         d.appendVar("PACKAGE_DEPENDS", "desktop-file-utils-native")
 }