Make gn_run_binary.py print out the exit code on error.

Make gn_run_binary.py print out the exit code on error.

Review-Url: https://codereview.chromium.org/2820753004
Cr-Commit-Position: refs/heads/master@{#464890}
Committed: https://chromium.googlesource.com/chromium/src/+/b43ecb203b3ee459e3564e7957ab2cfbd2b040f6

[Lei Zhang, 2017-04-15 09:24:42]

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

[commit-bot: I haz the power, 2017-04-15 09:24:51]

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

[vabr (Chromium), 2017-04-15 10:02:43]

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

[vabr (Chromium), 2017-04-15 10:02:44]

LGTM, especially when I'm sheriff I would welcome this!
Cheers,
Vaclav

[commit-bot: I haz the power, 2017-04-15 10:26:07]

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

[commit-bot: I haz the power, 2017-04-15 10:26:07]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-04-15 21:31:13]

thestig@chromium.org changed reviewers:
+ dpranke@chromium.org

[Lei Zhang, 2017-04-15 21:31:13]

+dpranke for OWNERS

[Lei Zhang, 2017-04-15 21:33:23]

For context, we saw a mysterious bro.exe failure when compressing
about_credits.html and there was nothing in the log to help figure out why it
happened. This is one step towards that.

Next, we need to make bro.exe return something other than just 0 or 1. I have a
patch for that.

[Dirk Pranke, 2017-04-16 19:08:08]

lgtm

[Lei Zhang, 2017-04-16 20:36:13]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-04-16 20:36:17]

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, 2017-04-16 21:39:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492374973112280, "parent_rev":
"12f029dbc038cc206214e43fd83aa362bb9e649f", "commit_rev":
"b43ecb203b3ee459e3564e7957ab2cfbd2b040f6"}

[commit-bot: I haz the power, 2017-04-16 21:40:46]

Description was changed from

==========
Make gn_run_binary.py print out the exit code on error.
==========

to

==========
Make gn_run_binary.py print out the exit code on error.

Review-Url: https://codereview.chromium.org/2820753004
Cr-Commit-Position: refs/heads/master@{#464890}
Committed:
https://chromium.googlesource.com/chromium/src/+/b43ecb203b3ee459e3564e7957ab...
==========

[commit-bot: I haz the power, 2017-04-16 21:40:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b43ecb203b3ee459e3564e7957ab...