commit 60aa31fc9deb166a04e11747a25d0f4966d3e42a
from: Stefan Sperling <stsp@stsp.name>
date: Wed Sep 11 11:16:55 2024 UTC

the pointer between commits must point the other way

pointed out by olasd

commit - 19231ab78f7566fb47c90eb83ccc54d4341bc176
commit + 60aa31fc9deb166a04e11747a25d0f4966d3e42a
blob - eb13f37423fcf1b24f6257c832bcdc3e9282fb6d
blob + 1adf1246871406ca6e02b4d4623b4fc60c71fc0c
--- swh/loader/git/loader.py
+++ swh/loader/git/loader.py
@@ -741,7 +741,7 @@ class GitLoader(BaseGitLoader):
                     if parent_hex in self.pack:
                         parents.append(parent_hex)
                         parent_hash = hashutil.bytehex_to_hash(parent_hex)
-                        commit_edges.append((parent_hash, commit_hash))
+                        commit_edges.append((commit_hash, parent_hash))
 
         # Add commits and root trees to the graph
         self._object_graph.add_vertices(list(commits.keys()))
@@ -793,15 +793,15 @@ class GitLoader(BaseGitLoader):
 
         for v in self._object_graph.vs:
             name = hashutil.hash_to_bytehex(v["name"])
-            predecessors = []
+            successors = []
             vertices = [v]
             while len(vertices) > 0:
                 v = vertices.pop(0)
-                for p in self._object_graph.predecessors(v):
-                    pname = self._object_graph.vs["name"][p]
-                    predecessors.append(hashutil.hash_to_bytehex(pname))
-                    vertices.append(p)
-            logger.debug(f"object {name} has predecessors {predecessors}")
+                for s in self._object_graph.successors(v):
+                    sname = self._object_graph.vs["name"][s]
+                    successors.append(hashutil.hash_to_bytehex(sname))
+                    vertices.append(s)
+            logger.debug(f"object {name} depends on {successors}")
 
     def save_data(self) -> None:
         """Store a pack for archival"""