V8: Remove xcode-ninja bot

V8: Remove xcode-ninja bot

Reduces maintenance as it is unlikely that we will break
gn's xcode generator.

BUG=645890
Committed: https://chromium.googlesource.com/chromium/tools/build/+/5c1d87a88517d25aef1d7a4349dc827167d72b1c

[Michael Achenbach, 2016-09-20 14:26:07]

Description was changed from

==========
V8: Switch xcode-ninja bot to gn

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

to

==========
V8: Switch xcode-ninja bot to gn

BUG=645091
==========

[Michael Achenbach, 2016-09-20 14:31:21]

machenbach@chromium.org changed reviewers:
+ dpranke@chromium.org, jochen@chromium.org, kjellander@chromium.org

[Michael Achenbach, 2016-09-20 14:31:22]

PTAL. This is one of two V8 bots not supporting MB, therefore this asynchronous
switch to GN on the infra-side.

I'm not entirely sure if this bot still serves a good purpose. We were able to
break the old xcode-ninja generator several times (hence the bot). Not sure if
the same holds for gn's xcode ide option.

[Michael Achenbach, 2016-09-20 14:32:04]

Description was changed from

==========
V8: Switch xcode-ninja bot to gn

BUG=645091
==========

to

==========
V8: Switch xcode-ninja bot to gn

BUG=645890
==========

[Nico, 2016-09-20 14:33:27]

thakis@chromium.org changed reviewers:
+ thakis@chromium.org

[Nico, 2016-09-20 14:33:28]

Are you aware of instances where you broke gn --ide=xcode? If not, maybe delete
the bot for now and add it back in the new form if it turns out it's needed?

[Michael Achenbach, 2016-09-20 14:38:51]

On 2016/09/20 14:33:28, Nico wrote:
> Are you aware of instances where you broke gn --ide=xcode? If not, maybe
delete
> the bot for now and add it back in the new form if it turns out it's needed?

No, nobody complained and I'm not aware of any bot verifying this anywhere.

Deleting the bot is an option if you think it's highly unlikely that gn's option
is breakable (by us), though it doesn't cost much to have it either.

[Dirk Pranke, 2016-09-20 21:26:04]

On 2016/09/20 14:38:51, machenbach (slow) wrote:
> On 2016/09/20 14:33:28, Nico wrote:
> > Are you aware of instances where you broke gn --ide=xcode? If not, maybe
> delete
> > the bot for now and add it back in the new form if it turns out it's needed?
> 
> No, nobody complained and I'm not aware of any bot verifying this anywhere.
> 
> Deleting the bot is an option if you think it's highly unlikely that gn's
option
> is breakable (by us), though it doesn't cost much to have it either.

I would delete the bot. 

If we were worried about breaking this, I'd rather have that
monitored/maintained/tested by something closer to the main GN devs. But for now
I wouldn't worry about it at all.

[Michael Achenbach, 2016-09-21 12:47:47]

Description was changed from

==========
V8: Switch xcode-ninja bot to gn

BUG=645890
==========

to

==========
V8: Remove xcode-ninja bot

Reduces maintenance as it is unlikely that we will break
gn's xcode generator.

BUG=645890
==========

[Michael Achenbach, 2016-09-21 12:52:46]

PTAL - removing the bot. The "M" at the expectation file is a bit odd. Locally,
this file is removed.

[Dirk Pranke, 2016-09-21 16:38:29]

lgtm

[Michael Achenbach, 2016-09-24 12:03:38]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-09-24 12:03:41]

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-24 12:07:20]

Description was changed from

==========
V8: Remove xcode-ninja bot

Reduces maintenance as it is unlikely that we will break
gn's xcode generator.

BUG=645890
==========

to

==========
V8: Remove xcode-ninja bot

Reduces maintenance as it is unlikely that we will break
gn's xcode generator.

BUG=645890

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/5c1d87a88517d25aef1d...
==========

[commit-bot: I haz the power, 2016-09-24 12:07:21]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/build/+/5c1d87a88517d25aef1d...