gyp_skia: make default GYP_GENERATORS for each platform explicit

gyp_skia: make default GYP_GENERATORS for each platform explicit

BUG=skia:2317
Committed: http://code.google.com/p/skia/source/detail?r=13883

[epoger, 2014-03-20 19:15:26]

PTAL

https://codereview.chromium.org/199873003/diff/1/gyp_skia
File gyp_skia (right):

https://codereview.chromium.org/199873003/diff/1/gyp_skia#newcode83
gyp_skia:83: if not os.getenv(ENVVAR_GYP_GENERATORS):
gyp already picks a default generator per-platform if it isn't specified; this
just makes it explicit, and keeps it under our control.

My hope is to change this to "ninja" in most cases, but for now, I just wanted
to make it explicit but unchanged.

[bungeman-skia, 2014-03-20 19:20:49]

https://codereview.chromium.org/199873003/diff/1/gyp_skia
File gyp_skia (right):

https://codereview.chromium.org/199873003/diff/1/gyp_skia#newcode83
gyp_skia:83: if not os.getenv(ENVVAR_GYP_GENERATORS):
On 2014/03/20 19:15:26, epoger wrote:
> gyp already picks a default generator per-platform if it isn't specified; this
> just makes it explicit, and keeps it under our control.
> 
> My hope is to change this to "ninja" in most cases, but for now, I just wanted
> to make it explicit but unchanged.

This is what gyp does
https://code.google.com/p/gyp/source/browse/trunk/pylib/gyp/__init__.py#368 . It
adds cygwin, which may or may not be of interest to those out there who may be
using it.

[epoger, 2014-03-20 19:23:07]

Added cygwin, PTAL

[bungeman-skia, 2014-03-20 19:24:14]

lgtm

[epoger, 2014-03-20 19:24:25]

The CQ bit was checked by epoger@google.com

[commit-bot: I haz the power, 2014-03-20 19:24:27]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/epoger@google.com/199873003/2

[commit-bot: I haz the power, 2014-03-20 19:59:11]

Change committed as 13883