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

Stop bundling setuptools for Python 3.12+ #301

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

edmorley
Copy link
Contributor

For parity with the ensurepip and venv modules, which no longer install setuptools as of Python 3.12:

Fixes #300.

@indygreg
Copy link
Owner

This PR seems technically sound. But I'd like a few questions answered at #300 (comment) before merging this.

For parity with the `ensurepip` and `venv` modules, which no
longer install setuptools as of Python 3.12:
- python/cpython#95299
- python/cpython@ece20db

Fixes indygreg#300.
@edmorley
Copy link
Contributor Author

edmorley commented Sep 7, 2024

I've rebased on main after the recent PR merges.

@edmorley
Copy link
Contributor Author

edmorley commented Sep 9, 2024

Are you happy for this to be merged? :-)

@zanieb
Copy link
Collaborator

zanieb commented Sep 9, 2024

Thanks for rebasing! I might ship this after the other changes?

@zanieb
Copy link
Collaborator

zanieb commented Sep 9, 2024

Eh, after thinking a bit more I don't really want to ship setuptools with Python 3.13 if we're going to remove it anyway, so it seems better to do it all at once.

@zanieb zanieb merged commit 3d279ba into indygreg:main Sep 9, 2024
291 checks passed
@edmorley edmorley deleted the python-312-no-setuptools branch September 9, 2024 12:11
@edmorley
Copy link
Contributor Author

edmorley commented Sep 9, 2024

Many thanks :-)

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.

Stop bundling setuptools for Python 3.12+
3 participants