Merge lp:~cjwatson/lazr.restful/py3-zope.publisher-6.0.0 into lp:lazr.restful

Proposed by Colin Watson
Status: Merged
Merged at revision: 280
Proposed branch: lp:~cjwatson/lazr.restful/py3-zope.publisher-6.0.0
Merge into: lp:lazr.restful
Diff against target: 26 lines (+4/-1)
2 files modified
NEWS.rst (+2/-0)
setup.py (+2/-1)
To merge this branch: bzr merge lp:~cjwatson/lazr.restful/py3-zope.publisher-6.0.0
Reviewer Review Type Date Requested Status
Thiago F. Pappacena (community) Approve
Review via email: mp+396612@code.launchpad.net

Commit message

Require zope.publisher >= 6.0.0 on Python 3.

Description of the change

It works on Python 2 as well, but we can get away with older versions there. On Python 3 we need the changes in 6.0.0 to avoid https://bugs.python.org/issue27777.

See https://github.com/zopefoundation/zope.publisher/pull/55 and the various linked issues and PRs for the gory details.

To post a comment you must log in.
Revision history for this message
Thiago F. Pappacena (pappacena) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'NEWS.rst'
2--- NEWS.rst 2021-01-20 11:23:16 +0000
3+++ NEWS.rst 2021-01-20 22:37:00 +0000
4@@ -18,6 +18,8 @@
5 ``lazr.restful.testing.webservice.pprint_collection`` recurse into lists in
6 order to print text string representations in the Python 3 style.
7
8+Require zope.publisher >= 6.0.0 on Python 3.
9+
10 0.23.0 (2020-09-28)
11 ===================
12
13
14=== modified file 'setup.py'
15--- setup.py 2021-01-04 11:24:14 +0000
16+++ setup.py 2021-01-20 22:37:00 +0000
17@@ -73,7 +73,8 @@
18 'zope.pagetemplate',
19 'zope.processlifetime',
20 'zope.proxy',
21- 'zope.publisher',
22+ 'zope.publisher; python_version < "3"',
23+ 'zope.publisher>=6.0.0; python_version >= "3"',
24 'zope.schema',
25 'zope.security',
26 'zope.traversing',

Subscribers

People subscribed via source and target branches