Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reland of Change default Windows compiler to VS 2015 (patchset chromi…
…um#1 id:1 of https://codereview.chromium.org/1680533004/ ) Reason for revert: The GPU bot problem (a bug with swarming tests on component builds) is fixed with crrev.com/1676943002 Original issue's description: > Revert of Change default Windows compiler to VS 2015 (patchset chromium#1 id:1 of https://codereview.chromium.org/1679693002/ ) > > Reason for revert: > This breaks GPU win debug bots (https://build.chromium.org/p/chromium.gpu/builders/Win7%20Debug%20%28NVIDIA%29/builds/36633 for example, every GPU test is crashing) > > Original issue's description: > > Reland of Change default Windows compiler to VS 2015 (patchset chromium#1 id:1 of https://codereview.chromium.org/1678663002/ ) > > > > Reason for revert: > > Issues that forced revert of VS 2015 as default compiler are believed to be fixed. > > > > Original issue's description: > > > Revert of Change default Windows compiler to VS 2015 (patchset chromium#10 id:180001 of https://codereview.chromium.org/1598493004/ ) > > > > > > Reason for revert: > > > Speculative for https://code.google.com/p/chromium/issues/detail?id=498544#c20 . Will reland if this doesn't help. > > > > > > Original issue's description: > > > > Change default Windows compiler to VS 2015 > > > > > > > > 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 > > > > > > > > Committed: https://crrev.com/4c17ac0db59d7dbe78734950f83912ab564bd6fe > > > > Cr-Commit-Position: refs/heads/master@{#373955} > > > > > > TBR=dpranke@chromium.org,scottmg@chromium.org,sebmarchand@chromium.org,brucedawson@chromium.org > > > # Skipping CQ checks because original CL landed less than 1 days ago. > > > NOPRESUBMIT=true > > > NOTREECHECKS=true > > > NOTRY=true > > > BUG=440500,584782 > > > > > > Committed: https://crrev.com/bbc766aef1bd8cb0d9546fec9abf264e7d8c1d96 > > > Cr-Commit-Position: refs/heads/master@{#374013} > > > > TBR=dpranke@chromium.org,scottmg@chromium.org,sebmarchand@chromium.org,thakis@chromium.org > > # Not skipping CQ checks because original CL landed more than 1 days ago. > > BUG=440500,584782 > > > > Committed: https://crrev.com/20e99ecb08da22582d435c1a5bba0a5c96345da3 > > Cr-Commit-Position: refs/heads/master@{#374058} > > TBR=dpranke@chromium.org,scottmg@chromium.org,sebmarchand@chromium.org,thakis@chromium.org,brucedawson@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=440500,584782 > > Committed: https://crrev.com/03f6b0af0a3a0127f5e939095157079659ab7cc8 > Cr-Commit-Position: refs/heads/master@{#374062} TBR=dpranke@chromium.org,scottmg@chromium.org,sebmarchand@chromium.org,thakis@chromium.org,zmo@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=440500,584782 Review URL: https://codereview.chromium.org/1676263003 Cr-Commit-Position: refs/heads/master@{#374324}
- Loading branch information