From 6e918d63b6613f1190ccba1613346163271a85a1 Mon Sep 17 00:00:00 2001
From: Ludwig Nussel <ludwig.nussel@suse.de>
Date: Thu, 28 Nov 2019 10:30:37 +0100
Subject: [PATCH] check_source: query all repos if source info yields error

---
 check_source.py | 38 +++++++++++++++++++++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

diff --git a/check_source.py b/check_source.py
index 8e1d0aae..790ea61f 100755
--- a/check_source.py
+++ b/check_source.py
@@ -278,10 +278,12 @@ class CheckSource(ReviewBot.ReviewBot):
             sys.stdout = _stdout
         return result
 
-    def package_source_parse(self, project, package, revision=None):
+    def _package_source_parse(self, project, package, revision=None, repository=None):
         query = {'view': 'info', 'parse': 1}
         if revision:
             query['rev'] = revision
+        if repository:
+            query['repository'] = repository
         url = osc.core.makeurl(self.apiurl, ['source', project, package], query)
 
         ret = {'name': None, 'version': None}
@@ -292,6 +294,10 @@ class CheckSource(ReviewBot.ReviewBot):
             self.logger.error('ERROR in URL %s [%s]' % (url, e))
             return ret
 
+        if xml.find('error') is not None:
+            self.logger.error("%s/%s/%s: %s", project, package, repository, xml.find('error').text)
+            return ret
+
         # ET boolean check fails.
         if xml.find('name') is not None:
             ret['name'] = xml.find('name').text
@@ -302,6 +308,36 @@ class CheckSource(ReviewBot.ReviewBot):
         if xml.find('filename') is not None:
             ret['filename'] = xml.find('filename').text
 
+        self.logger.debug("%s/%s/%s: %s", project, package, repository, ret)
+
+        return ret
+
+    def package_source_parse(self, project, package, revision=None):
+        ret = self._package_source_parse(project, package, revision)
+
+        if ret['name'] is not None:
+            return ret
+
+        d = {}
+        for repo in osc.core.get_repositories_of_project(self.apiurl, project):
+            r = self._package_source_parse(project, package, revision, repo)
+            if r['name'] is not None:
+                d[r['name']] = r
+
+        if len(d) == 1:
+            # here is only one so use that
+            ret = d[next(iter(d))]
+        else:
+            # check if any name matches
+            self.logger.debug("found multiple names %s", ', '.join(d.keys()))
+            for n, r in d.items():
+                if n == package:
+                    ret = r
+                    break
+
+            if ret['name'] is None:
+                self.logger.error("none of the names matched")
+
         return ret
 
     def only_changes(self):