convert: Fix semantic version comparison.

Author: heinezen

openage/game/main.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2024 the openage authors. See copying.md for legal info.
+# Copyright 2015-2025 the openage authors. See copying.md for legal info.
 #
 # pylint: disable=too-many-locals
 
@@ -93,6 +93,7 @@ def main(args, error):
     # ensure that the openage API is present
     if api_export_required(asset_path):
         # export to assets folder
+        info("Updating outdated nyan API modpack")
         converted_path = asset_path / "converted"
         converted_path.mkdirs()
         export_api(converted_path)

openage/main/main.py

@@ -1,4 +1,4 @@
-# Copyright 2015-2024 the openage authors. See copying.md for legal info.
+# Copyright 2015-2025 the openage authors. See copying.md for legal info.
 
 """
 Main engine entry point for openage.
@@ -88,6 +88,7 @@ def main(args, error):
     # ensure that the openage API is present
     if api_export_required(asset_path):
         # export to assets folder
+        info("Updating outdated nyan API modpack")
         converted_path = asset_path / "converted"
         converted_path.mkdirs()
         export_api(converted_path)

openage/util/version.py

@@ -1,4 +1,4 @@
-# Copyright 2024-2024 the openage authors. See copying.md for legal info.
+# Copyright 2024-2025 the openage authors. See copying.md for legal info.
 
 """
 Handling of version information for openage.
@@ -36,26 +36,30 @@ def __init__(self, version: str) -> None:
         self.buildmetadata = match.group("buildmetadata")
 
     def __lt__(self, other: SemanticVersion) -> bool:
-        if self.major < other.major:
-            return True
-        if self.minor < other.minor:
-            return True
-        if self.patch < other.patch:
-            return True
+        if self.major != other.major:
+            return self.major < other.major
 
+        if self.minor != other.minor:
+            return self.minor < other.minor
+
+        if self.patch != other.patch:
+            return self.patch < other.patch
+
+        # else: versions are equal
         return False
 
     def __le__(self, other: SemanticVersion) -> bool:
-        if self.major <= other.major:
-            return True
+        if self.major != other.major:
+            return self.major < other.major
 
-        if self.minor <= other.minor:
-            return True
+        if self.minor != other.minor:
+            return self.minor < other.minor
 
-        if self.patch <= other.patch:
-            return True
+        if self.patch != other.patch:
+            return self.patch < other.patch
 
-        return False
+        # else: versions are equal
+        return True
 
     def __eq__(self, other: SemanticVersion) -> bool:
         return (self.major == other.major and