From 5d96223e312fe8276ab9684e13cb2af41536e48a Mon Sep 17 00:00:00 2001 From: Stephano Cetola Date: Thu, 29 Sep 2016 15:50:24 -0700 Subject: 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 ] (From OE-Core rev: 0d6b7276003f1afabc6de683f663540327d52bdc) Signed-off-by: Stephano Cetola Signed-off-by: Richard Purdie --- meta/classes/spdx.bbclass | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'meta/classes/spdx.bbclass') diff --git a/meta/classes/spdx.bbclass b/meta/classes/spdx.bbclass index 0c9276584c..89394d3a96 100644 --- a/meta/classes/spdx.bbclass +++ b/meta/classes/spdx.bbclass @@ -219,14 +219,13 @@ def hash_string(data): def run_fossology(foss_command, full_spdx): import string, re import subprocess - - p = subprocess.Popen(foss_command.split(), - stdout=subprocess.PIPE, stderr=subprocess.PIPE) - foss_output, foss_error = p.communicate() - if p.returncode != 0: + + try: + foss_output = subprocess.check_output(foss_command.split(), + stderr=subprocess.STDOUT).decode('utf-8') + except subprocess.CalledProcessError as e: return None - foss_output = unicode(foss_output, "utf-8") foss_output = string.replace(foss_output, '\r', '') # Package info -- cgit v1.2.3-54-g00ecf