From: Richard Purdie Date: Tue, 25 Aug 2009 15:37:50 +0000 (+0100) Subject: base.bbclass: Use subprocess rather than os.system for do_unpack X-Git-Tag: 2011-1~7051 X-Git-Url: https://code.ossystems.io/gitweb?a=commitdiff_plain;h=c2b4308119d32c07e9c7a120e06b78afd26ff58c;p=openembedded-core.git base.bbclass: Use subprocess rather than os.system for do_unpack gzip reports broken pipe errors with do_unpack on Fedora with certain builds of gzip and bash. By avoding python's SIGPIPE handler we can work correctly on these distributions. Patch based on a patch from the OE-devel mailing list, thanks to Khem Raj and Holger Freyther Signed-off-by: Richard Purdie --- diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass index 3704cce01a..4f1e2dd788 100644 --- a/meta/classes/base.bbclass +++ b/meta/classes/base.bbclass @@ -627,8 +627,16 @@ base_do_buildall() { } + +def subprocess_setup(): + import signal + # Python installs a SIGPIPE handler by default. This is usually not what + # non-Python subprocesses expect. + # SIGPIPE errors are known issues with gzip/bash + signal.signal(signal.SIGPIPE, signal.SIG_DFL) + def oe_unpack_file(file, data, url = None): - import bb, os + import bb, os, subprocess if not url: url = "file://%s" % file dots = file.split(".") @@ -694,7 +702,7 @@ def oe_unpack_file(file, data, url = None): cmd = "PATH=\"%s\" %s" % (bb.data.getVar('PATH', data, 1), cmd) bb.note("Unpacking %s to %s/" % (file, os.getcwd())) - ret = os.system(cmd) + ret = subprocess.call(cmd, preexec_fn=subprocess_setup, shell=True) os.chdir(save_cwd)