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

Issue 174143003: Don't use automatic toolchain if GYP_MSVS_VERSION overridden to another version (Closed)

Created:
6 years, 10 months ago by scottmg
Modified:
6 years, 10 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews
Visibility:
Public.

Description

Don't use automatic toolchain if GYP_MSVS_VERSION overridden to another version This accomplishes three things: - simple switch back to 2010 temporarily, if necessary - making sure that all bots don't switch at exactly the same time (almost all have GYP_MSVS_VERSION=2010 in their settings) - in particular, not having Official builders switch Unfortunately, this will need to be merged on to the 34 branch, as it has the current "auto-use" code, so otherwise it'd switch to 2013 as soon as depot_tools starts updating the toolchain (previously, the only check was whether the toolchain exists on the local hard drive). R=maruel@chromium.org BUG=323300 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252604

Patch Set 1 #

Patch Set 2 : Don't use automatic toolchain if GYP_MSVS_VERSION overridden to another version #

Patch Set 3 : . #

Patch Set 4 : . #

Patch Set 5 : reitveld #

Patch Set 6 : reitveld #

Total comments: 4

Patch Set 7 : upload #

Patch Set 8 : rebase #

Patch Set 9 : fix rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M build/gyp_chromium View 1 2 3 4 5 6 7 8 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
scottmg
6 years, 10 months ago (2014-02-21 03:34:10 UTC) #1
M-A Ruel
lgtm https://codereview.chromium.org/174143003/diff/90001/build/gyp_chromium File build/gyp_chromium (right): https://codereview.chromium.org/174143003/diff/90001/build/gyp_chromium#newcode447 build/gyp_chromium:447: gyp_msvs_version = os.environ.get('GYP_MSVS_VERSION') gyp_msvs_version = os.environ.get('GYP_MSVS_VERSION', '2013') https://codereview.chromium.org/174143003/diff/90001/build/gyp_chromium#newcode449 ...
6 years, 10 months ago (2014-02-21 13:20:55 UTC) #2
scottmg
https://codereview.chromium.org/174143003/diff/90001/build/gyp_chromium File build/gyp_chromium (right): https://codereview.chromium.org/174143003/diff/90001/build/gyp_chromium#newcode447 build/gyp_chromium:447: gyp_msvs_version = os.environ.get('GYP_MSVS_VERSION') On 2014/02/21 13:20:55, M-A Ruel wrote: ...
6 years, 10 months ago (2014-02-21 16:52:22 UTC) #3
scottmg
6 years, 10 months ago (2014-02-21 18:44:52 UTC) #4
Message was sent while issue was closed.
Committed patchset #9 manually as r252604 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698