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 106243002: Treat cygwin the same as Windows when selecting the GN binary. (Closed)

Created:
7 years ago by brettw
Modified:
7 years ago
Reviewers:
M-A Ruel
CC:
chromium-reviews
Visibility:
Public.

Description

Treat cygwin the same as Windows when selecting the GN binary. R=maruel@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=239028

Patch Set 1 #

Total comments: 1

Patch Set 2 : Review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M build/gyp_chromium View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
7 years ago (2013-12-05 06:14:15 UTC) #1
M-A Ruel
lgtm https://codereview.chromium.org/106243002/diff/1/build/gyp_chromium File build/gyp_chromium (right): https://codereview.chromium.org/106243002/diff/1/build/gyp_chromium#newcode149 build/gyp_chromium:149: if sys.platform in [ 'win32', 'cygwin' ]: if ...
7 years ago (2013-12-05 14:38:31 UTC) #2
brettw
7 years ago (2013-12-05 20:35:42 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 manually as r239028 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698