sync-and-gyp: make smarter

sync-and-gyp: make smarter

This fixes two issues: If gyp is run with the CXX and CC environment
variables set, it will produce different ninja build files.
sync-and-gyp now tracks the values of those variables; if they change,
re-run gyp.

Secondly, we introduced find.py, which is used inside gyp files.  If
the contents of certain directories changes, gyp should be re-run.  So
now we track those directories too.

Committed: https://skia.googlesource.com/skia/+/43315bbe4b39073bb8a585cc806eb50a72224e5a

[hal.canary, 2015-05-26 14:06:01]

halcanary@google.com changed reviewers:
+ scroggo@google.com

[hal.canary, 2015-05-26 14:06:02]

PTAL

This fixes two issues:  If gyp is run with the CXX and CC environment variables
set, it will produce different ninja build files.  sync-and-gyp now tracks the
values of those variables; if they change, re-run gyp.

Secondly, we introduced find.py, which is used inside gyp files.  If the
contents of certain directories changes, gyp should be re-run.  So now we track
those directories too.

[scroggo, 2015-05-26 14:12:55]

On 2015/05/26 14:06:02, Hal Canary wrote:
> PTAL
> 
> This fixes two issues:  If gyp is run with the CXX and CC environment
variables
> set, it will produce different ninja build files.  sync-and-gyp now tracks the
> values of those variables; if they change, re-run gyp.
> 
> Secondly, we introduced find.py, which is used inside gyp files.  If the
> contents of certain directories changes, gyp should be re-run.  So now we
track
> those directories too.

Did you mean to add find.py? (It looks like you just use the bash "find"?)

Shouldn't these comments go in the change description?

[hal.canary, 2015-05-26 14:49:35]

On 2015/05/26 14:12:55, scroggo wrote:
> Did you mean to add find.py? (It looks like you just use the bash "find"?)

Nope.  This script assumes a mostly-Posix-compatible system.  gyp_skia does not.

> Shouldn't these comments go in the change description?

Right

[scroggo, 2015-05-26 14:54:07]

On 2015/05/26 14:49:35, Hal Canary wrote:
> On 2015/05/26 14:12:55, scroggo wrote:
> > Did you mean to add find.py? (It looks like you just use the bash "find"?)
> 
> Nope.  This script assumes a mostly-Posix-compatible system.  gyp_skia does
not.

Ah, when you say "we introduced find.py", you meant that it was introduced prior
to this CL.

> 
> > Shouldn't these comments go in the change description?
> 
> Right

lgtm

[hal.canary, 2015-05-26 14:54:42]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-05-26 14:55:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1153083002/1

[commit-bot: I haz the power, 2015-05-26 15:01:23]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/43315bbe4b39073bb8a585cc806eb50a72224e5a