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

setup.py: Import distutils directly instead of through setuptools #81

Merged

Conversation

mweinelt
Copy link
Contributor

@mweinelt mweinelt commented Jan 6, 2023

In setuptools 65.6.0 the vendored distutils package changed to relative
imports. Importing Command from setuptools._distutils previously
worked but now fails because it imports an unpatched version of
distutils, which breaks the build.

Importing Command directly from distutils or setuputils will
receive a monkey patched version that works.

Reported upstream at pypa/setuptools#3743 and the problem was investigated by Anderson Bravalheri.

Closes: #77

@mweinelt mweinelt mentioned this pull request Jan 6, 2023
@abravalheri
Copy link

@mweinelt you can also do from setuptools import Command (probably that is the best).

@mweinelt mweinelt force-pushed the setuptools-65.6.0-compat branch 2 times, most recently from 9c460ca to 9bd950b Compare January 6, 2023 13:24
In setuptools 65.6.0 the vendored distutils package changed to relative
imports. Importing `Command` from `setuptools._distutils` previously
worked but now fails because it imports an unpatched version of
distutils, which breaks the build.

Importing `Command` directly from `distutils` or `setuputils` will
receive a monkey patched version that works.

Reported upstream at pypa/setuptools#3743 and
the problem was investigated by Anderson Bravalheri.

Closes: alan-turing-institute#77
@GjjvdBurg GjjvdBurg merged commit 51569e7 into alan-turing-institute:master Jan 9, 2023
@GjjvdBurg
Copy link
Collaborator

Thanks for opening this PR @mweinelt, much appreciated!

@mweinelt mweinelt deleted the setuptools-65.6.0-compat branch January 9, 2023 22:26
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.

0.7.4: pep517 build fails
3 participants