Skip to content

Commit 3a827f2

Browse files
committed
[skip ci] build: clarify Eigen 5 module fallback comment
Explain that the MODULE-mode fallback only exists for older Eigen 3 setups so the remaining fallback path does not look like an unresolved Eigen 5 issue. Made-with: Cursor
1 parent f303db9 commit 3a827f2

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

tests/CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,8 @@ if(PYBIND11_TEST_FILES_EIGEN_I GREATER -1)
309309
if(NOT EIGEN3_FOUND)
310310
# Couldn't load via target, so fall back to allowing module mode finding, which will pick up
311311
# tools/FindEigen3.cmake
312-
# XXX: MODULE mode does not work with Eigen 5
312+
# This MODULE-mode fallback is for older Eigen 3 setups; Eigen 5 is expected to be found
313+
# via the CONFIG-mode probes above.
313314
find_package(Eigen3 3.2.7 QUIET)
314315
endif()
315316
endif()

0 commit comments

Comments
 (0)