Update to avoid merge conflicts with PR jazzband#531.

Pull request 531, among many other things, includes a similar fix for the use
of `stdout` before it's been assigned. That implementation is a nicer read, so
I've switched over to it.

Author: davidt

pipeline/compilers/__init__.py

@@ -111,13 +111,11 @@ def execute_command(self, command, cwd=None, stdout_captured=None):
             else:
                 argument_list.extend(flattening_arg)
 
+        stdout = None
         try:
-            stdout_name = None
-
             # We always catch stdout in a file, but we may not have a use for it.
             temp_file_container = cwd or os.path.dirname(stdout_captured or "") or os.getcwd()
             with NamedTemporaryFile(delete=False, dir=temp_file_container) as stdout:
-                stdout_name = stdout.name
                 compiling = subprocess.Popen(argument_list, cwd=cwd,
                                              stdout=stdout,
                                              stderr=subprocess.PIPE)
@@ -138,8 +136,8 @@ def execute_command(self, command, cwd=None, stdout_captured=None):
             raise CompilerError(e)
         finally:
             # Decide what to do with captured stdout.
-            if stdout_name:
+            if stdout:
                 if stdout_captured:
-                    os.rename(stdout_name, os.path.join(cwd or os.curdir, stdout_captured))
+                    os.rename(stdout.name, os.path.join(cwd or os.curdir, stdout_captured))
                 else:
-                    os.remove(stdout_name)
+                    os.remove(stdout.name)