]> code.ossystems Code Review - openembedded-core.git/commitdiff
insane: optimise buildpath search
authorRoss Burton <ross.burton@intel.com>
Thu, 12 Jul 2018 13:44:06 +0000 (14:44 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sun, 29 Jul 2018 23:01:02 +0000 (00:01 +0100)
Instead of decoding every file we open as UTF-8 (with many errors as machine
code isn't UTF-8), convert the build path to the UTF-8 byte representation and
search for that instead.

(From OE-Core rev: ffb52d383bfe413cf31fef13663fe9937a146c76)

Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
meta/classes/insane.bbclass

index fa154608420126ec2b5fef4707dc554ad1c99534..eb2d96711e18545a97758b735e2f326b46e1ddb0 100644 (file)
@@ -534,9 +534,9 @@ def package_qa_check_buildpaths(path, name, d, elf, messages):
     if path.find(name + "/CONTROL/") != -1 or path.find(name + "/DEBIAN/") != -1:
         return
 
-    tmpdir = d.getVar('TMPDIR')
+    tmpdir = bytes(d.getVar('TMPDIR'), encoding="utf-8")
     with open(path, 'rb') as f:
-        file_content = f.read().decode('utf-8', errors='ignore')
+        file_content = f.read()
         if tmpdir in file_content:
             package_qa_add_message(messages, "buildpaths", "File %s in package contained reference to tmpdir" % package_qa_clean_path(path,d))