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

Migrate setup.py from distutils to setuptools #269

Closed
codesue opened this issue May 6, 2023 · 2 comments · Fixed by #290
Closed

Migrate setup.py from distutils to setuptools #269

codesue opened this issue May 6, 2023 · 2 comments · Fixed by #290
Assignees
Labels
good first issue Good for newcomers maintenance Chores and process improvements

Comments

@codesue
Copy link
Collaborator

codesue commented May 6, 2023

distutils will be removed in Python 3.12 and there's an existing TODO to remove it from MCT's setup.py.

Refer to PEP 632 for migration advice.

@codesue codesue added good first issue Good for newcomers contributions welcome This issue is ready to be worked on labels May 6, 2023
@codesue codesue added the maintenance Chores and process improvements label May 14, 2023
@BhavyaBh289
Copy link

is this still active ?

@codesue
Copy link
Collaborator Author

codesue commented Jun 11, 2023

Hi @BhavyaBh289, after looking into this some more, it seems there isn't a good setuptools alternative yet for distutils.command.build, and we can continue to use distutils through setuptools' bundled copy after it's removed from the standard library. See pypa/setuptools#2928. This is still up for grabs, but the only work required now is to delete the TODO comment.

@codesue codesue self-assigned this Jul 15, 2023
@codesue codesue removed the contributions welcome This issue is ready to be worked on label Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers maintenance Chores and process improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants