jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[bugfix] Check for outdated setuptools first

setuptools 18.2 shipped with Python 3.5.0 fails for
parsing dependencies. check for setuptools first
within pwb.py.

Bug: T286980
Change-Id: Ifdad279d2794ea7bf95411a969b6958743aabcfd
---
M pwb.py
1 file changed, 19 insertions(+), 5 deletions(-)

diff --git a/pwb.py b/pwb.py
index 8d9ed39..1ab0d74 100755
--- a/pwb.py
+++ b/pwb.py
@@ -175,14 +175,27 @@
:raise RuntimeError: wrong Python version found in setup.py
"""
import pkg_resources
- if script:
- from setup import script_deps
- dependencies = script_deps.get(Path(script).name, [])
- else:
- from setup import dependencies
+
+ from setup import dependencies, script_deps

missing_requirements = []
version_conflicts = []
+
+ try:
+ requirement = next(pkg_resources.parse_requirements(dependencies))
+ except ValueError as e:
+ # T286980: setuptools is too old and requirement parsing fails
+ import setuptools
+ setupversion = tuple(int(num)
+ for num in setuptools.__version__.split('.'))
+ if setupversion < (20, 8, 1):
+ # print the minimal requirement
+ _print_requirements(['setuptools==20.8.1'], None, 'outdated')
+ return
+ raise e
+
+ if script:
+ dependencies = script_deps.get(Path(script).name, [])
for requirement in pkg_resources.parse_requirements(dependencies):
if requirement.marker is None \
or pkg_resources.evaluate_marker(str(requirement.marker)):
@@ -197,6 +210,7 @@

del pkg_resources
del dependencies
+ del script_deps

_print_requirements(missing_requirements, script, 'missing')
_print_requirements(version_conflicts, script, 'outdated')

To view, visit change 705619. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ifdad279d2794ea7bf95411a969b6958743aabcfd
Gerrit-Change-Number: 705619
Gerrit-PatchSet: 3
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged