Build: Fix bug in version computation

Author: howetuft

build-system/luxmake/check.py

@@ -11,8 +11,11 @@
 
 from .utils import logger, fail
 
+
 @dataclass
 class Require:
+    """Requirement."""
+
     name: str
     min_version: tuple
     mandatory: bool
@@ -28,6 +31,17 @@ class Require:
 )
 
 
+def _version_tuple(version_str):
+    """Translate version into a tuple of int."""
+    if not version_str:
+        return None
+    if "-" in version_str:
+        version_str, *_ = version_str.rpartition("-")
+    try:
+        return tuple(int(i) for i in version_str.split("."))
+    except ValueError:
+        return None
+
 
 def check(name, min_version=None, mandatory=True):
     """Check whether an app exists and whether its version is correct."""
@@ -43,27 +57,21 @@ def error(*args):
 
     # Get version
     result = subprocess.run(
-        [app, "--version"],
-        capture_output=True,
-        text=True,
+        [app, "--version"], capture_output=True, text=True, check=False
     )
     output = result.stdout.strip() or result.stderr.strip()
     # Match version patterns like 1.2.3, 4.5, 2.0.1-alpha, etc.
-    match = re.search(r'\d+\.\d+(?:\.\d+)?(?:[-.\w]*)?', output)
-    if match:
-        version = match.group(0)
-        if '-' in version:
-            version, *_ = version.rpartition("-")
-        version = tuple(int(i) for i in version.split("."))
-        version_str = ".".join(str(i) for i in version)
+    if match := re.search(r"\d+\.\d+(?:\.\d+)?(?:[-.\w]*)?", output):
+        version_str = match.group(0)
     else:
-        version = None
         version_str = None
 
     # Check Version, if necessary
     if min_version:
         min_version_str = ".".join(str(i) for i in min_version)
-        if not version:
+
+        # Translate version
+        if not (version := _version_tuple(version_str)):
             return error("Cannot read '%s' version", name)
         if version < min_version:
             return error(