Don't capture stderr in _check_and_log_subprocess.

anntzer · anntzer · commit 31e1d95edad4 · 2019-06-26T17:11:34.000+02:00
... to avoid capturing kpsewhich's "warning: running with administrator
privileges" (that occurs at the only place where we actually care about
the return value of _check_and_log_subprocess).
diff --git a/lib/matplotlib/cbook/__init__.py b/lib/matplotlib/cbook/__init__.py
@@ -2030,24 +2030,25 @@ def _pformat_subprocess(command):
 
 def _check_and_log_subprocess(command, logger, **kwargs):
     """
-    Run *command* using `subprocess.check_output`.  If it succeeds, return the
-    output (stdout and stderr); if not, raise an exception whose text includes
-    the failed command and captured output.  Both the command and the output
-    are logged at DEBUG level on *logger*.
+    Run *command*.  If it succeeds, return the its stdout output; if not, raise
+    an exception whose text includes the failed command and captured stdout and
+    stderr output.  Both the command and the output are logged at DEBUG level
+    on *logger*.
     """
     logger.debug('%s', _pformat_subprocess(command))
-    try:
-        report = subprocess.check_output(
-            command, stderr=subprocess.STDOUT, **kwargs)
-    except subprocess.CalledProcessError as exc:
+    proc = subprocess.run(
+        command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, **kwargs)
+    if proc.returncode:
         raise RuntimeError(
-            'The command\n'
-            '    {}\n'
-            'failed and generated the following output:\n'
-            '{}'
-            .format(_pformat_subprocess(command), exc.output.decode('utf-8')))
-    logger.debug(report)
-    return report
+            f"The command\n"
+            f"    {_pformat_subprocess(command)}\n"
+            f"failed and generated the following output:\n"
+            f"{proc.stdout.decode('utf-8')}\n"
+            f"and the following error:\n"
+            f"{proc.stderr.decode('utf-8')}")
+    logger.debug("stdout:\n%s", proc.stdout)
+    logger.debug("stderr:\n%s", proc.stderr)
+    return proc.stdout
 
 
 def _check_isinstance(types, **kwargs):
