From 6cf251b5179f2cd5157cc314b60ef461d863eca1 Mon Sep 17 00:00:00 2001 From: Fumitoshi Ukai Date: Mon, 17 Feb 2025 16:22:33 -0800 Subject: [PATCH] autoninja: fix UnboundLocalError Change-Id: Iff96b697b3c1f1d88c13d099ffec4c4159e002bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6276006 Auto-Submit: Fumitoshi Ukai Commit-Queue: Fumitoshi Ukai Reviewed-by: Matt Stark --- autoninja.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/autoninja.py b/autoninja.py index abd79a91e..6e9b65823 100755 --- a/autoninja.py +++ b/autoninja.py @@ -235,12 +235,12 @@ def _main_inner(input_args, build_id, should_collect_logs=False): use_remoteexec = False use_reclient = _get_use_reclient_value(output_dir) use_android_build_server = False + use_siso = None # Attempt to auto-detect remote build acceleration. We support gn-based # builds, where we look for args.gn in the build tree, and cmake-based # builds where we look for rules.ninja. if gn_helper.exists(output_dir): - use_siso = None for k, v in gn_helper.args(output_dir): # use_remoteexec will activate build acceleration. # @@ -514,6 +514,7 @@ def main(args): # splitting ourselves. This means that arguments containing actual spaces # are not supported by autoninja, but that is not a real limitation. input_args = args + exit_code = 127 if sys.platform.startswith("win") and len(args) == 2: input_args = args[:1] + args[1].split() try: