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

Issue 1112753003: win: prefer amd64_x86 compiler on >= 2013, not just 2013 (Closed)

Created:
5 years, 7 months ago by scottmg
Modified:
5 years, 7 months ago
Reviewers:
Dirk Pranke, Nico
CC:
gyp-developer_googlegroups.com
Target Ref:
refs/heads/master
Project:
gyp
Visibility:
Public.

Description

win: prefer amd64_x86 compiler on >= 2013, not just 2013 [[ Because I always find binary string ops a bit odd: >>> '2003' >= '2013' False >>> '2005' >= '2013' False >>> '2008' >= '2013' False >>> '2010' >= '2013' False >>> '2012' >= '2013' False >>> '2013' >= '2013' True >>> '2015' >= '2013' True >>> '2017' >= '2013' True >>> '2090' >= '2013' True ]] R=dpranke@chromium.org, thakis@chromium.org BUG=chromium:482238 Committed: https://chromium.googlesource.com/external/gyp/+/8866260996c063f514dfafd0e05743a42709a49c

Patch Set 1 #

Patch Set 2 : handle e #

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

Messages

Total messages: 4 (1 generated)
scottmg
5 years, 7 months ago (2015-04-29 00:28:52 UTC) #1
Dirk Pranke
lgtm
5 years, 7 months ago (2015-04-29 00:33:36 UTC) #3
scottmg
5 years, 7 months ago (2015-04-29 00:40:58 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
8866260996c063f514dfafd0e05743a42709a49c.

Powered by Google App Engine
This is Rietveld 408576698