Add wheel repair dependencies

Signed-off-by: Cristian Le <[email protected]>

Author: LecrisUT

pyproject.toml

@@ -68,6 +68,8 @@ test-hatchling = [
 test-meta = [
     "hatch-fancy-pypi-readme>=22.3",
     "setuptools-scm",
+    "auditwheel; platform_system=='Linux'",
+    "delocate; platform_system=='Darwin'",
 ]
 test-numpy = [
     "numpy; python_version<'3.13' and platform_python_implementation!='PyPy'",
@@ -179,7 +181,15 @@ disallow_untyped_defs = true
 disallow_incomplete_defs = true
 
 [[tool.mypy.overrides]]
-module = ["numpy", "pathspec", "setuptools_scm", "hatch_fancy_pypi_readme", "virtualenv"]
+module = [
+    "numpy",
+    "pathspec",
+    "setuptools_scm",
+    "hatch_fancy_pypi_readme",
+    "virtualenv",
+    "auditwheel.*",
+    "delocate.*",
+]
 ignore_missing_imports = true
 
 

src/scikit_build_core/build/__init__.py

@@ -148,6 +148,7 @@ def get_requires_for_build_sdist(
     return [
         *cmake_requires,
         *requires.dynamic_metadata(),
+        *requires.other_dynamic_requires(),
     ]
 
 
@@ -166,6 +167,7 @@ def get_requires_for_build_wheel(
     return [
         *cmake_requires,
         *requires.dynamic_metadata(),
+        *requires.other_dynamic_requires(),
     ]
 
 
@@ -184,4 +186,5 @@ def get_requires_for_build_editable(
     return [
         *cmake_requires,
         *requires.dynamic_metadata(),
+        *requires.other_dynamic_requires(),
     ]

src/scikit_build_core/builder/__main__.py

@@ -31,7 +31,12 @@ def main() -> None:
 
     if Path("pyproject.toml").is_file():
         req = GetRequires()
-        all_req = [*req.cmake(), *req.ninja(), *req.dynamic_metadata()]
+        all_req = [
+            *req.cmake(),
+            *req.ninja(),
+            *req.dynamic_metadata(),
+            *req.other_dynamic_requires(),
+        ]
         rich_print(f"{{bold.red}}Get Requires:{{normal}} {all_req!r}")
 
     ip_program_search(color="magenta")

src/scikit_build_core/builder/get_requires.py

@@ -4,6 +4,8 @@
 import functools
 import importlib.util
 import os
+import platform
+import shutil
 import sysconfig
 from typing import TYPE_CHECKING, Literal
 
@@ -137,17 +139,28 @@ def ninja(self) -> Generator[str, None, None]:
             return
         yield f"ninja{ninja_verset}"
 
-    def dynamic_metadata(self) -> Generator[str, None, None]:
-        if self.settings.fail:
-            return
-
+    def other_dynamic_requires(self) -> Generator[str, None, None]:
         for build_require in self.settings.build.requires:
             yield build_require.format(
                 **pyproject_format(
                     settings=self.settings,
                 )
             )
 
+        if self.settings.wheel.repair:
+            platform_system = platform.system()
+            if platform_system == "Linux":
+                yield "auditwheel"
+                patchelf_path = shutil.which("patchelf")
+                if patchelf_path is None:
+                    yield "patchelf"
+            elif platform_system == "Darwin":
+                yield "delocate"
+
+    def dynamic_metadata(self) -> Generator[str, None, None]:
+        if self.settings.fail:
+            return
+
         for dynamic_metadata in self.settings.metadata.values():
             if "provider" in dynamic_metadata:
                 config = dynamic_metadata.copy()

src/scikit_build_core/hatch/plugin.py

@@ -111,7 +111,11 @@ def dependencies(self) -> list[str]:
 
         # These are only injected if cmake is required
         cmake_requires = [*requires.cmake(), *requires.ninja()] if required else []
-        return [*cmake_requires, *requires.dynamic_metadata()]
+        return [
+            *cmake_requires,
+            *requires.dynamic_metadata(),
+            *requires.other_dynamic_requires(),
+        ]
 
     def initialize(self, version: str, build_data: dict[str, Any]) -> None:
         if version == "editable":

tests/test_dynamic_metadata.py

@@ -371,18 +371,18 @@ def test_build_requires_field(override, monkeypatch) -> None:
     settings_reader.validate_may_exit()
 
     if override is None:
-        assert set(GetRequires().dynamic_metadata()) == {
+        assert set(GetRequires().other_dynamic_requires()) == {
             "foo",
         }
     elif override == "env":
         # evaluate ../foo as uri
         foo_path = pyproject_path.absolute().parent.parent / "foo"
         foo_path = foo_path.absolute()
-        assert set(GetRequires().dynamic_metadata()) == {
+        assert set(GetRequires().other_dynamic_requires()) == {
             f"foo @ {foo_path.as_uri()}",
         }
     elif override == "sdist":
-        assert set(GetRequires().dynamic_metadata()) == {
+        assert set(GetRequires().other_dynamic_requires()) == {
             # TODO: Check if special handling should be done for sdist
             "foo",
         }