From 40c68b29c46f28a10cbba0d725afd4f055cb5d5a Mon Sep 17 00:00:00 2001
From: Joel Grunbaum <joelgrun@gmail.com>
Date: Fri, 20 Feb 2026 03:52:01 +0000
Subject: [PATCH] More tries at dep res

---
 src/archrepobuild/resolver.py |   60 ++++++++++++++++++++++++++++++++++++++++++++++++------------
 1 files changed, 48 insertions(+), 12 deletions(-)

diff --git a/src/archrepobuild/resolver.py b/src/archrepobuild/resolver.py
index f969989..70d9f3b 100644
--- a/src/archrepobuild/resolver.py
+++ b/src/archrepobuild/resolver.py
@@ -105,15 +105,15 @@
             logger.warning(f"Failed to get pacman package list: {e}")
             self._pacman_cache = {}
 
-    def is_in_official_repos(self, name: str, include_all: bool = True) -> bool:
-        """Check if package is available in official repositories.
+    def is_in_repos(self, name: str, include_all: bool = True) -> str | None:
+        """Check if package is available in repositories.
 
         Args:
             name: Package name (without version constraint)
             include_all: If True, check all enabled repos. If False, only official ones.
 
         Returns:
-            True if available in repos
+            Name of repository where package was found, or None if not found
         """
         if not self._pacman_checked:
             self._refresh_pacman_cache()
@@ -125,8 +125,38 @@
             if not include_all and repo not in OFFICIAL_REPOS:
                 continue
             if base_name in pkgs:
-                return True
-        return False
+                return repo
+        
+        # Fallback: check provides via pacman -Sp
+        try:
+            # Use pacman -Sp --noconfirm to see if pacman can resolve it
+            result = subprocess.run(
+                ["pacman", "-Sp", "--noconfirm", base_name],
+                capture_output=True,
+                text=True,
+            )
+            if result.returncode == 0:
+                # Successfully resolved. Find what it resolved to.
+                # Output looks like: file:///var/cache/pacman/pkg/name-version...
+                output = result.stdout.strip().split("\n")[-1]
+                if output:
+                    filename = output.split("/")[-1]
+                    # The package name is before the version
+                    import re
+                    match = re.search(r"^(.*?)-[0-9].*", filename)
+                    if match:
+                        resolved_name = match.group(1)
+                        # Now check which repo this resolved_name belongs to
+                        # We already have resolved_name in our cache if it's in a repo
+                        for repo, pkgs in self._pacman_cache.items():
+                            if not include_all and repo not in OFFICIAL_REPOS:
+                                continue
+                            if resolved_name in pkgs:
+                                return repo
+        except Exception as e:
+            logger.debug(f"Failed to resolve provides for {base_name}: {e}")
+
+        return None
 
     def is_installed(self, name: str) -> bool:
         """Check if package is already installed.
@@ -170,14 +200,15 @@
         all_deps: list[str] = []
         all_deps.extend(package.depends)
         all_deps.extend(package.makedepends)
+        all_deps.extend(package.checkdepends)
 
         aur_deps: list[str] = []
         for dep in all_deps:
             dep_parsed = Dependency.parse(dep)
             base_name = dep_parsed.name
 
-            # Skip if in official repos or already installed
-            if self.is_in_official_repos(base_name):
+            # Skip if in repos or already installed
+            if self.is_in_repos(base_name):
                 continue
             if self.is_installed(base_name):
                 continue
@@ -292,11 +323,11 @@
         Raises:
             ValueError: If package not found or circular dependency
         """
-        # Filter out packages already in official repos or installed
+        # Filter out packages already in repos or installed
         aur_package_names = []
         for name in package_names:
-            if self.is_in_official_repos(name):
-                logger.info(f"Package {name} found in official repositories, skipping AUR lookup")
+            if self.is_in_repos(name):
+                logger.info(f"Package {name} found in repositories, skipping AUR lookup")
                 continue
             if self.is_installed(name):
                 logger.info(f"Package {name} is already installed, skipping AUR lookup")
@@ -339,12 +370,17 @@
                 deps: list[Dependency] = []
                 for dep in pkg.depends:
                     parsed = Dependency.parse(dep, DependencyType.RUNTIME)
-                    if not self.is_in_official_repos(parsed.name):
+                    if not self.is_in_repos(parsed.name):
                         parsed.is_aur = True
                     deps.append(parsed)
                 for dep in pkg.makedepends:
                     parsed = Dependency.parse(dep, DependencyType.BUILD)
-                    if not self.is_in_official_repos(parsed.name):
+                    if not self.is_in_repos(parsed.name):
+                        parsed.is_aur = True
+                    deps.append(parsed)
+                for dep in pkg.checkdepends:
+                    parsed = Dependency.parse(dep, DependencyType.CHECK)
+                    if not self.is_in_repos(parsed.name):
                         parsed.is_aur = True
                     deps.append(parsed)
                 aur_deps[name] = deps

--
Gitblit v1.10.0