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

Issue 1576203002: Change VS 2015 Update 1 package to VM version (Closed)

Created:
4 years, 11 months ago by brucedawson
Modified:
4 years, 11 months ago
Reviewers:
scottmg
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Change VS 2015 Update 1 package to VM version The original VS 2015 Update 1 toolchain package was created on a development machine and was not reproducible. This package was created on a clean VM in hopes of making it reproducible. The steps to create it are: Install Windows 7 Professional, 64-bit Install VS 2015 Professional Update 1. Select custom install and these minimum options: - Visual C++ (which selects three sub-categories including MFC) - Universal Windows App Development Tools - Tools (1.2) and Windows 10 SDK (10.0.10586) Then download the Windows 10 SDK. The version must match the one installed with VS 2015 Update 1 or else there will be warnings about the need to uninstall the previous version. Select "Debugging Tools for Windows" in addition to the defaults (Windows App Certification Kit, Windows Software Development Kit). Note, however, that this just downloads the installers. A message will suggest that running setup.exe will install the requested components but actually you have to go into the Downloads\Windows Kits\10\StandaloneSDK\Installers directory to run the x86 and x64 debugger installers. Then install depot_tools and run: > python depot_tools\win_toolchain\package_from_installed.py 2015 The package is uploaded with this syntax (replace hash.zip): > python depot_tools\third_party\gsutil\gsutil cp hash.zip gs://chrome-wintoolchain/hash.zip This process arguably packages up more files than are needed. The sys32\api_ms-* files can't actually be used in this form - the Universal Windows 10 C Runtime needs to be installed. These files are also different on difference OSes, so packaging them doesn't really make sense. Also the VC\lib\onecore and VC\redist\onecore files are large and appear to be unneeded. But, removing these is not important. BUG=440500 Committed: https://crrev.com/94c4e7106c3783cf35f5fe00561ce445382fc0ae Cr-Commit-Position: refs/heads/master@{#368760}

Patch Set 1 #

Patch Set 2 : Tweaking description #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M build/vs_toolchain.py View 1 chunk +1 line, -1 line 1 comment Download

Dependent Patchsets:

Messages

Total messages: 14 (7 generated)
brucedawson
I'm currently doing a VS 2015 build with this package, which is working. I'm not ...
4 years, 11 months ago (2016-01-11 23:34:23 UTC) #3
scottmg
On 2016/01/11 23:34:23, brucedawson wrote: > I'm currently doing a VS 2015 build with this ...
4 years, 11 months ago (2016-01-11 23:36:56 UTC) #4
scottmg
https://codereview.chromium.org/1576203002/diff/20001/build/vs_toolchain.py File build/vs_toolchain.py (right): https://codereview.chromium.org/1576203002/diff/20001/build/vs_toolchain.py#newcode1 build/vs_toolchain.py:1: #!/usr/bin/env python (description comment) Is there still an option ...
4 years, 11 months ago (2016-01-11 23:37:01 UTC) #5
brucedawson
On 2016/01/11 23:37:01, scottmg wrote: > https://codereview.chromium.org/1576203002/diff/20001/build/vs_toolchain.py > File build/vs_toolchain.py (right): > > https://codereview.chromium.org/1576203002/diff/20001/build/vs_toolchain.py#newcode1 > ...
4 years, 11 months ago (2016-01-12 00:15:18 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1576203002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1576203002/20001
4 years, 11 months ago (2016-01-12 00:56:35 UTC) #10
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 11 months ago (2016-01-12 01:30:53 UTC) #12
commit-bot: I haz the power
4 years, 11 months ago (2016-01-12 01:32:36 UTC) #14
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/94c4e7106c3783cf35f5fe00561ce445382fc0ae
Cr-Commit-Position: refs/heads/master@{#368760}

Powered by Google App Engine
This is Rietveld 408576698