Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(800)

Unified Diff: src/native_client/site_scons/site_tools/target_platform_windows.py

Issue 6719038: Revert r4644, "Updated NaCl's build system to use SCons 2.0.1." (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: Created 9 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/native_client/site_scons/site_tools/target_platform_windows.py
diff --git a/src/native_client/site_scons/site_tools/target_platform_windows.py b/src/native_client/site_scons/site_tools/target_platform_windows.py
index b8cead30509debe84caf215f5bd23ec726ea64c8..9633d62d3333e8904f73b3bffb6be4b4398902fe 100755
--- a/src/native_client/site_scons/site_tools/target_platform_windows.py
+++ b/src/native_client/site_scons/site_tools/target_platform_windows.py
@@ -261,11 +261,9 @@ def generate(env):
# NOTE: SCons requires the use of this name, which fails gpylint.
"""SCons entry point for this tool."""
- # TODO(ncbray): Several sections here are gated out on to prevent failure on
- # non-windows platforms. This appears to be SCons issue 1720 manifesting
- # itself. A more principled fix would be nice.
-
- use_msvc_tools = env['PLATFORM'] in ('win32', 'cygwin')
+ # TODO: Several sections here are gated out on the mac to prevent failure.
+ # This appears to be SCons issue 1720 manifesting itself on Mac when using
+ # windows tools like msvs or msvc.
# Preserve some variables that get blown away by the tools.
saved = dict()
@@ -280,14 +278,14 @@ def generate(env):
env.AppendENVPath('LIB', os.environ.get('LIB', '[]'))
# Load various Visual Studio related tools.
- if use_msvc_tools:
+ if env['PLATFORM'] != 'darwin':
env.Tool('as')
env.Tool('msvs')
env.Tool('windows_hard_link')
pre_msvc_env = env['ENV'].copy()
- if use_msvc_tools:
+ if env['PLATFORM'] != 'darwin':
env.Tool('msvc')
env.Tool('mslib')
env.Tool('mslink')
@@ -416,7 +414,7 @@ def generate(env):
# TODO: mslink.py creates a shlibLinkAction which doesn't specify
# '$SHLINKCOMSTR' as its command string. This breaks --brief. For now,
# hack into the existing action and override its command string.
- if use_msvc_tools:
+ if env['PLATFORM'] != 'darwin':
env['SHLINKCOM'].list[0].cmdstr = '$SHLINKCOMSTR'
# Restore saved flags.
« no previous file with comments | « src/native_client/site_scons/site_tools/component_builders.py ('k') | src/native_client/tools/modular-build/treemappers.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698