Find cycles in recipes

Author: singiamtel

alibuild_helpers/utilities.py

@@ -65,6 +65,24 @@ def topological_sort(specs):
     edges = [(pkg, dep) for pkg, dep in edges if dep != current_package]
     # ...but keep blocking those that still depend on other stuff!
     leaves.extend(new_leaves - {pkg for pkg, _ in edges})
+  # If we have any edges left, we have a cycle
+  if edges:
+    # Find a cycle by following dependencies
+    cycle = []
+    start = edges[0][0]  # Start with any remaining package
+    current = start
+    while True:
+      cycle.append(current)
+      # Find what current depends on
+      for pkg, dep in edges:
+        if pkg == current:
+          current = dep
+          break
+      if current in cycle:  # We found a cycle
+        cycle = cycle[cycle.index(current):]  # Trim to just the cycle
+        dieOnError(True, "Dependency cycle detected: " + " -> ".join(cycle + [cycle[0]]))
+      if current == start:  # We've gone full circle
+        break
 
 
 def resolve_store_path(architecture, spec_hash):

tests/test_utilities.py

@@ -12,6 +12,7 @@
 from alibuild_helpers.utilities import resolve_version
 from alibuild_helpers.utilities import topological_sort
 from alibuild_helpers.utilities import resolveFilename, resolveDefaultsFilename
+import alibuild_helpers
 import os
 import string
 
@@ -344,5 +345,18 @@ def test_dont_drop_packages(self) -> None:
         self.assertEqual(frozenset(specs.keys()),
                          frozenset(topological_sort(specs)))
 
+    def test_cycle(self) -> None:
+        """Test that dependency cycles are detected and reported."""
+        specs = {
+            "A": {"package": "A", "requires": ["B"]},
+            "B": {"package": "B", "requires": ["C"]},
+            "C": {"package": "C", "requires": ["D"]},
+            "D": {"package": "D", "requires": ["A"]}
+        }
+        with patch.object(alibuild_helpers.log, 'error') as mock_error:
+          with self.assertRaises(SystemExit) as cm:
+            list(topological_sort(specs))
+          self.assertEqual(cm.exception.code, 1)
+          mock_error.assert_called_once_with("%s", "Dependency cycle detected: A -> B -> C -> D -> A")
 if __name__ == '__main__':
     unittest.main()