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

Pin setuptools 72.1.0 #1995

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Pin setuptools 72.1.0 #1995

merged 2 commits into from
Sep 18, 2024

Conversation

atalman
Copy link
Contributor

@atalman atalman commented Sep 18, 2024

if "%%v" == "3.11" call conda create -n py!PYTHON_VERSION_STR! -y -q -c=conda-forge numpy=2.0.1 pyyaml boto3 cmake ninja typing_extensions setuptools python=%%v
if "%%v" == "3.12" call conda create -n py!PYTHON_VERSION_STR! -y -q -c=conda-forge numpy=2.0.1 pyyaml boto3 cmake ninja typing_extensions setuptools python=%%v
if "%%v" == "3.13" call conda create -n py!PYTHON_VERSION_STR! -y -q -c=conda-forge numpy=2.0.1 pyyaml boto3 cmake ninja typing_extensions setuptools python=%%v
if "%%v" == "3.8" call conda create -n py!PYTHON_VERSION_STR! -y -q numpy=1.11 pyyaml boto3 cmake ninja typing_extensions setuptools=72.1.0 python=%%v
Copy link
Member

Choose a reason for hiding this comment

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

do we still need python 3.8?

@atalman atalman changed the title pin_setuptools_72.1.0 Pin setuptools 72.1.0 Sep 18, 2024
windows/condaenv.bat Outdated Show resolved Hide resolved
@atalman atalman merged commit 65eb45c into main Sep 18, 2024
2 checks passed
@atalman atalman deleted the pin_setuptools_72.1.0 branch September 18, 2024 23:47
atalman added a commit that referenced this pull request Sep 23, 2024
Same underlyning issue as wheel/libtorch build failure fixed by: #1995
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Sep 25, 2024
FIXES: #136541

Setuptools>=74.0.0 has deprecated support for some functions in distutils, and so the builds run into error such as ```AttributeError: module 'distutils' has no attribute '_msvccompiler'```. Also, the pytorch builds have setuptools pin to 72.1.0 according to these PRs: pytorch/builder#1995 and pytorch/builder@89d9a8c. So, until there is a fix to change the function usage in accordance with latest setuptools, the 72.1.0 version works fine.

Also observed in CI jobs: https://github.com/pytorch/pytorch/actions/runs/10979326524
Pull Request resolved: #136489
Approved by: https://github.com/malfet

Co-authored-by: Nikita Shulga <2453524+malfet@users.noreply.github.com>
kit1980 pushed a commit that referenced this pull request Sep 26, 2024
* pin_setuptools_72.1.0

* Update windows/condaenv.bat
kit1980 pushed a commit that referenced this pull request Sep 26, 2024
Same underlyning issue as wheel/libtorch build failure fixed by: #1995
pytorchbot pushed a commit to pytorch/pytorch that referenced this pull request Sep 26, 2024
FIXES: #136541

Setuptools>=74.0.0 has deprecated support for some functions in distutils, and so the builds run into error such as ```AttributeError: module 'distutils' has no attribute '_msvccompiler'```. Also, the pytorch builds have setuptools pin to 72.1.0 according to these PRs: pytorch/builder#1995 and pytorch/builder@89d9a8c. So, until there is a fix to change the function usage in accordance with latest setuptools, the 72.1.0 version works fine.

Also observed in CI jobs: https://github.com/pytorch/pytorch/actions/runs/10979326524
Pull Request resolved: #136489
Approved by: https://github.com/malfet

Co-authored-by: Nikita Shulga <2453524+malfet@users.noreply.github.com>
(cherry picked from commit 8d65d9f)
kit1980 pushed a commit to pytorch/pytorch that referenced this pull request Sep 26, 2024
Constraint setuptools to 72.1.0 or older in requirements.txt (#136489)

FIXES: #136541

Setuptools>=74.0.0 has deprecated support for some functions in distutils, and so the builds run into error such as ```AttributeError: module 'distutils' has no attribute '_msvccompiler'```. Also, the pytorch builds have setuptools pin to 72.1.0 according to these PRs: pytorch/builder#1995 and pytorch/builder@89d9a8c. So, until there is a fix to change the function usage in accordance with latest setuptools, the 72.1.0 version works fine.

Also observed in CI jobs: https://github.com/pytorch/pytorch/actions/runs/10979326524
Pull Request resolved: #136489
Approved by: https://github.com/malfet

Co-authored-by: Nikita Shulga <2453524+malfet@users.noreply.github.com>
(cherry picked from commit 8d65d9f)

Co-authored-by: ratnampa <ratnam.parikh@intel.com>
pytorchmergebot pushed a commit to sanchitintel/pytorch that referenced this pull request Sep 26, 2024
…#136489)

FIXES: pytorch#136541

Setuptools>=74.0.0 has deprecated support for some functions in distutils, and so the builds run into error such as ```AttributeError: module 'distutils' has no attribute '_msvccompiler'```. Also, the pytorch builds have setuptools pin to 72.1.0 according to these PRs: pytorch/builder#1995 and pytorch/builder@89d9a8c. So, until there is a fix to change the function usage in accordance with latest setuptools, the 72.1.0 version works fine.

Also observed in CI jobs: https://github.com/pytorch/pytorch/actions/runs/10979326524
Pull Request resolved: pytorch#136489
Approved by: https://github.com/malfet

Co-authored-by: Nikita Shulga <2453524+malfet@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants