matplotlib
diff --git a/‎lib/matplotlib/animation.py
Lines changed: 18 additions & 22 deletions b/‎lib/matplotlib/animation.py
Lines changed: 18 additions & 22 deletions
@@ -19,7 +19,7 @@
 import abc
 import base64
 import contextlib
-from io import BytesIO
+from io import BytesIO, TextIOWrapper
 import itertools
 import logging
 import os
@@ -346,12 +346,11 @@ def _run(self):
         # movie file.  *args* returns the sequence of command line arguments
         # from a few configuration options.
         command = self._args()
-        output = subprocess.PIPE
         _log.info('MovieWriter.run: running command: %s', command)
-        self._proc = subprocess.Popen(command, shell=False,
-                                      stdout=output, stderr=output,
-                                      stdin=subprocess.PIPE,
-                                      creationflags=subprocess_creation_flags)
+        PIPE = subprocess.PIPE
+        self._proc = subprocess.Popen(
+            command, stdin=PIPE, stdout=PIPE, stderr=PIPE,
+            creationflags=subprocess_creation_flags)
 
     def finish(self):
         '''Finish any processing for writing the movie.'''
@@ -392,10 +391,20 @@ def _args(self):
 
     def cleanup(self):
         '''Clean-up and collect the process used to write the movie file.'''
-        out, err = self._proc.communicate()
+        self._proc.wait()
         self._frame_sink().close()
-        _log.debug('MovieWriter -- Command stdout:\n%s', out)
-        _log.debug('MovieWriter -- Command stderr:\n%s', err)
+        # Use the encoding/errors that universal_newlines would use.
+        out = TextIOWrapper(self._proc.stdout).read()
+        err = TextIOWrapper(self._proc.stderr).read()
+        _log.debug("MovieWriter: stdout: %s", out)
+        _log.debug("MovieWriter: stderr: %s", err)
+        if self._proc.returncode:
+            raise RuntimeError('Error creating movie, return code {}\n'
+                               'stdout:\n'
+                               '{}\n'
+                               'stderr:\n'
+                               '{}\n'
+                               .format(self._proc.returncode, out, err))
 
     @classmethod
     def bin_path(cls):
@@ -520,19 +529,6 @@ def finish(self):
         self._run()
         MovieWriter.finish(self)  # Will call clean-up
 
-        # Check error code for creating file here, since we just run
-        # the process here, rather than having an open pipe.
-        if self._proc.returncode:
-            try:
-                stdout = [s.decode() for s in self._proc._stdout_buff]
-                stderr = [s.decode() for s in self._proc._stderr_buff]
-                _log.info("MovieWriter.finish: stdout: %s", stdout)
-                _log.info("MovieWriter.finish: stderr: %s", stderr)
-            except Exception as e:
-                pass
-            raise RuntimeError('Error creating movie, return code: {}'
-                               .format(self._proc.returncode))
-
     def cleanup(self):
         MovieWriter.cleanup(self)