Insert a few missing "format_returncode" calls.

vdbergh · ppigazzini · commit ea62bc72e1ae · 2025-09-18T18:45:57.000+02:00
Raise worker version to 296 (also server side)
diff --git a/server/fishtest/api.py b/server/fishtest/api.py
@@ -36,7 +36,7 @@
 according to the route/URL mapping defined in `__init__.py`.
 """
 
-WORKER_VERSION = 295
+WORKER_VERSION = 296
 
 
 @exception_view_config(HTTPException)
diff --git a/worker/games.py b/worker/games.py
@@ -464,7 +464,7 @@ def run_single_bench(engine, hash_size, threads, depth, timeout=600):
                 message = f"Bench of {engine.name} timed out after {timeout} seconds."
                 raise RunException(message) from e
             if p.returncode != 0:
-                message = f"Bench run failed with exit code {p.returncode}."
+                message = f"Bench run failed with exit code {format_returncode(p.returncode)}."
                 raise RunException(message)
             for line in stderr_data.splitlines():
                 if "Total time (ms)" in line:
diff --git a/worker/sri.txt b/worker/sri.txt
@@ -1 +1 @@
-{"__version": 295, "updater.py": "eDDBPKA/vrTCadgtEJFdL06vSoiysF0JhiHKdEnjQv3zS4kfdOAqnco/DJpDWbvh", "worker.py": "LFWwPl6NeiFOWQzj5TPiMPuP0wdaS22jLJMa2PLyhl9F8rSemFPNq3Rpk7tKf5uk", "games.py": "sbNzCoZRY4NoJoOYMVevv0JRbfRWYndkukrXe2krhD6h3ktmo2YL/Z5NTJyY96iL"}
+{"__version": 296, "updater.py": "eDDBPKA/vrTCadgtEJFdL06vSoiysF0JhiHKdEnjQv3zS4kfdOAqnco/DJpDWbvh", "worker.py": "zNLe3/BhRGUwIttR+kXMQW8uN1J39NHBi/NATt7QRTSOd2tbjAEomo+zqbV6Y/Zs", "games.py": "bMlVV6EIMp6Cul5LzOHxG+46dgTInfzde11K99WGRDcqBmUM+9pvKkUrG/0ijlRA"}
diff --git a/worker/worker.py b/worker/worker.py
@@ -73,7 +73,7 @@
 
 FASTCHESS_SHA = "66cac47f06dc1a09d3d1865cdbf560a7814f82ea"
 
-WORKER_VERSION = 295
+WORKER_VERSION = 296
 FILE_LIST = ["updater.py", "worker.py", "games.py"]
 HTTP_TIMEOUT = 30.0
 INITIAL_RETRY_TIME = 15.0
@@ -1024,7 +1024,9 @@ def gcc_version():
         print("No g++ or g++ is not executable.")
         return None
     if p.returncode != 0:
-        print(f"g++ version query failed with return code {p.returncode}.")
+        print(
+            f"g++ version query failed with return code {format_returncode(p.returncode)}."
+        )
         return None
 
     try:
@@ -1067,7 +1069,9 @@ def clang_version():
         print("No clang++ or clang++ is not executable.")
         return None
     if p.returncode != 0:
-        print(f"clang++ version query failed with return code {p.returncode}.")
+        print(
+            f"clang++ version query failed with return code {format_returncode(p.returncode)}."
+        )
         return None
     try:
         major = int(clang_major)
@@ -1129,7 +1133,7 @@ def verify_toolchain():
             p = subprocess.run(cmd, stdout=subprocess.DEVNULL, stderr=subprocess.PIPE)
             if p.returncode != 0:
                 print(
-                    f"'{cmd_str}' returned code: {p.returncode} and error: {p.stderr.decode().strip()}"
+                    f"'{cmd_str}' returned code: {format_returncode(p.returncode)} and error: {p.stderr.decode().strip()}"
                 )
                 missing_tools.append(name)
         except (OSError, subprocess.SubprocessError) as e:

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-{"__version": 295, "updater.py": "eDDBPKA/vrTCadgtEJFdL06vSoiysF0JhiHKdEnjQv3zS4kfdOAqnco/DJpDWbvh", "worker.py": "LFWwPl6NeiFOWQzj5TPiMPuP0wdaS22jLJMa2PLyhl9F8rSemFPNq3Rpk7tKf5uk", "games.py": "sbNzCoZRY4NoJoOYMVevv0JRbfRWYndkukrXe2krhD6h3ktmo2YL/Z5NTJyY96iL"}`
	`1`	`+{"__version": 296, "updater.py": "eDDBPKA/vrTCadgtEJFdL06vSoiysF0JhiHKdEnjQv3zS4kfdOAqnco/DJpDWbvh", "worker.py": "zNLe3/BhRGUwIttR+kXMQW8uN1J39NHBi/NATt7QRTSOd2tbjAEomo+zqbV6Y/Zs", "games.py": "bMlVV6EIMp6Cul5LzOHxG+46dgTInfzde11K99WGRDcqBmUM+9pvKkUrG/0ijlRA"}`