MNT: update how subprocess is called

tacaswell · tacaswell · commit 90086d7e42a4 · 2017-10-21T23:51:03.000-04:00
diff --git a/examples/tests/backend_driver_sgskip.py b/examples/tests/backend_driver_sgskip.py
@@ -54,7 +54,7 @@
 
 def run(arglist):
     try:
-        ret = subprocess.call(arglist)
+        ret = subprocess.check_output(arglist)
     except KeyboardInterrupt:
         sys.exit()
     else:
@@ -139,12 +139,15 @@ def drive(backend, directories, python=[sys.executable], switches=[]):
         tmpfile.close()
         start_time = time.time()
         program = [x % {'name': basename} for x in python]
-        ret = run(program + [tmpfile_name] + switches)
-        end_time = time.time()
-        print("%s %s" % ((end_time - start_time), ret))
-        os.remove(tmpfile_name)
-        if ret:
+        try:
+            ret = run(program + [tmpfile_name] + switches)
+        except subprocess.CalledProcessError:
             failures.append(fullpath)
+        finally:
+            end_time = time.time()
+            print("%s %s" % ((end_time - start_time), ret))
+            os.remove(tmpfile_name)
+
     return failures
 
 
