Skip to content

Respect configured NuGet package sources - #2966

Open
CAOShurong wants to merge 2 commits into
pypa:mainfrom
CAOShurong:agent/remove-nuget-fallback-source
Open

Respect configured NuGet package sources#2966
CAOShurong wants to merge 2 commits into
pypa:mainfrom
CAOShurong:agent/remove-nuget-fallback-source

Conversation

@CAOShurong

Copy link
Copy Markdown

Closes #2961.

Summary

  • Remove the hard-coded nuget.org -FallbackSource from Windows CPython installs.
  • Let NuGet honor the package sources selected by its configuration.
  • Add regression coverage for regular AMD64 and free-threaded ARM64 package names.

Testing

  • pytest unit_test/get_platform_test.py (7 passed)
  • PYTHONUTF8=1 nox -s tests -- unit_test (854 passed, 68 skipped, 3 xfailed)
  • pre-commit run --files cibuildwheel/platforms/windows.py unit_test/get_platform_test.py
  • pylint cibuildwheel/platforms/windows.py
  • Exercised install_cpython with an empty local-only NuGet source: it reported only that source and failed as expected.
  • Exercised install_cpython with nuget.org explicitly configured: it installed CPython 3.14.1 successfully.

@joerick joerick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CI is passing so I'm happy

Comment thread unit_test/get_platform_test.py Outdated
("arch", "free_threaded", "package_name"),
[("AMD64", False, "python"), ("ARM64", True, "pythonarm64-freethreaded")],
)
def test_nuget_args_use_configured_sources(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
def test_nuget_args_use_configured_sources(
def test_get_nuget_args(

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Applied in 51c5d3f; the focused Windows tests still pass. 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.

Allow skipping of -FallbackSource in cpython NuGet installs

2 participants