Skip to content

Commit

Permalink
PYBIND11_FINDPYTHON=ON
Browse files Browse the repository at this point in the history
Reuse our `find_package(Python ...)` call and use new CMake logic in
pybind11.
https://pybind11.readthedocs.io/en/stable/compiling.html#modules-with-cmake

Signed-off-by: Axel Huebl <[email protected]>
  • Loading branch information
ax3l committed Oct 11, 2024
1 parent 240592f commit a4c250b
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions cmake/dependencies/pybind11.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,11 @@ function(find_pybind11)
message(STATUS "pybind11 repository: ${openPMD_pybind11_repo} (${openPMD_pybind11_branch})")
endif()
endif()

# Propagate Python to pybind11 in superbuilds
if(openPMD_USE_INTERNAL_PYBIND11)
# we found Python via find_package(Python ...) below, but pybind11 has some
# additional internal logic in FindPythonLibsNew.cmake can mix things up
# https://github.com/openPMD/openPMD-api/pull/1677#issuecomment-2407743771
set(PYTHON_EXECUTABLE ${Python_EXECUTABLE})
endif()


# rely on our find_package(Python ...) call
# https://pybind11.readthedocs.io/en/stable/compiling.html#modules-with-cmake
set(PYBIND11_FINDPYTHON ON)

if(TARGET pybind11::module)
# nothing to do, target already exists in the superbuild
elseif(openPMD_USE_INTERNAL_PYBIND11 AND openPMD_pybind11_src)
Expand Down

0 comments on commit a4c250b

Please sign in to comment.