Skip to content

Conversation

@ThreeMonth03
Copy link
Collaborator

@ThreeMonth03 ThreeMonth03 commented Jan 25, 2026

Problem

According to the discussion in issue #648, we decide to upgrade the CMake to version>=4.

Solution

The latest version of cmake is 4.2.2. Due to the script in devenv, this pull request upgrade the CMake to version=4.0.1, including CMakeLists.txt and .github/**.yml.

	- Modify the minimum required version to 4.0.1 in **/CMakeLists.txt
	- Install CMake=4.0.1 to every GA runner.
Copy link
Collaborator Author

@ThreeMonth03 ThreeMonth03 left a comment

Choose a reason for hiding this comment

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

@yungyuc Please review this issue. Thanks.

Comment on lines -40 to +48
sudo curl git build-essential make cmake libc6-dev gcc g++ \
sudo curl git build-essential make libc6-dev gcc g++ \
python3 python3-dev python3-venv

# The latest CMake version from apt-get is 3.28.3, so we install CMake>=4.0.1 from github marketplace.
- name: install CMake=4.0.1
uses: lukka/get-cmake@latest
with:
cmakeVersion: 4.0.1

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Upgrade CMake from another github actions.

- name: setup.py install build_ext
run: |
sudo python3 setup.py install build_ext \
sudo env "PATH=$PATH" python3 setup.py install build_ext \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

When running the command with sudo, the command would use the original cmake and raise error.
In this discussion, the author suggests to pass the enviroment variables to bypass the problem.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant