[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [38777] trunk/blender/release/scripts/ startup/bl_operators/screen_play_rendered_anim.py: fix for error in last commit if the program didnt exist at all.

Campbell Barton ideasman42 at gmail.com
Thu Jul 28 07:19:17 CEST 2011


Revision: 38777
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=38777
Author:   campbellbarton
Date:     2011-07-28 05:19:17 +0000 (Thu, 28 Jul 2011)
Log Message:
-----------
fix for error in last commit if the program didnt exist at all.

Modified Paths:
--------------
    trunk/blender/release/scripts/startup/bl_operators/screen_play_rendered_anim.py

Modified: trunk/blender/release/scripts/startup/bl_operators/screen_play_rendered_anim.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_operators/screen_play_rendered_anim.py	2011-07-28 05:09:31 UTC (rev 38776)
+++ trunk/blender/release/scripts/startup/bl_operators/screen_play_rendered_anim.py	2011-07-28 05:19:17 UTC (rev 38777)
@@ -112,22 +112,29 @@
         if preset == 'BLENDER24':
             # -----------------------------------------------------------------
             # Check blender is not 2.5x until it supports playback again
-            process = subprocess.Popen([player_path, '--version'],
-                                       stdout=subprocess.PIPE,
-                                       )
-            process.wait()
-            out = process.stdout.read()
-            process.stdout.close()
-            out_split = out.strip().split()
-            if out_split[0] == b'Blender':
-                if not out_split[1].startswith(b'2.4'):
-                    self.report({'ERROR'},
-                                "Blender %s does not support playback: %r" %
-                                (out_split[1].decode('ASCII'), player_path))
-                    return {'CANCELLED'}
-            del out, out_split, process
+            try:
+                process = subprocess.Popen([player_path, '--version'],
+                                           stdout=subprocess.PIPE,
+                                           )
+            except:
+                # ignore and allow the main execution to catch the problem.
+                process = None
+
+            if process is not None:
+                process.wait()
+                out = process.stdout.read()
+                process.stdout.close()
+                out_split = out.strip().split()
+                if out_split[0] == b'Blender':
+                    if not out_split[1].startswith(b'2.4'):
+                        self.report({'ERROR'},
+                                    "Blender %s doesn't support playback: %r" %
+                                    (out_split[1].decode(), player_path))
+                        return {'CANCELLED'}
+                del out, out_split
+            del process
             # -----------------------------------------------------------------
-            
+
             opts = ["-a", "-f", str(rd.fps), str(rd.fps_base), file]
             cmd.extend(opts)
         elif preset == 'DJV':




More information about the Bf-blender-cvs mailing list