Merge pull request #150 from martinholmer/use-run

Switch from subprocess.check_call to newer subprocess.run

RELEASES.md

@@ -4,6 +4,23 @@ Go [here](https://github.com/PSLmodels~/Package-Builder/pulls?q=is%3Apr+is%3Aclo
 for a complete commit history.
 
 
+2018-12-18 Release 0.18.0
+-------------------------
+(last merged pull request is
+[#150](https://github.com/PSLmodels/Package-Builder/pull/150))
+
+**API Changes**
+- None
+
+**New Features**
+- Use newer `subprocess.run` in `os_call` utility function
+  [[#150](https://github.com/PSLmodels/Package-Builder/pull/150)
+  by Martin Holmer]
+
+**Bug Fixes**
+- None
+
+
 2018-12-17 Release 0.17.0
 -------------------------
 (last merged pull request is

pkgbld/utils.py

@@ -13,13 +13,14 @@
 
 def os_call(cmd):
     """
-    Execute specified cmd in operating-system subprocess.
+    Execute specified cmd string in operating-system subprocess.
     """
+    assert isinstance(cmd, str)
     try:
-        subprocess.check_call(cmd, shell=True)
+        subprocess.run(cmd, shell=True, check=True, stdout=subprocess.PIPE)
     except subprocess.CalledProcessError as err:
-        msg = 'non-zero return code {} generated by command:\n{}'
-        raise OSError(msg.format(err.returncode, cmd))
+        msg = 'non-zero return code {} generated by command:\n{}\n>output: {}'
+        raise OSError(msg.format(err.returncode, cmd, err.output))
 
 
 def file_revision(filename, pattern, replacement):