Skip to content

Update CMake to 3.27 in Windows CI#153380

Closed
cyyever wants to merge 2 commits intopytorch:mainfrom
cyyever:minimal_cmake3
Closed

Update CMake to 3.27 in Windows CI#153380
cyyever wants to merge 2 commits intopytorch:mainfrom
cyyever:minimal_cmake3

Conversation

@cyyever
Copy link
Collaborator

@cyyever cyyever commented May 12, 2025

Before it's possible to use enable newer CMake.

@cyyever cyyever requested a review from a team as a code owner May 12, 2025 15:43
@pytorch-bot
Copy link

pytorch-bot bot commented May 12, 2025

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ You can merge normally! (2 Unrelated Failures)

As of commit eab12aa with merge base 9785b32 (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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

@cyyever
Copy link
Collaborator Author

cyyever commented May 12, 2025

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label May 12, 2025
@cyyever cyyever requested a review from Skylion007 May 12, 2025 15:46
@cyyever cyyever marked this pull request as draft May 13, 2025 00:12
@cyyever cyyever marked this pull request as ready for review May 13, 2025 01:00
@cyyever cyyever requested a review from Skylion007 May 13, 2025 01:00
@cyyever cyyever changed the title Update CMake to 3.26 in Windows CI Update CMake to 3.28 in Windows CI May 13, 2025
@cyyever cyyever requested review from atalman and clee2000 May 13, 2025 01:28
@cyyever cyyever added the ciflow/trunk Trigger trunk jobs on your pull request label May 13, 2025
@cyyever cyyever changed the title Update CMake to 3.28 in Windows CI Update CMake to 3.27 in Windows CI May 13, 2025
@cyyever cyyever requested review from albanD and malfet May 14, 2025 00:57
Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

Sounds ok beside maybe upgrading to an even newer version.

if not errorlevel 0 goto fail

:: Update CMake
call choco upgrade -y cmake --no-progress --installargs 'ADD_CMAKE_TO_PATH=System' --apply-install-arguments-to-dependencies --version=3.27.9
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isn't linux on 4.0.0 ? Should we upgrade all the way there while we're at it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tried 3.28, there are some breaking changes make CI fail...

@cyyever
Copy link
Collaborator Author

cyyever commented May 15, 2025

@pytorchbot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 2 checks: pull / cuda12.4-py3.10-gcc9-sm75 / test (pr_time_benchmarks, 1, 1, linux.g4dn.metal.nvidia.gpu), pull / linux-jammy-py3-clang12-executorch / test (executorch, 1, 1, lf.linux.2xlarge)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@cyyever cyyever deleted the minimal_cmake3 branch November 25, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged open source topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants