Skip to content

Advise against using distutils directly, take 2#1028

Merged
di merged 8 commits intopypa:mainfrom
astrojuanlu:issue-667-note-problem-with-old-distutils-core-examples-redux
Dec 16, 2021
Merged

Advise against using distutils directly, take 2#1028
di merged 8 commits intopypa:mainfrom
astrojuanlu:issue-667-note-problem-with-old-distutils-core-examples-redux

Conversation

@astrojuanlu
Copy link
Copy Markdown
Contributor

@astrojuanlu astrojuanlu commented Dec 7, 2021

Closes #667. Continuation of #672, closes #672.

I tried to respect the original spirit of #672, even if that meant not updating the rest of the page (i.e. noting the deprecation of direct setup.py invokations and encouragement for pyproject.toml for static metadata), as this would require a bit more work. I prefer to defer that to a future PR.

@astrojuanlu
Copy link
Copy Markdown
Contributor Author

In particular, I made some minimal changes to the wording and the intensity of the admonition to address @pfmoore comments here: #672 (comment)

Comment thread source/tutorials/packaging-projects.rst Outdated
@webknjaz
Copy link
Copy Markdown
Member

webknjaz commented Dec 7, 2021

@astrojuanlu
Copy link
Copy Markdown
Contributor Author

AFAIU, that blog post is about the deprecation of direct setup.py invocations, and this PR is solely about the deprecation of distutils.

@webknjaz
Copy link
Copy Markdown
Member

webknjaz commented Dec 7, 2021

Ah, okay, fair enough.

Comment thread source/tutorials/packaging-projects.rst Outdated
Comment thread source/tutorials/packaging-projects.rst Outdated
Comment thread source/tutorials/packaging-projects.rst Outdated
Co-authored-by: Brian Rutledge <brian@bhrutledge.com>
Copy link
Copy Markdown
Member

@di di left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once comments are resolved.

@astrojuanlu
Copy link
Copy Markdown
Contributor Author

Comments addressed 👍

@di di merged commit caeae8f into pypa:main Dec 16, 2021
@di
Copy link
Copy Markdown
Member

di commented Dec 16, 2021

Thanks!

@astrojuanlu astrojuanlu deleted the issue-667-note-problem-with-old-distutils-core-examples-redux branch December 16, 2021 22:42
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.

Formally drop support for distutils-only projects?

5 participants