diff options
author | Jason R. Coombs <jaraco@jaraco.com> | 2020-07-12 05:00:58 -0400 |
---|---|---|
committer | Jason R. Coombs <jaraco@jaraco.com> | 2020-07-12 05:00:58 -0400 |
commit | 80258b1004931aadc560459780911639ac17b6e7 (patch) | |
tree | ccc42c8730de72e0cedc2dce617011a084e119c5 /setuptools | |
parent | 6d3250fa1ea1a309e8e45737d0a1dccbbd3f95e4 (diff) | |
parent | 30de1af5460998284435dc8ec63dc259f1ef96de (diff) | |
download | external_python_setuptools-80258b1004931aadc560459780911639ac17b6e7.tar.gz external_python_setuptools-80258b1004931aadc560459780911639ac17b6e7.tar.bz2 external_python_setuptools-80258b1004931aadc560459780911639ac17b6e7.zip |
Merge branch 'clean' of https://github.com/pypa/distutils
Diffstat (limited to 'setuptools')
-rw-r--r-- | setuptools/_distutils/_msvccompiler.py | 8 | ||||
-rw-r--r-- | setuptools/_distutils/spawn.py | 8 |
2 files changed, 6 insertions, 10 deletions
diff --git a/setuptools/_distutils/_msvccompiler.py b/setuptools/_distutils/_msvccompiler.py index af8099a4..0e98692e 100644 --- a/setuptools/_distutils/_msvccompiler.py +++ b/setuptools/_distutils/_msvccompiler.py @@ -501,12 +501,8 @@ class MSVCCompiler(CCompiler) : log.debug("skipping %s (up-to-date)", output_filename) def spawn(self, cmd): - old_path = os.getenv('path') - try: - os.environ['path'] = self._paths - return super().spawn(cmd) - finally: - os.environ['path'] = old_path + env = dict(os.environ, path=self._paths) + return super().spawn(cmd, env=env) # -- Miscellaneous methods ----------------------------------------- # These are all used by the 'gen_lib_options() function, in diff --git a/setuptools/_distutils/spawn.py b/setuptools/_distutils/spawn.py index 0d1bd039..fc592d4a 100644 --- a/setuptools/_distutils/spawn.py +++ b/setuptools/_distutils/spawn.py @@ -20,7 +20,7 @@ if sys.platform == 'darwin': _cfg_target_split = None -def spawn(cmd, search_path=1, verbose=0, dry_run=0): +def spawn(cmd, search_path=1, verbose=0, dry_run=0, env=None): """Run another program, specified as a command list 'cmd', in a new process. 'cmd' is just the argument list for the new process, ie. @@ -49,7 +49,8 @@ def spawn(cmd, search_path=1, verbose=0, dry_run=0): if executable is not None: cmd[0] = executable - env = None + env = env if env is not None else dict(os.environ) + if sys.platform == 'darwin': global _cfg_target, _cfg_target_split if _cfg_target is None: @@ -68,8 +69,7 @@ def spawn(cmd, search_path=1, verbose=0, dry_run=0): 'now "%s" but "%s" during configure' % (cur_target, _cfg_target)) raise DistutilsPlatformError(my_msg) - env = dict(os.environ, - MACOSX_DEPLOYMENT_TARGET=cur_target) + env.update(MACOSX_DEPLOYMENT_TARGET=cur_target) try: proc = subprocess.Popen(cmd, env=env) |