Skip to content

Conversation

@NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Sep 30, 2024

This PR enable the build of M1 wheels to rely on python -m build instead of python setup.py. The default is still python setup.py.
This is the same as #5071 and #5076 that were landed for linux wheels.

We need this for torchcodec to support M1 wheels meta-pytorch/torchcodec#230

CC @scotts @atalman

@vercel
Copy link

vercel bot commented Sep 30, 2024

@NicolasHug is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 30, 2024
Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

@atalman atalman merged commit 9aa668e into pytorch:main Oct 1, 2024
NicolasHug added a commit to NicolasHug/test-infra that referenced this pull request Nov 18, 2024
This PR enable the build of M1 wheels to rely on `python -m build`
instead of `python setup.py`. The default is still `python setup.py`.
This is the same as pytorch#5071 and
pytorch#5076 that were landed for
linux wheels.

We need this for torchcodec to support M1 wheels
meta-pytorch/torchcodec#230

CC @scotts @atalman
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants