Skip to content

Conversation

mhucka
Copy link
Collaborator

@mhucka mhucka commented Oct 2, 2025

This matches the recent changes in other files to use cmake 3.28.

mhucka added 3 commits October 2, 2025 02:08
This matches the recent changes in other files to use cmake 3.28.
Some additional Python files need to be changed too.
@github-actions github-actions bot added the size: S 10< lines changed <50 label Oct 2, 2025
@mhucka mhucka marked this pull request as ready for review October 2, 2025 02:55
@mhucka mhucka changed the title Change Dockerfiles to use cmake 3.28 Update additional files use cmake 3.28 instead of 3.31 Oct 2, 2025
@mhucka mhucka changed the title Update additional files use cmake 3.28 instead of 3.31 Update additional files to use cmake 3.28 instead of 3.31 Oct 21, 2025
@mhucka mhucka changed the title Update additional files to use cmake 3.28 instead of 3.31 Update additional files to use CMake 3.28 instead of 3.31 Oct 21, 2025
Copy link
Collaborator

@fdmalone fdmalone left a comment

Choose a reason for hiding this comment

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

Why was it rolled back?

@mhucka
Copy link
Collaborator Author

mhucka commented Oct 21, 2025

Why was it rolled back?

Sergei noted in his recent big PR that the default version of cmake on Ubuntu Linux is currently 3.28. Originally, I thought there was something in version 3.31 that was needed, but after testing with 3.28, I couldn't find anything breaking or even warnings, so whatever it was, it doesn't seem to have been critical.

(Some references to 3.31 in the qsim cmake files were already changed to 3.28, but I missed some others. This PR should catch the rest.)

@mhucka mhucka added this pull request to the merge queue Oct 21, 2025
Merged via the queue into quantumlib:main with commit beacb4f Oct 21, 2025
57 checks passed
@mhucka mhucka deleted the mh-misc-cmake-version-changes branch October 21, 2025 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants