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

Merge pypa/distutils #4649

Merged
merged 31 commits into from
Sep 15, 2024
Merged

Merge pypa/distutils #4649

merged 31 commits into from
Sep 15, 2024

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Sep 15, 2024

  • Ignore TRY400 globally
  • Clean up ruff ignore
  • Clean up cruft in errors docstring.
  • Rely on dependencies instead of vendoring them.
  • Simply log the directory being created, rather than the whole ancestry.
  • Updated mkpath to use pathlib.Path.mkdir.
  • Refactored mkpath as a singledispatch function.
  • Replaced global variable with a custom cache wrapper.
  • Extracted _copy_one function.
  • "Removed support for 'compress="compress"' to archive_util.make_tarball."
  • Removed deprecated 'check_warnings' from 'sdist' and 'regitser' commands.
  • Enabled TestPyPIRCCommand to run its tests.
  • Removed dependence of TestSdist on test_config.
  • Replaced open/read/close logic with 'read_text' and added encoding to address EncodingWarning.
  • Remove collect_ignore referring to msvc9compiler.
  • Flagged register and upload commands and deprecated.
  • Decouple sdist tests from pypirc fixture.
  • Removed PyPI commands (register, upload) and supporting logic.
  • Remove borland compiler.
  • cygwinccompiler: Get the compilers from sysconfig
  • Fix cross-platform compilation using distutils._msvccompiler.MSVCCompiler Actually use the plat_name param in MSVCCompiler.initialize Added tests
  • Move the comment to the docstring.
  • Use double-backticks for rst compatibility.
  • Remove news fragments, not useful here.

Summary of changes

Closes

Pull Request Checklist

DimitriPapadopoulos and others added 30 commits September 4, 2024 09:12
While the suggestion might be valid in some cases, it doesn't fit all
use cases and ends up being a nuisance.
Removed PyPI commands (register, upload) and supporting logic.
On the CLANG64 environment of MSYS2, we should use clang
instead of gcc. This patch gets the compiler from sysconfig which
would be set when Python was built.

Without this patch, the build fails when it's trying to check if the
compiler is cygwin's one.
…piler`

Actually use the `plat_name` param in `MSVCCompiler.initialize`
Added tests
…piler`

Merge pull request pypa/distutils#298 from Avasam/MSVCCompiler-cross-platform-bug
Merge pull request pypa/distutils#296 from msys2-contrib/clang-compilers
@jaraco jaraco marked this pull request as ready for review September 15, 2024 14:57
@jaraco jaraco merged commit af9e245 into main Sep 15, 2024
46 checks passed
@jaraco jaraco deleted the feature/distutils-7283751 branch September 15, 2024 15:31
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.

4 participants