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

gh-100238: Vendor distutils for peg_generator #104744

Closed
wants to merge 2 commits into from

Conversation

lysnikolaou
Copy link
Member

@lysnikolaou lysnikolaou commented May 22, 2023

This puts all of distutils (except for the tests) under Tools and uses that both in the c-analyzer and the peg-generator. Stripping all the unnecessary parts of distutils should be done after b1 is released.

@lysnikolaou lysnikolaou changed the title gh-92584: Vendor distutils for peg_generator gh-100238: Vendor distutils for peg_generator May 22, 2023
@ned-deily
Copy link
Member

Is there a reason we can't use setuptools via pip install rather than bringing back all this code we just removed?

@lysnikolaou
Copy link
Member Author

Not really. I opened this in a hurry to give @Yhg1s the option to unskip test_peg_generator before beta1, since the associated issue was marked as a release blocker. Since the consensus is that it's not, I'll take the time to port this to setuptools instead.

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

Successfully merging this pull request may close these issues.

3 participants