From 586439b625ea7ba2ea85e7926340b11b7c6bffaf Mon Sep 17 00:00:00 2001 From: zachcran Date: Mon, 9 Jun 2025 09:37:13 -0600 Subject: [PATCH 1/2] Fix incorrect PyBind11 headers target reference --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 9c855e62..27117395 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -72,7 +72,7 @@ if("${BUILD_PYBIND11_PYBINDINGS}") nwx_find_pybind11() list( APPEND pluginplay_depends - pybind11_headers pybind11::embed Python::Python + pybind11::headers pybind11::embed Python::Python ) endif() From 288862932cff39442de4324d6a5d59541cb03e44 Mon Sep 17 00:00:00 2001 From: zachcran Date: Mon, 9 Jun 2025 14:01:26 -0600 Subject: [PATCH 2/2] Add more explanation to incomplete PyBind11 import explanation (and rerun tests) --- CMakeLists.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 27117395..5d55870e 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -66,7 +66,8 @@ cmaize_find_optional_dependency( set(pluginplay_depends utilities parallelzone libfort Boost::boost RocksDB) # As of 1.0.0 CMaize does not support multiple build or find targets. This will -# be fixed in a future feature release. For now we handle the +# be fixed in a future feature release. For now we handle the exposed CMake +# targets separately instead of a CMaize target include(nwx_pybind11) if("${BUILD_PYBIND11_PYBINDINGS}") nwx_find_pybind11()