[oe-commits] Scott Garman : insane.bbclass: use bb.process.Popen instead of subprocess. check_output
git at git.openembedded.org
git at git.openembedded.org
Thu Jan 5 22:25:13 UTC 2012
Module: openembedded-core.git
Branch: master
Commit: 0443487fe0bc628db9b03306bdc9dcdb39a121dc
URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=0443487fe0bc628db9b03306bdc9dcdb39a121dc
Author: Scott Garman <scott.a.garman at intel.com>
Date: Thu Jan 5 13:12:50 2012 -0800
insane.bbclass: use bb.process.Popen instead of subprocess.check_output
subprocess.check_output was only introduced in Python v2.7, so we
cannot use it. This refactors the QA test to use bb.process.Popen
instead.
This fixes the error:
AttributeError: 'module' object has no attribute 'check_output'
It no longer checks the return status of prelink-rtld, as that
case was simply adding noise. This QA test is intended to only
warn about specific paths that binaries could be linking to, not
handle the case where there is a missing library.
Signed-off-by: Scott Garman <scott.a.garman at intel.com>
Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
---
meta/classes/insane.bbclass | 16 ++++++----------
1 files changed, 6 insertions(+), 10 deletions(-)
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index b8d4507..8d6b11c 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -227,16 +227,12 @@ def package_qa_check_unsafe_references_in_binaries(path, name, d, elf, messages)
sysroot_path_usr = sysroot_path + exec_prefix
try:
- ldd_output = sub.check_output(["prelink-rtld", "--root", sysroot_path, path])
- except sub.CalledProcessError as e:
- if e.returncode != 127:
- error_msg = pn + ": prelink-rtld aborted when processing %s" % path
- package_qa_handle_error("unsafe-references-in-binaries", error_msg, d)
- return False
- else:
- # Sometimes this is done deliberately (e.g, e2fsprogs), so only warn
- bb.warn("%s has missing library dependencies" % path)
- return
+ ldd_output = bb.process.Popen(["prelink-rtld", "--root", sysroot_path, path], stdout=sub.PIPE).stdout.read()
+ except bb.process.CmdError:
+ error_msg = pn + ": prelink-rtld aborted when processing %s" % path
+ package_qa_handle_error("unsafe-references-in-binaries", error_msg, d)
+ return False
+
if sysroot_path_usr in ldd_output:
error_msg = pn + ": %s links to something under exec_prefix" % path
package_qa_handle_error("unsafe-references-in-binaries", error_msg, d)
More information about the Openembedded-commits
mailing list