From f6df5edc8d00666621e5330019ef2607684e9d2e Mon Sep 17 00:00:00 2001
From: Simon Glass <sjg@chromium.org>
Date: Wed, 19 Jul 2023 17:48:59 -0600
Subject: [PATCH] buildman: Use get_alow_missing() directly to avoid var
Avoid an unnecessary local variable by moving this code to a function.
This fixes the pylint warning about too many local variables.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
tools/buildman/control.py | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index a20fa6286f..b388d760ba 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -603,10 +603,6 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
options.verbose)
return 0
- allow_missing = get_allow_missing(options.allow_missing,
- options.no_allow_missing, len(selected),
- options.branch)
-
# Create a new builder with the selected options
builder = Builder(toolchains, output_dir, git_dir,
options.threads, options.jobs, checkout=True,
@@ -622,7 +618,10 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
test_thread_exceptions=test_thread_exceptions,
adjust_cfg=calc_adjust_cfg(options.adjust_cfg,
options.reproducible_builds),
- allow_missing=allow_missing, no_lto=options.no_lto,
+ allow_missing=get_allow_missing(options.allow_missing,
+ options.no_allow_missing,
+ len(selected), options.branch),
+ no_lto=options.no_lto,
reproducible_builds=options.reproducible_builds,
force_build = options.force_build,
force_build_failures = options.force_build_failures,
--
2.39.5