-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CRITICAL] Sphinx doesn't work with rc releases of docutils due to version parsing #3846
Comments
The issue also happens on Python2.7 |
Updating to Sphinx 1.6.2 fixes the issue to the extent that Summary: |
The 1.6.2 was just released last Sunday. What is your Sphinx version ? |
@jfbu |
ah, so something beyond core sphinx is at work in your project. Would some monkey-patch in conf.py be acceptable ? (notwithstanding date of 1.6.3 release) |
This is not a part of v1.6.2. Please check your version again. This is line 27 of |
with this at top of
as a temporary fix. But best would be already to update to Sphinx 1.6.2. ;-) (edited:) 1.6.2's compatibility with Docutils pre-release version string is only partial, as deprecated module I tested the above |
@tk0miya @jfbu I didn't check the |
@userzimmermann yes the issue is there at 1.6.2 but no part of Sphinx imports anymore the module. Surely, some extension possible does. Whih ones are used in your project? |
@jfbu Only |
coincidentally this is precisely the extension I now tested with ;-) that's funny! |
see comment above which I was just editing ;-) |
@jfbu And I just also saw that we have to increase the |
Yes, It seems the Sphinx on CircleCI is not 1.6 series. As a quick look, it might be 1.5.x series. So your case is simply a bug of old Sphinx. So I recommend you to update it to 1.6 series. (or pin the docutils to 0.13.1.) I believe it resolves your case. |
@tk0miya @jfbu Thanks again! And according to my #3846 (comment) before... Sorry for the wasted time ;) |
@jfbu @tk0miya Maybe the time was actually not wasted ;) I updated
The only plugin used is still |
@userzimmermann Thanks for reporting, indeed (I have re-worded my earlier comment so as not to suggest upgrading to 1.6.2 fixes completely the issue. Only 1.6.3 will) |
@jfbu I did the workaround in
Now everything works fine :) Many thanks again! |
I quickly created a PR for |
What is current status of this issue? Is this closable? |
This is a duplicate which was kept open to avoid new ones popping up until Docutils 0.14 is released and the problem disappears on older Sphinx'es ;-) |
Okay. I'll keep this opened for a while. |
I'm closing this now. It's time to ship next stable release :-) |
As you can see here: https://circleci.com/gh/coala/coala/12852?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
cc @sils @jayvdb @thisch
The text was updated successfully, but these errors were encountered: