vpython: Version parser from PEP440.

vpython: Version parser from PEP440.

Use canonical version regexp from PEP440 instead of crude
"major.minor.patch" version parsing. We still parse into the previous
form, but we tolerate the full set of canonical strings.

This is in response to observed "2.7.13+" in the wild.

BUG=None
TEST=unit

Review-Url: https://codereview.chromium.org/2874683002
Committed: https://github.com/luci/luci-go/commit/0eb69ab195d7c4a88cd41d26b53f9a868c15713f

[dnj (Google), 2017-05-10 04:10:31]

dnj@google.com changed reviewers:
+ dnj@google.com, iannucci@chromium.org

[dnj (Google), 2017-05-10 04:10:32]

PTAL

[dnj (Google), 2017-05-10 04:10:37]

The CQ bit was checked by dnj@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-10 04:10:48]

Dry run: 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-05-10 04:17:38]

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

[commit-bot: I haz the power, 2017-05-10 04:17:39]

Dry run: This issue passed the CQ dry run.

[iannucci, 2017-05-10 19:43:52]

lgtm

https://codereview.chromium.org/2874683002/diff/1/vpython/python/version_test.go
File vpython/python/version_test.go (right):

https://codereview.chromium.org/2874683002/diff/1/vpython/python/version_test...
vpython/python/version_test.go:39: {"1!1.0", Version{1, 0, 0}},
2!1.0

[dnj (Google), 2017-05-10 21:42:09]

The CQ bit was checked by dnj@google.com

[dnj (Google), 2017-05-10 21:42:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2874683002/#ps20001
(title: "stronger test case")

[commit-bot: I haz the power, 2017-05-10 21:42:19]

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-05-10 21:49:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494452529334970,
"parent_rev": "b2274e876e2f891c8cafee769e0ae59e986d4c3c", "commit_rev":
"0eb69ab195d7c4a88cd41d26b53f9a868c15713f"}

[commit-bot: I haz the power, 2017-05-10 21:49:13]

Description was changed from

==========
vpython: Version parser from PEP440.

Use canonical version regexp from PEP440 instead of crude
"major.minor.patch" version parsing. We still parse into the previous
form, but we tolerate the full set of canonical strings.

This is in response to observed "2.7.13+" in the wild.

BUG=None
TEST=unit
==========

to

==========
vpython: Version parser from PEP440.

Use canonical version regexp from PEP440 instead of crude
"major.minor.patch" version parsing. We still parse into the previous
form, but we tolerate the full set of canonical strings.

This is in response to observed "2.7.13+" in the wild.

BUG=None
TEST=unit

Review-Url: https://codereview.chromium.org/2874683002
Committed:
https://github.com/luci/luci-go/commit/0eb69ab195d7c4a88cd41d26b53f9a868c15713f
==========

[commit-bot: I haz the power, 2017-05-10 21:49:14]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-go/commit/0eb69ab195d7c4a88cd41d26b53f9a868c15713f