Updata ninja to v1.7.2.

Updata ninja to v1.7.2.

New binaries for Mac, Linux32, Linux64, Windows.  Also update shell script.
See the bug for how these were built.
The main "new" thing is a regression fix where 1.7.1 broke %e %r in
NINJA_STATUS.

BUG=663749
Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/28a1ff42d9477f548a37afc89813012b673b406c

[Nico, 2016-11-09 17:16:59]

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

[commit-bot: I haz the power, 2016-11-09 17:17:04]

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

[Nico, 2016-11-09 17:17:09]

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

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

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

[commit-bot: I haz the power, 2016-11-09 17:20:15]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2016-11-09 17:24:22]

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

[Dirk Pranke, 2016-11-09 17:24:22]

lgtm.

Though, I don't think we technically need the linux32 binary any more ...

[scottmg, 2016-11-09 17:24:33]

lgtm

[Nico, 2016-11-09 17:26:05]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-11-09 17:26:15]

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

[Nico, 2016-11-09 17:26:36]

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

[Nico, 2016-11-09 17:26:36]

> Though, I don't think we technically need the linux32 binary any more ...

thestig, you wanted the 32-bit binary back then. Can we get rid of it?

[commit-bot: I haz the power, 2016-11-09 17:29:13]

Description was changed from

==========
Updata ninja to v1.7.2.

New binaries for Mac, Linux32, Linux64, Windows.  Also update shell script.
See the bug for how these were built.
The main "new" thing is a regression fix where 1.7.1 broke %e %r in
NINJA_STATUS.

BUG=663749
==========

to

==========
Updata ninja to v1.7.2.

New binaries for Mac, Linux32, Linux64, Windows.  Also update shell script.
See the bug for how these were built.
The main "new" thing is a regression fix where 1.7.1 broke %e %r in
NINJA_STATUS.

BUG=663749

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/28a1ff42d9477f...
==========

[commit-bot: I haz the power, 2016-11-09 17:29:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/28a1ff42d9477f...

[Lei Zhang, 2016-11-09 19:09:02]

On 2016/11/09 17:26:36, Nico wrote:
> > Though, I don't think we technically need the linux32 binary any more ...
> 
> thestig, you wanted the 32-bit binary back then. Can we get rid of it?

Probably not, maybe delete it and see what happens?

I have buildtools/linux32/gn, but I'm not sure where it came from. Maybe Dirk
knows if it is suppose to exist? If not, that's one more argument against
ninja-linux32.

[Dirk Pranke, 2016-11-09 19:13:10]

On 2016/11/09 19:09:02, Lei Zhang (slow) wrote:
> On 2016/11/09 17:26:36, Nico wrote:
> > > Though, I don't think we technically need the linux32 binary any more ...
> > 
> > thestig, you wanted the 32-bit binary back then. Can we get rid of it?
> 
> Probably not, maybe delete it and see what happens?
> 
> I have buildtools/linux32/gn, but I'm not sure where it came from. Maybe Dirk
> knows if it is suppose to exist? If not, that's one more argument against
> ninja-linux32.

We used to have 32-bit linux builds, and even after Chromium killed ours, NaCl
had
one for a while longer, but I think they've gotten rid of theirs now, too. I can
double-check.

[Dirk Pranke, 2016-11-09 19:22:42]

Looks like NaCl still has them, so we need to keep them for now. I'll follow up
w/ Bradley to see if we can shut them down.