Build director: define the environment just once for a set of cmds

Author: humitos

readthedocs/doc_builder/director.py

@@ -298,11 +298,12 @@ def run_build_job(self, job):
             return
 
         cwd = self.data.project.checkout_path(self.data.version.slug)
+        environment = self.vcs_environment
+        if job not in ("pre_checkout", "post_checkout"):
+            environment = self.build_environment
+
         commands = getattr(self.data.config.build.jobs, job, [])
         for command in commands:
-            environment = self.vcs_environment
-            if job not in ("pre_checkout", "post_checkout"):
-                environment = self.build_environment
             environment.run(*command.split(), escape_command=False, cwd=cwd)
 
     # Helpers