[Mlir-commits] [llvm] [mlir] [MLIR][Python] remove pybind11 support (PR #172581)

[Maksim Levental]

================
@@ -46,81 +46,41 @@ macro(mlir_configure_python_dev_packages)
     message(STATUS "Found python include dirs: ${Python3_INCLUDE_DIRS}")
     message(STATUS "Found python libraries: ${Python3_LIBRARIES}")
     message(STATUS "Found numpy v${Python3_NumPy_VERSION}: ${Python3_NumPy_INCLUDE_DIRS}")
-    mlir_detect_pybind11_install()
-    find_package(pybind11 2.10 CONFIG REQUIRED)
-    message(STATUS "Found pybind11 v${pybind11_VERSION}: ${pybind11_INCLUDE_DIR}")
-    message(STATUS "Python prefix = '${PYTHON_MODULE_PREFIX}', "
-                  "suffix = '${PYTHON_MODULE_SUFFIX}', "
-                  "extension = '${PYTHON_MODULE_EXTENSION}")
-
-    mlir_detect_nanobind_install()
+    message(STATUS "Python extension suffix for modules: '${Python3_SOABI}'")
+    if(nanobind_DIR)
+      message(STATUS "Using explicit nanobind cmake directory: ${nanobind_DIR} (-Dnanobind_DIR to change)")
+    else()
+      message(STATUS "Checking for nanobind in python path...")
+      execute_process(
+        COMMAND "${Python3_EXECUTABLE}"
+        -c "import nanobind;print(nanobind.cmake_dir(), end='')"
+        WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+        RESULT_VARIABLE STATUS
+        OUTPUT_VARIABLE PACKAGE_DIR
+        ERROR_QUIET)
+      if(NOT STATUS EQUAL "0")
+        message(STATUS "not found (install via 'pip install nanobind' or set nanobind_DIR)")
+        return()
----------------
makslevental wrote:

Sure that's my mistake (this used to be part of a function). Can you send a PR or do you want me to do it?

https://github.com/llvm/llvm-project/pull/172581