jachinlin / geektime_dl

把极客时间装进 Kindle
https://jachinlin.github.io/geektime_dl
1.37k stars 356 forks source link

报错了 #9

Closed wajika closed 5 years ago

wajika commented 5 years ago

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] 系统版本:CentOS Linux release 7.5.1804 (Core) python版本3.7.1 Describe the solution you'd like A clear and concise description of what you want to happen. 直接运行geektime login或是加上--account和--password参数,都会提示maximum recursion depth exceeded Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here. /root/venv3/lib/python3.7/site-packages/geektime_dl/utils/m3u8downloader.py:5: MonkeyPatchWarning: Monkey-patching ssl after ssl has already been imported may lead to errors, including RecursionError on Python 3.6. It may also silently lead to incorrect behaviour on Python 3.7. Please monkey-patch earlier. See https://github.com/gevent/gevent/issues/1016. Modules that had direct imports (NOT patched): ['urllib3.util (/root/venv3/lib/python3.7/site-packages/urllib3/util/init.py)', 'urllib3.util.ssl (/root/venv3/lib/python3.7/site-packages/urllib3/util/ssl_.py)']. monkey.patch_all() maximum recursion depth exceeded

jachinlin commented 5 years ago

Monkey-patching ssl after ssl has already been imported may lead to errors, including RecursionError on Python 3.6. It may also silently lead to incorrect behaviour on Python 3.7. Please monkey-patch earlier. See gevent/gevent#1016.

py3.6+ 还有问题 试下py3.5吧

jachinlin commented 5 years ago

fixed