Skip to content
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

Spinx 1.7+ ImportError workaround #69

Closed

Conversation

z00sts
Copy link

@z00sts z00sts commented Oct 7, 2019

Allows you at least see application help instead of Traceback print.

Fixes #66

Copy link

@paddy-hack paddy-hack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misleading comment. It's not the Python version that causes the ImportError. It's upgrading Sphinx from 1.6.5 to 1.7.0 that does. Suggest the comment gets fixed.
Applying the fix worked fine for me with Python 3.4.

@z00sts
Copy link
Author

z00sts commented Nov 19, 2019

@paddy-hack thanks for clarification. So you closing the PR?

@paddy-hack
Copy link

No, I'm not closing this PR. I'm just suggesting you fix up the comment in this/your PR so people looking at the code later don't get misled. That's all.

See #70 (comment) for some background info on this.

@z00sts z00sts changed the title python3.6: ImportError workaround Spinx 1.7+ ImportError workaround Nov 21, 2019
@TimKam
Copy link
Member

TimKam commented Dec 3, 2019

Can yo control-merge master into your branch/rebase, @z00sts? Also, if I understand it correctly, updating sphinxcontrib-versioning without updating a very old Sphinx install (< 1.7) would cause problems given the changes in this PR, but IMHO we can live with this.

@z00sts
Copy link
Author

z00sts commented Dec 5, 2019

I updated branch state with upstream and created new PR with correct branch-name and commit message #78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ImportError: cannot import name 'build_main'
3 participants