From 83b896ffafe1b6e3be5b666b658346bffa6ac062 Mon Sep 17 00:00:00 2001 From: Alexa <53471801+Alexa-RR@users.noreply.github.com> Date: Sat, 24 Oct 2026 20:30:40 +0300 Subject: [PATCH] fix: drop an unused import and make the build scripts' exits explicit --- DorkNet.Quest/scripts/build.py | 6 +++--- tools/build_pc.py | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/DorkNet.Quest/scripts/build.py b/DorkNet.Quest/scripts/build.py index 6dc2021..ef29c0e 100644 --- a/DorkNet.Quest/scripts/build.py +++ b/DorkNet.Quest/scripts/build.py @@ -32,7 +32,6 @@ import argparse import hashlib import json import os -import platform import re import shutil import subprocess @@ -40,6 +39,7 @@ import sys import tempfile import zipfile from pathlib import Path +from typing import NoReturn from xml.sax.saxutils import escape as xml_escape SCRIPTS = Path(__file__).resolve().parent @@ -53,7 +53,7 @@ def step(msg: str) -> None: print(f"==> {msg}", flush=True) -def fail(msg: str) -> None: +def fail(msg: str) -> NoReturn: raise SystemExit(f"error: {msg}") @@ -143,7 +143,7 @@ def resolve_apktool(arg: str | None) -> Path: root = sdk_root() if root and (root / "apktool.jar").exists(): return root / "apktool.jar" - fail("apktool.jar not found. Pass --apktool-jar or set APKTOOL_JAR.") + raise SystemExit("error: apktool.jar not found. Pass --apktool-jar or set APKTOOL_JAR.") def cfg_value(config: Path, key: str) -> str | None: diff --git a/tools/build_pc.py b/tools/build_pc.py index 7615b55..49d3e5e 100644 --- a/tools/build_pc.py +++ b/tools/build_pc.py @@ -28,6 +28,7 @@ import tempfile import urllib.request import zipfile from pathlib import Path +from typing import NoReturn REPO = Path(__file__).resolve().parent.parent CLIENT_MOD = REPO / "DorkNet.ClientMod" @@ -50,7 +51,7 @@ def step(msg: str) -> None: print(f"==> {msg}", flush=True) -def fail(msg: str) -> None: +def fail(msg: str) -> NoReturn: raise SystemExit(f"error: {msg}")