Simplify octal constants in Python code (e.g. 0o0700 -> 0o700).

Simplify octal constants in Python code (e.g. 0o0700 -> 0o700).

Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/1dfe43ca1aec9d1e6f8982450dbc7bda6750d910

[Matt Giuca, 2016-06-17 04:14:26]

mgiuca@chromium.org changed reviewers:
+ vapier@chromium.org

[Matt Giuca, 2016-06-17 04:14:26]

Amendment to https://codereview.chromium.org/2076613002.

[vapier, 2016-06-17 04:16:02]

lgtm

[nodir, 2016-06-17 04:56:35]

On 2016/06/17 04:16:02, vapier wrote:
> lgtm

How about just 0?

[Matt Giuca, 2016-06-17 04:57:42]

On 2016/06/17 04:56:35, nodir wrote:
> On 2016/06/17 04:16:02, vapier wrote:
> > lgtm
> 
> How about just 0?

0 doesn't work in Python 3 (the whole point of vapier's previous CL was to make
it compatible with py3, see https://codereview.chromium.org/2076613002).

[Matt Giuca, 2016-06-17 04:58:29]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-06-17 04:58:35]

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

[commit-bot: I haz the power, 2016-06-17 04:58:36]

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

[commit-bot: I haz the power, 2016-06-17 04:58:38]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[Matt Giuca, 2016-06-17 05:31:00]

mgiuca@chromium.org changed reviewers:
+ nodir@chromium.org

[Matt Giuca, 2016-06-17 05:31:15]

Oops, didn't realise an OWNER was required. PTAL, nodir. thx

[nodir, 2016-06-17 05:58:27]

ah, I just read the diff. lgtm

before that I read only commit message and 0o0 -> 0o looked strange to me

[Matt Giuca, 2016-06-17 06:06:31]

On 2016/06/17 05:58:27, nodir wrote:
> before that I read only commit message and 0o0 -> 0o looked strange to me

Ah, gotcha. I'll make it clearer. Ty.

[Matt Giuca, 2016-06-17 06:07:12]

Description was changed from

==========
Simplify octal constants in Python code (0o0 -> 0o).
==========

to

==========
Simplify octal constants in Python code (e.g. 0o0700 -> 0o700).
==========

[Matt Giuca, 2016-06-17 06:07:22]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-06-17 06:07:34]

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

[commit-bot: I haz the power, 2016-06-17 06:10:48]

Description was changed from

==========
Simplify octal constants in Python code (e.g. 0o0700 -> 0o700).
==========

to

==========
Simplify octal constants in Python code (e.g. 0o0700 -> 0o700).

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/1dfe43ca1aec9d...
==========

[commit-bot: I haz the power, 2016-06-17 06:10:49]

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