-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Merge with distutils @ 5589d7527 #4865
Conversation
Add a few missing
Another missing
# Conflicts: # ruff.toml
…ypeshed-annotations
Fix failing Ruff test
Remove Python version gate in _make_strs.
…_check-an-actual-boolean Make `sdist.metadata_check` an actual boolean
…tations Merge typeshed's `setuptools._distutils` annotations
In build_scripts, unconditionally set shebang to #!python.
|
@@ -255,10 +257,10 @@ class Compiler(base.Compiler): | |||
obj_extension = '.obj' | |||
static_lib_extension = '.lib' | |||
shared_lib_extension = '.dll' | |||
static_lib_format = shared_lib_format = '%s%s' | |||
static_lib_format = static_lib_format = '%s%s' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NVM, looks like you are already aware of this https://github.com/python/typeshed/blob/f6e99d20b503225404c638a536c39759b252dbc2/stubs/setuptools/setuptools/_distutils/compilers/C/msvc.pyi#L16
setuptools._distutils
annotationsSummary of changes
Closes
Pull Request Checklist
newsfragments/
.(See documentation for details)