-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Make make.py build docs under python 3 #1361
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
Conversation
+1 Good stuff (not tested, but looks fine to me, and is backwards compatible to python2.6) |
It also needs raise Error, errors updated to raise Error(errors) on line 105. Other than that, this looks good. |
Looks like "as" was only introduced in Python 2.6: because this will break pre 2.6 is it simpler to follow the note in make.py and replace the entire compatibility block with: which gets make.py working for me on Python 3.2. But there were a couple of other files in the document build system I had to patch too that I include below. And also 2to3 misses removing the exceptions module from the import in lib/matplotlib/sphinxext/plot_directive.py (it's after a line continuation character). I can't verify the documents output though as my document build breaks with: Don't think that is related and I'm still investigating, I don't have all the Python modules installed to support all the various back-ends.
|
Note: We don't support pre-2.6 anymore. |
On it.
As Ben said, python 2.5 and before are not supported. Cheers for the patches, @cjmayo. It looks like this is ready to merge once I push @mdboom's suggestion. I think I will put the other doc fixes into another pull request so they don't hold this one up. Thanks again. |
@cjmayo Those patches break doc building with python 2.7 (and possibly 2.6, too) the |
On the but I have raised a new one as it doesn't work for me in 2.7,3.2 or 3.3: |
@cjmayo Weird. The reproducible example you gave in that bug report (I haven't tried it) couldn't be simpler. |
Looks like the |
Ended up using |
These commits have been moved to #1383 to keep everything in one place. |
Fixes #1356.
@cjmayo Could you confirm this works as intended?