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

Issue 2966713003: VS 2017 Update 3 Preview 3 with fixed SDK (Closed)

Created:
3 years, 5 months ago by brucedawson
Modified:
3 years, 5 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

VS 2017 Update 3 Preview 3 with fixed SDK This change switches the VS 2017 package to use the 10.0.15063.468 SDK and the third preview of VS 2017 Update 3. Packaging was done on a Windows Server 2016 VM, cleanly created for this purpose. Compiler was packaged up by downloading the VS 2017 Update 3 Preview 3, from https://www.visualstudio.com/vs/preview/, and then running this command (executable name was different): vs_professional.exe --add Microsoft.VisualStudio.Workload.NativeDesktop --add Microsoft.VisualStudio.Component.VC.ATLMFC --includeRecommended --passive Then the Windows 10.0.15063.468 SDK installer was used to install the Debuggers package and patch wrl\event.h (still broken in the SDK installed by VS 2017 Updat3 3). The packaging script was run like this: python depot_tools\win_toolchain\package_from_installed.py 2017 -w 10.0.15063.0 VS 2015 builds are still the default so this makes no immediate difference. R=dpranke@chromium.org BUG=683729 Review-Url: https://codereview.chromium.org/2966713003 Cr-Commit-Position: refs/heads/master@{#484715} Committed: https://chromium.googlesource.com/chromium/src/+/9da63a8364a4cdec18db79b537ac819d29dd4284

Patch Set 1 #

Patch Set 2 : Reset default to VS 2015 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M build/vs_toolchain.py View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 13 (9 generated)
brucedawson
Try bots passed with default temporarily set to VS 2017. Actual change to be committed ...
3 years, 5 months ago (2017-07-05 17:45:46 UTC) #7
Dirk Pranke
rs- lgtm.
3 years, 5 months ago (2017-07-06 02:32:04 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2966713003/20001
3 years, 5 months ago (2017-07-06 17:59:38 UTC) #10
commit-bot: I haz the power
3 years, 5 months ago (2017-07-06 19:38:48 UTC) #13
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9da63a8364a4cdec18db79b537ac...

Powered by Google App Engine
This is Rietveld 408576698