[DEBUG] Add some more debug options

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

Author: LecrisUT

src/scikit_build_core/repair_wheel/__init__.py

@@ -206,6 +206,14 @@ def get_wheel_repairer(
                 wheel_dirs=wheel_dirs,
             )
 
+        logger.debug(
+            "wheel_dirs[platlib] = {platlib}\n"
+            "sysconfig.get_path(platlib) = {sysconfig_path}\n"
+            "install_dir = {install_dir}\n",
+            platlib=wheel_dirs["platlib"],
+            sysconfig_path=sysconfig.get_path("platlib"),
+            install_dir=install_dir,
+        )
         WheelRepairer.initialize()
         if not (
             repairer_cls := WheelRepairer._platform_repairers.get(platform.system())

src/scikit_build_core/repair_wheel/linux.py

@@ -48,12 +48,23 @@ def patch_linux_library_rpath(self, artifact: Path, rpaths: list[str]) -> None:
                 final_rpaths.add(rpath_str)
                 continue
             # Otherwise change the RPATH to point use $ORIGIN
+            logger.debug(
+                "Patch path from patch_linux_library_rpath: "
+                "rpath={rpath}, artifact={artifact}",
+                rpath=rpath_path,
+                artifact=artifact,
+            )
             new_rpath = self.path_relative_site_packages(rpath_path, artifact.parent)
             new_rpath_str = f"$ORIGIN/{new_rpath}"
             final_rpaths.add(new_rpath_str)
         # Merge with all the rpaths we were given
         final_rpaths = final_rpaths.union(rpaths)
         patcher = Patchelf()
+        logger.debug(
+            "Setting RPATH for {artifact} to: {rpath}",
+            artifact=artifact,
+            rpath=final_rpaths,
+        )
         patcher.set_rpath(artifact, ":".join(final_rpaths))
 
     def get_dependency_rpaths(self, target: Target, install_path: Path) -> list[str]:
@@ -64,6 +75,12 @@ def get_dependency_rpaths(self, target: Target, install_path: Path) -> list[str]
             dep_install_paths = self.get_wheel_install_paths(dep_target)
             assert len(dep_install_paths) == 1
             dep_install_path = self.install_dir / next(iter(dep_install_paths))
+            logger.debug(
+                "Patch path from get_dependency_rpaths: "
+                "dep_install_path={dep_install_path}, target_path={target_path}",
+                dep_install_path=dep_install_path,
+                target_path=target_path,
+            )
             rpath = self.path_relative_site_packages(dep_install_path, target_path)
             new_rpath_str = f"$ORIGIN/{rpath}"
             rpaths.append(new_rpath_str)
@@ -106,6 +123,13 @@ def get_package_rpaths(self, target: Target, install_path: Path) -> list[str]:
                         # Skip empty rpaths. Most likely will have on at the end
                         continue
                     rpath = Path(rpath_str)
+                    logger.debug(
+                        "Patch path from get_package_rpaths (1): "
+                        "rpath={rpath}, install_path={install_path}, is_in_site={is_in_site}",
+                        rpath=rpath,
+                        install_path=install_path,
+                        is_in_site=self.path_is_in_site_packages(rpath),
+                    )
                     if not self.path_is_in_site_packages(rpath):
                         # Skip any paths that cannot be handled. We do not check for paths in
                         # the build directory, it should be covered by `get_dependency_rpaths`
@@ -118,16 +142,24 @@ def get_package_rpaths(self, target: Target, install_path: Path) -> list[str]:
             try:
                 # TODO: how to best catch if a string is a valid path?
                 rpath = Path(link_command.fragment)
+                logger.debug(
+                    "Patch path from get_package_rpaths (2): "
+                    "rpath={rpath}, install_path={install_path}, is_in_site={is_in_site}",
+                    rpath=rpath,
+                    install_path=install_path,
+                    is_in_site=self.path_is_in_site_packages(rpath),
+                )
                 if not rpath.is_absolute():
                     # Relative paths should be handled by `get_dependency_rpaths`
                     continue
                 rpath = self.path_relative_site_packages(rpath, install_path)
                 new_rpath_str = f"$ORIGIN/{rpath.parent}"
                 rpaths.append(new_rpath_str)
-            except Exception:
+            except Exception as exc:
                 logger.warning(
-                    "Could not parse link-library as a path: {fragment}",
+                    "Could not parse link-library as a path: {fragment}\nexc = {exc}",
                     fragment=link_command.fragment,
+                    exc=exc,
                 )
                 continue
         return rpaths

tests/test_repair_wheel.py

@@ -35,7 +35,9 @@ def test_full_build(isolated, isolate, manual_repair, monkeypatch):
             isolated.install("delvewheel")
         isolated.install("./extern", isolated=isolate)
 
-    isolated.install("-v", "./extern", ".", isolated=isolate)
+    isolated.install(
+        "-v", "--config-settings=logging.level=DEBUG", "./extern", ".", isolated=isolate
+    )
 
     isolated.run("main")
     isolated.module("repair_wheel")