summaryrefslogtreecommitdiff
path: root/meta
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2013-07-15 19:17:44 +0100
committerSaul Wold <sgw@linux.intel.com>2013-07-18 07:14:40 -0700
commit3bdbec1bdecc52828cbbf8108786ff076c981845 (patch)
tree7e067c979c6030bfc689ccb9af36c36e15155987 /meta
parent194e47e6d8d9b9ee98e0203f0ebb574084277c46 (diff)
downloadopenembedded-core-3bdbec1bdecc52828cbbf8108786ff076c981845.tar.gz
openembedded-core-3bdbec1bdecc52828cbbf8108786ff076c981845.tar.bz2
openembedded-core-3bdbec1bdecc52828cbbf8108786ff076c981845.zip
classes/insane: allow libdir QA check to be skipped using INSANE_SKIP
This path check isn't handled in the normal way where a QA check function is called for every file (there's some minor setup that we want to avoid doing for every file) so we need to check INSANE_SKIP explicitly. In the process, change the code structure a little bit so that we can report the package that contains the errant file. Fixes [YOCTO #4822]. Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Diffstat (limited to 'meta')
-rw-r--r--meta/classes/insane.bbclass38
1 files changed, 24 insertions, 14 deletions
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index b875ac08c1..75bd2e2b22 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -252,29 +252,39 @@ def package_qa_check_libdir(d):
"""
import re
- pkgd = d.getVar('PKGD', True)
+ pkgdest = d.getVar('PKGDEST', True)
base_libdir = d.getVar("base_libdir",True) + os.sep
libdir = d.getVar("libdir", True) + os.sep
exec_prefix = d.getVar("exec_prefix", True) + os.sep
messages = []
- my_files = []
-
- for root, dirs, files in os.walk(pkgd):
- for file in files:
- full_path = os.path.join(root,file)
- my_files.append(full_path[len(pkgd):])
lib_re = re.compile("^/lib.+\.so(\..+)?$")
exec_re = re.compile("^%s.*/lib.+\.so(\..+)?$" % exec_prefix)
- for file in my_files:
- if lib_re.match(file):
- if base_libdir not in file:
- messages.append("Found library in wrong location: %s" % file)
- if exec_re.match(file):
- if libdir not in file:
- messages.append("Found library in wrong location: %s" % file)
+ for root, dirs, files in os.walk(pkgdest):
+ if root == pkgdest:
+ # Skip subdirectories for any packages with libdir in INSANE_SKIP
+ skippackages = []
+ for package in dirs:
+ if 'libdir' in (d.getVar('INSANE_SKIP_' + package, True) or "").split():
+ bb.note("Package %s skipping libdir QA test" % (package))
+ skippackages.append(package)
+ for package in skippackages:
+ dirs.remove(package)
+ for file in files:
+ full_path = os.path.join(root, file)
+ rel_path = os.path.relpath(full_path, pkgdest)
+ if os.sep in rel_path:
+ package, rel_path = rel_path.split(os.sep, 1)
+ rel_path = os.sep + rel_path
+ if lib_re.match(rel_path):
+ if base_libdir not in rel_path:
+ messages.append("%s: found library in wrong location: %s" % (package, rel_path))
+ if exec_re.match(rel_path):
+ if libdir not in rel_path:
+ messages.append("%s: found library in wrong location: %s" % (package, rel_path))
+
if messages:
package_qa_handle_error("libdir", "\n".join(messages), d)