Skip to content

Commit

Permalink
fix: need gevent >= 1.2, not > 1.1
Browse files Browse the repository at this point in the history
Python 2.6 support was dropped in 1.2, so 2e8dcd3 simplified the dependency graph to require `gevent>1.1`. However, bugfix releases to the 1.1 series (1.1.1, etc) will satisfy this requirement, so we should actually be requiring `gevent >= 1.2`

Additionally, these requirements should be specified in setup.py as
well.
  • Loading branch information
jeffwidman committed Nov 3, 2017
1 parent cf7bdc2 commit 18f3531
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion requirements_gevent.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
gevent>1.1
gevent>=1.2
12 changes: 6 additions & 6 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,16 +28,16 @@

if not (PYTHON3 or PYPY):
tests_require += [
'gevent',
'eventlet',
'gevent>=1.2',
'eventlet>=0.17.1',
]

on_rtd = os.environ.get('READTHEDOCS', None) == 'True'
if on_rtd:
install_requires.extend([
'gevent',
'eventlet',
])
install_requires += [
'gevent>=1.2',
'eventlet>=0.17.1',
]

setup(
name='kazoo',
Expand Down

0 comments on commit 18f3531

Please sign in to comment.