Skip to content

[build] remove upper version pin for setuptools<80.0#156049

Closed
XuehaiPan wants to merge 97 commits intogh/XuehaiPan/290/basefrom
gh/XuehaiPan/290/head
Closed

[build] remove upper version pin for setuptools<80.0#156049
XuehaiPan wants to merge 97 commits intogh/XuehaiPan/290/basefrom
gh/XuehaiPan/290/head

Conversation

@XuehaiPan
Copy link
Collaborator

@XuehaiPan XuehaiPan commented Jun 16, 2025

Invoking setup.py commands is deprecated (especially, python3 setup.py develop), and the support has been removed in the latest setuptools. See also:

This PR removes the setuptools>80.0 pin because the setup.py command migration has been resolved by previous PRs in the stack:


Stack from ghstack (oldest at bottom):

cc @malfet @seemethere

[ghstack-poisoned]
@XuehaiPan XuehaiPan requested review from a team and jeffdaily as code owners June 16, 2025 04:00
@pytorch-bot pytorch-bot bot added the release notes: releng release notes category label Jun 16, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Jun 16, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/156049

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (4 Unrelated Failures)

As of commit b9a83c9 with merge base 7441a1b (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[ghstack-poisoned]
XuehaiPan added a commit that referenced this pull request Jun 16, 2025
[ghstack-poisoned]
XuehaiPan added a commit that referenced this pull request Jun 17, 2025
[ghstack-poisoned]
XuehaiPan added a commit that referenced this pull request Jun 19, 2025
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@iremyux
Copy link
Collaborator

iremyux commented Aug 14, 2025

Adding ciflow/win-arm64 label to trigger Windows Arm64 CI and its test purposes - nothing about this PR specifically. (It should not effect the acceptance of the PR even if it fails.)

[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries_libtorch Trigger binary build and upload jobs for libtorch on the PR ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR ciflow/binaries Trigger all binary build and upload jobs on the PR ciflow/inductor ciflow/nightly Trigger all jobs we run nightly (nightly.yml) ciflow/trunk Trigger trunk jobs on your pull request ciflow/win-arm64 Trigger Windows Arm64 CI Workflows merging module: build Build system issues open source release notes: build release notes category release notes: releng release notes category topic: build topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants