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

fix: support CMake 3.27, drop 3.4 #4719

Merged
merged 4 commits into from
Jul 12, 2023

Conversation

henryiii
Copy link
Collaborator

Description

This is roughly what I was thinking to support CMake 3.27.

Suggested changelog entry:

* Drop CMake 3.4 support, add CMake 3.27 support. FindPython will be used if FindPythonInterp is not present.

@henryiii
Copy link
Collaborator Author

CMake Error at tools/pybind11Common.cmake:166 (cmake_policy):
[20](https://github.com/pybind/pybind11/actions/runs/5394892712/jobs/9796691928#step:5:21)
  cmake_policy GET given policy "CMP0148" which is not known to this version
[21](https://github.com/pybind/pybind11/actions/runs/5394892712/jobs/9796691928#step:5:22)
  of CMake.

Okay, that's annoying. Will have to protect it, then.

Signed-off-by: Henry Schreiner <[email protected]>
@henryiii henryiii marked this pull request as ready for review June 27, 2023 23:09
@henryiii henryiii requested a review from rwgk July 9, 2023 14:45
docs/upgrade.rst Outdated Show resolved Hide resolved
@rwgk
Copy link
Collaborator

rwgk commented Jul 11, 2023

@henryiii is there a chance that you can finalize & merge this today? I want to update smart_holder & pywrapcc, to catch up with all the small updates here.

@rwgk
Copy link
Collaborator

rwgk commented Jul 12, 2023

@henryiii all tests pass. Looks great to me, could you please merge if the work is complete?

@henryiii henryiii merged commit 2e5f5c4 into pybind:master Jul 12, 2023
80 checks passed
@henryiii henryiii deleted the henryiii/fix/cmake27 branch July 12, 2023 18:10
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Jul 12, 2023
@rwgk rwgk removed the needs changelog Possibly needs a changelog entry label Jul 15, 2023
hoffmang9 added a commit to Chia-Network/bls-signatures that referenced this pull request Sep 2, 2023
Cmake is deprecating the way pybind finds the python interpreter. See https://cmake.org/cmake/help/latest/policy/CMP0148.html and pybind/pybind11#4719
hoffmang9 added a commit to Chia-Network/chiapos that referenced this pull request Sep 2, 2023
Cmake is deprecating the way pybind finds the python interpreter. See
https://cmake.org/cmake/help/latest/policy/CMP0148.html
and
pybind/pybind11#4719
hoffmang9 added a commit to Chia-Network/chiavdf that referenced this pull request Sep 2, 2023
Cmake is deprecating the way pybind finds the python interpreter. See
https://cmake.org/cmake/help/latest/policy/CMP0148.html and
pybind/pybind11#4719
hoffmang9 added a commit to Chia-Network/bls-signatures that referenced this pull request Nov 7, 2023
Cmake is deprecating the way pybind finds the python interpreter. See https://cmake.org/cmake/help/latest/policy/CMP0148.html and pybind/pybind11#4719
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.

2 participants