• Bruce Dawson's avatar
    Update toolchain package to fix dbghelp.dll bug · ac888745
    Bruce Dawson authored
    This change updates the VS 2017 package to fix a bad version of
    dbghelp.dll in the Debuggers package, caused by a now-obsolete
    workaround in the packaging script, removed in crrev.com/c/1086990.
    
    The package still uses VS 2017 Update 7.1 and the 10.0.17134.12 SDK.
    
    Packaging was done on a Windows Server 2016 VM, cleanly created for this
    purpose.
    
    Compiler was packaged up by downloading VS 2017 Update 7.1, from
    https://www.visualstudio.com/vs/, and then passing these parameters to
    the installer:
    
        --add Microsoft.VisualStudio.Workload.NativeDesktop
        --add Microsoft.VisualStudio.Component.VC.ATLMFC --includeRecommended
        --passive
    
    Then Add or Remove Programs was used to modify the 10.0.17134.0 SDK to add
    the Debuggers package.
    
    Then the packaging script was run like this:
    
      python depot_tools\win_toolchain\package_from_installed.py 2017 -w 10.0.17134.0
    
    The results were compared to make sure that there were no unintended changes. One
    quirk is that the new package is missing the arm/arm64 directories in Debuggers,
    which is correct, whereas the previous package was *not* missing them, for some
    reason. Other than that there are no unintended changes.
    
    Bug: 773476, 846313
    Change-Id: I43db2ea95999fb7b2aeb02ba078e70298b62ffad
    Reviewed-on: https://chromium-review.googlesource.com/1086992Reviewed-by: default avatarNico Weber <thakis@chromium.org>
    Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#565106}
    ac888745
vs_toolchain.py 19.1 KB