handle configparser rename in python3

handle configparser rename in python3

The ConfigParser module was renamed to configparser, so update the
two modules using it to try both (and default to the new name).

Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/9f343717f1f6d254b5b6992f2f82df28afc51033

[vapier, 2016-06-16 18:52:12]

vapier@chromium.org changed reviewers:
+ nodir@chromium.org, tandrii@chromium.org, vadimsh@chromium.org

[vapier, 2016-06-16 18:52:13]

spamming a few more people this time as an RFC.  if people are OK with this
style, there's a few more modules to change.  or maybe we want to go a different
route to handle py2/py3 compat in a single codebase.

[nodir, 2016-06-16 18:55:58]

lgtm

https://codereview.chromium.org/2076653002/diff/1/checkout.py
File checkout.py (right):

https://codereview.chromium.org/2076653002/diff/1/checkout.py#newcode19
checkout.py:19: # The configparser module was renamed in Python 3.
this comment should be inside except clause

[vapier, 2016-06-16 19:13:47]

https://codereview.chromium.org/2076653002/diff/1/checkout.py
File checkout.py (right):

https://codereview.chromium.org/2076653002/diff/1/checkout.py#newcode19
checkout.py:19: # The configparser module was renamed in Python 3.
"configparser" is the py3 name which is in the try block.

that said, i was copying existing style used in the code base.  check out the
upload.py diff and the "# The md5 module was deprecated in Python 2.5." comment.

[nodir, 2016-06-16 21:54:03]

lgtm

[tandrii(chromium), 2016-06-17 16:27:38]

lgtm

[vapier, 2016-06-22 14:09:52]

The CQ bit was checked by vapier@chromium.org

[commit-bot: I haz the power, 2016-06-22 14:10:01]

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

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

Description was changed from

==========
handle configparser rename in python3

The ConfigParser module was renamed to configparser, so update the
two modules using it to try both (and default to the new name).
==========

to

==========
handle configparser rename in python3

The ConfigParser module was renamed to configparser, so update the
two modules using it to try both (and default to the new name).

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/9f343717f1f6d2...
==========

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

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