summaryrefslogtreecommitdiff
path: root/meta/classes/buildstats.bbclass
diff options
context:
space:
mode:
authorStephano Cetola <stephano.cetola@linux.intel.com>2016-09-29 15:50:24 -0700
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-09-30 17:48:33 +0100
commit0d6b7276003f1afabc6de683f663540327d52bdc (patch)
tree8f54884c51aff575e8b392100b9e2a434c71eb18 /meta/classes/buildstats.bbclass
parent901ccb3e70e9036112c51acc6d18d05025f6e1bb (diff)
downloadopenembedded-core-0d6b7276003f1afabc6de683f663540327d52bdc.tar.gz
openembedded-core-0d6b7276003f1afabc6de683f663540327d52bdc.tar.bz2
openembedded-core-0d6b7276003f1afabc6de683f663540327d52bdc.zip
subprocess: remove Popen in favor of check_output
This begins moving away from the deprecated subprocess calls in an effort to eventually move to some more global abstraction using the run convenience method provided in python 3.5. [ YOCTO #9342 ] Signed-off-by: Stephano Cetola <stephano.cetola@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/buildstats.bbclass')
-rw-r--r--meta/classes/buildstats.bbclass6
1 files changed, 5 insertions, 1 deletions
diff --git a/meta/classes/buildstats.bbclass b/meta/classes/buildstats.bbclass
index 34ecb03861..8c2b7b3944 100644
--- a/meta/classes/buildstats.bbclass
+++ b/meta/classes/buildstats.bbclass
@@ -163,7 +163,11 @@ python run_buildstats () {
bs = os.path.join(bsdir, "build_stats")
with open(bs, "a") as f:
rootfs = d.getVar('IMAGE_ROOTFS', True)
- rootfs_size = subprocess.Popen(["du", "-sh", rootfs], stdout=subprocess.PIPE).stdout.read()
+ try:
+ rootfs_size = subprocess.check_output(["du", "-sh", rootfs],
+ stderr=subprocess.STDOUT).decode('utf-8')
+ except subprocess.CalledProcessError as e:
+ bb.error("Failed to get rootfs size: %s" % e.output)
f.write("Uncompressed Rootfs size: %s" % rootfs_size)
elif isinstance(e, bb.build.TaskFailed):