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

Issue 2950223002: Updating VS 2015 package with no vctip.exe (Closed)

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

Description

Updating VS 2015 package with no vctip.exe This packages VS 2015 with vctip.exe removed in order to avoid problems caused when it hangs around. This is also packaged with a new Windows 10 SDK - 10.0.15063.468, which fixes the header-file bugs that previously required patching. Packaging was done on a Windows Server 2016 VM, cleanly created for this purpose. The process for creating this package is: - Install depot tools, run ‘gclient’, and add depot_tools to the path - Install final version of VS 2015 Professional Update 3 with these options: * Visual C++ and make sure the three nodes underneath are also selected * Under Universal Windows App Development Tools make sure the "Tools (1.4.1) and Windows 10 SDK (10.0.14393)" node is selected Then the Windows 10.0.15063.468 SDK was installed, with all components. This also installs the x86 and x64 debuggers. Finally the packaging script (updated to filter out vctip.exe, with https://chromium-review.googlesource.com/c/544395/) was run: python depot_tools\win_toolchain\package_from_installed.py 2015 -w 10.0.15063.0 This CL does *not* change build\toolchain\win\setup_toolchain.py to request the 10.0.15063.0 SDK when building with DEPOT_TOOLS_WIN_TOOLCHAIN=0. That is because this SDK is not yet required (crrev.com/2938543002 is a benign NOP with the 14393 SDK). BUG=735226 Review-Url: https://codereview.chromium.org/2950223002 Cr-Commit-Position: refs/heads/master@{#481586} Committed: https://chromium.googlesource.com/chromium/src/+/48c3f86f33b07d26c075b502ef6d0ba063c74bf3

Patch Set 1 #

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

Messages

Total messages: 20 (12 generated)
brucedawson
This time it really is the last VS 2015 package! (and the fixed SDK worked). ...
3 years, 6 months ago (2017-06-22 17:12:25 UTC) #9
scottmg
15063.468 in the cl description, not .0, right? lgtm
3 years, 6 months ago (2017-06-22 17:19:12 UTC) #10
brucedawson
On 2017/06/22 17:19:12, scottmg wrote: > 15063.468 in the cl description, not .0, right? > ...
3 years, 6 months ago (2017-06-22 17:20:39 UTC) #11
scottmg
On 2017/06/22 17:20:39, brucedawson wrote: > On 2017/06/22 17:19:12, scottmg wrote: > > 15063.468 in ...
3 years, 6 months ago (2017-06-22 17:23:53 UTC) #12
brucedawson
> Just a couple refs there say .0. Or is it still in that folder? ...
3 years, 6 months ago (2017-06-22 17:29:54 UTC) #14
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/2950223002/1
3 years, 6 months ago (2017-06-22 17:31:06 UTC) #16
scottmg
On 2017/06/22 17:29:54, brucedawson wrote: > > Just a couple refs there say .0. Or ...
3 years, 6 months ago (2017-06-22 17:38:38 UTC) #17
commit-bot: I haz the power
3 years, 6 months ago (2017-06-22 17:41:00 UTC) #20
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/48c3f86f33b07d26c075b502ef6d...

Powered by Google App Engine
This is Rietveld 408576698