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

Issue 6538104: Causing gyp_chromium to fall back on win32 hermetic python if it is... (Closed)

Created:
9 years, 10 months ago by bradn
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Base URL:
svn://chrome-svn.corp.google.com/chrome/trunk/src/
Visibility:
Public.

Description

Causing gyp_chromium to fall back on win32 hermetic python if it is ever run with cygwin python. BUG=http://code.google.com/p/gyp/issues/detail?id=178 TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75695

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Patch Set 3 : '' #

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

Messages

Total messages: 3 (0 generated)
bradn
9 years, 10 months ago (2011-02-22 23:38:26 UTC) #1
Peter Kasting
Rubber-stamp LGTM http://codereview.chromium.org/6538104/diff/1/build/gyp_chromium File build/gyp_chromium (right): http://codereview.chromium.org/6538104/diff/1/build/gyp_chromium#newcode82 build/gyp_chromium:82: # Fall back on hermetic python if ...
9 years, 10 months ago (2011-02-23 00:04:49 UTC) #2
bradn
9 years, 10 months ago (2011-02-23 01:43:13 UTC) #3
Added and will commit shortly.
-BradN


On Tue, Feb 22, 2011 at 4:04 PM, <pkasting@chromium.org> wrote:

> Rubber-stamp LGTM
>
>
> http://codereview.chromium.org/6538104/diff/1/build/gyp_chromium
> File build/gyp_chromium (right):
>
> http://codereview.chromium.org/6538104/diff/1/build/gyp_chromium#newcode82
> build/gyp_chromium:82: # Fall back on hermetic python if we happen to
> get run under cygwin.
> Nit: Do you want to TODO this with the bug about making gyp work with
> cygwin python?
>
>
> http://codereview.chromium.org/6538104/
>

Powered by Google App Engine
This is Rietveld 408576698