Skip to content

Commit

Permalink
Change default Windows compiler to VS 2015
Browse files Browse the repository at this point in the history
The change to get_landmines.py is there because modifying this file
affects analyze behavior so that all tests run. Changing the printed
message is purely a side effect.

This change also removes some redundant INCLUDE paths. These are
unnecessary when building with VS 2015 (because it defaults to the
Windows 10 SDK) and actively harmful (they make the INCLUDE path
problematically long).

BUG=440500,584782

Review URL: https://codereview.chromium.org/1598493004

Cr-Commit-Position: refs/heads/master@{#373955}
  • Loading branch information
randomascii authored and Commit bot committed Feb 5, 2016
1 parent 5510c5b commit 4c17ac0
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 22 deletions.
10 changes: 6 additions & 4 deletions build/common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -5676,11 +5676,13 @@
],
}],
],
# 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\toolchain\win\setup_toolchain.py
'msvs_system_include_dirs': [
'<(windows_sdk_path)/Include/10.0.10586.0/shared',
'<(windows_sdk_path)/Include/10.0.10586.0/um',
'<(windows_sdk_path)/Include/10.0.10586.0/winrt',
'$(VSInstallDir)/VC/atlmfc/include',
],
'msvs_cygwin_shell': 0,
'msvs_disabled_warnings': [
Expand Down
4 changes: 1 addition & 3 deletions build/get_landmines.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,7 @@ def print_landmines():
print "Switched win x64 trybots from VS2010 to VS2012."
if (platform() == 'win' and builder() == 'ninja' and
gyp_msvs_version().startswith('2013')):
print "Switched win from VS2010 to VS2013."
print "Update to VS2013 Update 2."
print "Update to VS2013 Update 4."
print "Switch to VS2013"
if (platform() == 'win' and gyp_msvs_version().startswith('2015')):
print 'Switch to VS2015'
print 'Need to clobber everything due to an IDL change in r154579 (blink)'
Expand Down
22 changes: 10 additions & 12 deletions build/toolchain/win/setup_toolchain.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,18 +150,16 @@ def main():
vc_bin_dir = os.path.realpath(path)
break

# The Windows SDK include directories must be first. They both have a sal.h,
# and the SDK one is newer and the SDK uses some newer features from it not
# present in the Visual Studio one.
# Having the Windows SDK first is also the only way to control which SDK
# version is used.

if win_sdk_path:
additional_includes = ('{sdk_dir}\\Include\\10.0.10586.0\\shared;' +
'{sdk_dir}\\Include\\10.0.10586.0\\um;' +
'{sdk_dir}\\Include\\10.0.10586.0\\winrt;').format(
sdk_dir=win_sdk_path)
env['INCLUDE'] = additional_includes + env['INCLUDE']
# 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 = ('{sdk_dir}\\Include\\10.0.10586.0\\um;').format(
# sdk_dir=win_sdk_path)
# env['INCLUDE'] = additional_includes + env['INCLUDE']
env_block = _FormatAsEnvironmentBlock(env)
with open('environment.' + cpu, 'wb') as f:
f.write(env_block)
Expand Down
8 changes: 5 additions & 3 deletions build/vs_toolchain.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@
import gyp


# Use MSVS2013 as the default toolchain.
CURRENT_DEFAULT_TOOLCHAIN_VERSION = '2013'
# Use MSVS2015 as the default toolchain.
CURRENT_DEFAULT_TOOLCHAIN_VERSION = '2015'


def SetEnvironmentAndGetRuntimeDllDirs():
Expand Down Expand Up @@ -274,7 +274,6 @@ def _GetDesiredVsToolchainHashes():
# Update 1 with Debuggers, UCRT installers and ucrtbased.dll
return ['523b6c2d3df300b2c8538cdc0beac404726af051']
else:
# Default to VS2013.
return ['4087e065abebdca6dbd0caca2910c6718d2ec67f']


Expand Down Expand Up @@ -308,6 +307,9 @@ def Update(force=False):
depot_tools_win_toolchain):
import find_depot_tools
depot_tools_path = find_depot_tools.add_depot_tools_to_path()
# Necessary so that get_toolchain_if_necessary.py will put the VS toolkit
# in the correct directory.
os.environ['GYP_MSVS_VERSION'] = GetVisualStudioVersion()
get_toolchain_args = [
sys.executable,
os.path.join(depot_tools_path,
Expand Down

0 comments on commit 4c17ac0

Please sign in to comment.