8000 Possibly fixing vim/python import machinery error by diraol · Pull Request #1028 · python-mode/python-mode · GitHub
[go: up one dir, main page]

Skip to content

Possibly fixing vim/python import machinery error #1028

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

Merged
merged 1 commit into from
May 11, 2019

Conversation

diraol
Copy link
Contributor
@diraol diraol commented May 11, 2019

This is a possible fix for #972.

And it was based on pypa/setuptools#1563 (comment)

@diraol diraol merged commit 08dbb8c into python-mode:develop May 11, 2019
@diraol diraol deleted the fix_pkg_resources branch May 11, 2019 16:46
wookayin added a commit to wookayin/python-mode that referenced this pull request Sep 22, 2023
PathFinder.find_module() has been deprecated since Python 3.4 in favor of
find_spec(), and it's finally removed in Python 3.12. This line will
throw an AttributeError which makes pymode completely unusable with
python 3.12.

It was a hacky workaround introduced in python-mode#1028. Maybe we can completely
remove this workaround because it's 4 years ago and the minimum
supported python version is now 3.6+.
wookayin added a commit to wookayin/python-mode that referenced this pull request Sep 22, 2023
PathFinder.find_module() has been deprecated since Python 3.4 in favor of
find_spec(), and it's finally removed in Python 3.12. This line will
throw an AttributeError which makes pymode completely unusable with
python 3.12.

It was a hacky workaround introduced in python-mode#1028. Maybe we can completely
remove this workaround because it's 4 years ago and the minimum
supported python version is now 3.6+.
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.

1 participant
0