[gn] Switch off gyp in runhooks by default

[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Committed: https://crrev.com/9a3b720395416cf33f0d1c001ae4eeadad01b0a3
Cr-Original-Commit-Position: refs/heads/master@{#39308}
Cr-Commit-Position: refs/heads/master@{#39311}

[Michael Achenbach, 2016-09-09 10:15:30]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

BUG=
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev.

BUG=chromium:474921
==========

[Michael Achenbach, 2016-09-09 10:16:03]

The CQ bit was checked by machenbach@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 10:16:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Michael Achenbach, 2016-09-09 10:16:41]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev.

BUG=chromium:474921
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
==========

[commit-bot: I haz the power, 2016-09-09 10:38:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-09 10:38:15]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2016-09-09 13:13:47]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-09-09 13:13:48]

PTAL

A PSA is out:
https://groups.google.com/forum/#!topic/v8-dev/f5PMQT2BDe8

Our deprecated gyp documentation doesn't rely on gyp in
runhooks anyways - gyp_v8 is called explicitly, so it will
continue to work after this. Same with Makefiles.

Infrastructure preparation landed: https://codereview.chromium.org/2320853005/

[Michael Achenbach, 2016-09-09 13:15:49]

https://codereview.chromium.org/2327033002/diff/1/gypfiles/gyp_v8
File gypfiles/gyp_v8 (right):

https://codereview.chromium.org/2327033002/diff/1/gypfiles/gyp_v8#newcode130
gypfiles/gyp_v8:130: print '`python build/gyp_v8` explicitly by hand '
hmm - typo - will fix

[Michael Achenbach, 2016-09-09 13:18:35]

Typo fix in patch 2

[Michael Achenbach, 2016-09-09 13:18:51]

The CQ bit was checked by machenbach@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 13:19:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jochen (gone - plz use gerrit), 2016-09-09 13:26:42]

lgtm

[Michael Achenbach, 2016-09-09 13:31:06]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true
==========

[Michael Achenbach, 2016-09-09 13:31:10]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2016-09-09 13:31:13]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-09-09 13:31:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 13:33:01]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true
==========

[commit-bot: I haz the power, 2016-09-09 13:33:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-09 13:33:35]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

[commit-bot: I haz the power, 2016-09-09 13:33:35]

Patchset 2 (id:??) landed as
https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}

[Michael Achenbach, 2016-09-09 14:09:09]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2324173002/ by machenbach@chromium.org.

The reason for reverting is: Broke stuff - need to fix the argument.

[Michael Achenbach, 2016-09-09 14:11:25]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

[Michael Achenbach, 2016-09-09 14:19:30]

Patch 3 is another fix. Porting:
https://cs.chromium.org/chromium/src/build/gyp_chromium.py?q=%22--running-as-...

[Michael Achenbach, 2016-09-09 14:20:10]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2016-09-09 14:20:11]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2327033002/#ps40001
(title: "Fix")

[commit-bot: I haz the power, 2016-09-09 14:20:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 14:22:23]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

[commit-bot: I haz the power, 2016-09-09 14:22:24]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-09 14:23:24]

Description was changed from

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Cr-Commit-Position: refs/heads/master@{#39308}
==========

to

==========
[gn] Switch off gyp in runhooks by default

This will be synced with an announcement on v8-dev. This
doesn't affect directly calling gyp in our old Makefiles.

BUG=chromium:474921
NOTRY=true

Committed: https://crrev.com/94f813e8dcd4ed5a67cede8af0f949009beaa501
Committed: https://crrev.com/9a3b720395416cf33f0d1c001ae4eeadad01b0a3
Cr-Original-Commit-Position: refs/heads/master@{#39308}
Cr-Commit-Position: refs/heads/master@{#39311}
==========

[commit-bot: I haz the power, 2016-09-09 14:23:25]

Patchset 3 (id:??) landed as
https://crrev.com/9a3b720395416cf33f0d1c001ae4eeadad01b0a3
Cr-Commit-Position: refs/heads/master@{#39311}