diff options
author | thakis <thakis@chromium.org> | 2016-03-21 11:20:30 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-21 18:21:51 +0000 |
commit | 0917363388c80cb57ee5ceba36dcd047b4bf1d19 (patch) | |
tree | 568ec84091909239ac9d00d65aa0563f0dd4bfd6 /build | |
parent | 898c18b238708881d15c971886f6f4da82df55eb (diff) | |
download | chromium_src-0917363388c80cb57ee5ceba36dcd047b4bf1d19.zip chromium_src-0917363388c80cb57ee5ceba36dcd047b4bf1d19.tar.gz chromium_src-0917363388c80cb57ee5ceba36dcd047b4bf1d19.tar.bz2 |
Make setup_toolchain.py run on non-Windows.
Also delete an unused, commented-out block of code.
BUG=495204
Review URL: https://codereview.chromium.org/1822633002
Cr-Commit-Position: refs/heads/master@{#382330}
Diffstat (limited to 'build')
-rw-r--r-- | build/toolchain/win/setup_toolchain.py | 30 |
1 files changed, 10 insertions, 20 deletions
diff --git a/build/toolchain/win/setup_toolchain.py b/build/toolchain/win/setup_toolchain.py index df92f15..4dd077c 100644 --- a/build/toolchain/win/setup_toolchain.py +++ b/build/toolchain/win/setup_toolchain.py @@ -50,10 +50,11 @@ def _ExtractImportantEnvironment(output_of_set): setting = os.path.dirname(sys.executable) + os.pathsep + setting env[var.upper()] = setting break - for required in ('SYSTEMROOT', 'TEMP', 'TMP'): - if required not in env: - raise Exception('Environment variable "%s" ' - 'required to be set to valid path' % required) + if sys.platform in ('win32', 'cygwin'): + for required in ('SYSTEMROOT', 'TEMP', 'TMP'): + if required not in env: + raise Exception('Environment variable "%s" ' + 'required to be set to valid path' % required) return env @@ -108,9 +109,10 @@ def _LoadToolchainEnv(cpu, sdk_dir): variables = '\n'.join(varlines) # Check that the json file contained the same environment as the .cmd file. - script = os.path.normpath(os.path.join(sdk_dir, 'Bin/SetEnv.cmd')) - assert _ExtractImportantEnvironment(variables) == \ - _ExtractImportantEnvironment(_LoadEnvFromBat([script, '/' + cpu])) + if sys.platform in ('win32', 'cygwin'): + script = os.path.normpath(os.path.join(sdk_dir, 'Bin/SetEnv.cmd')) + assert _ExtractImportantEnvironment(variables) == \ + _ExtractImportantEnvironment(_LoadEnvFromBat([script, '/' + cpu])) else: if 'GYP_MSVS_OVERRIDE_PATH' not in os.environ: os.environ['GYP_MSVS_OVERRIDE_PATH'] = _DetectVisualStudioPath() @@ -171,7 +173,7 @@ def main(): for cpu in cpus: # Extract environment variables for subprocesses. env = _LoadToolchainEnv(cpu, win_sdk_path) - env['PATH'] = runtime_dirs + os.path.pathsep + env['PATH'] + env['PATH'] = runtime_dirs + os.pathsep + env['PATH'] if cpu == target_cpu: for path in env['PATH'].split(os.pathsep): @@ -179,18 +181,6 @@ def main(): vc_bin_dir = os.path.realpath(path) break - # Add extra include directories here that need to be in front of the - # installed and packaged include directories. This may be needed in - # order to force a particular SDK version, such as to get VS 2013 to use - # the Windows 10 SDK. Beware of making the INCLUDE variable excessively - # long and be sure to make corresponding changes to build\common.gypi. - # Not currently used. - #if win_sdk_path: - # additional_includes = [ - # os.path.join(win_sdk_path, 'Include', '10.0.10586.0', p) - # for p in ['shared', 'um', 'winrt']] - # additional_includes = os.path.pathsep.join(additional_includes) - # env['INCLUDE'] = additional_includes + os.path.pathsep + env['INCLUDE'] env_block = _FormatAsEnvironmentBlock(env) with open('environment.' + cpu, 'wb') as f: f.write(env_block) |