diff options
| author | Simon Glass <sjg@chromium.org> | 2023-07-19 17:48:59 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2023-07-24 09:34:11 -0600 |
| commit | f6df5edc8d00666621e5330019ef2607684e9d2e (patch) | |
| tree | 731c87ba51351b2a62cfd0364d4eebbfc16c96ac | |
| parent | 75584e1fa7f545c88a67b512e85c3ad59cd921d1 (diff) | |
| download | u-boot-f6df5edc8d00666621e5330019ef2607684e9d2e.zip u-boot-f6df5edc8d00666621e5330019ef2607684e9d2e.tar.gz u-boot-f6df5edc8d00666621e5330019ef2607684e9d2e.tar.bz2 | |
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>
| -rw-r--r-- | tools/buildman/control.py | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/tools/buildman/control.py b/tools/buildman/control.py index a20fa62..b388d76 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, |