diff options
author | Simon Glass <sjg@chromium.org> | 2023-07-19 17:48:32 -0600 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2023-07-24 09:34:10 -0600 |
commit | 309f0f15c9676bf2372e67bb2b9775277bb4bcbe (patch) | |
tree | d0545b409af16266ed07d2a03bfa97eedceadae7 /tools | |
parent | b8be2bd83194222006d2f41bb107790d88fb987b (diff) | |
download | u-boot-309f0f15c9676bf2372e67bb2b9775277bb4bcbe.tar.gz u-boot-309f0f15c9676bf2372e67bb2b9775277bb4bcbe.tar.bz2 u-boot-309f0f15c9676bf2372e67bb2b9775277bb4bcbe.zip |
buildman: Move full-help processing to main
This does not need any of the control features. Move it out of main to
reduce the size of the do_buildman() function.
For Python 3.6 the -H feature will not work, but this does not seem to be
a huge problem, as it dates from 2016.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/buildman/control.py | 11 | ||||
-rwxr-xr-x | tools/buildman/main.py | 9 |
2 files changed, 9 insertions, 11 deletions
diff --git a/tools/buildman/control.py b/tools/buildman/control.py index c4be1ad2f4..2e4ba0b217 100644 --- a/tools/buildman/control.py +++ b/tools/buildman/control.py @@ -8,11 +8,6 @@ This holds the main control logic for buildman, when not running tests. """ import multiprocessing -try: - import importlib.resources -except ImportError: - # for Python 3.6 - import importlib_resources import os import shutil import sys @@ -26,7 +21,6 @@ from patman import gitutil from patman import patchstream from u_boot_pylib import command from u_boot_pylib import terminal -from u_boot_pylib import tools from u_boot_pylib.terminal import tprint TEST_BUILDER = None @@ -177,11 +171,6 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None, # Used so testing can obtain the builder: pylint: disable=W0603 global TEST_BUILDER - if options.full_help: - with importlib.resources.path('buildman', 'README.rst') as readme: - tools.print_full_help(str(readme)) - return 0 - gitutil.setup() col = terminal.Color() diff --git a/tools/buildman/main.py b/tools/buildman/main.py index d9ad836215..9a1d59964e 100755 --- a/tools/buildman/main.py +++ b/tools/buildman/main.py @@ -6,6 +6,11 @@ """See README for more information""" +try: + from importlib.resources import files +except ImportError: + # for Python 3.6 + import importlib_resources import os import sys @@ -19,6 +24,7 @@ from buildman import bsettings from buildman import cmdline from buildman import control from u_boot_pylib import test_util +from u_boot_pylib import tools def run_tests(skip_net_tests, debug, verbose, args): """Run the buildman tests @@ -64,6 +70,9 @@ def run_buildman(): return run_tests(options.skip_net_tests, options.debug, options.verbose, args) + elif options.full_help: + tools.print_full_help(str(files('buildman').joinpath('README.rst'))) + # Build selected commits for selected boards else: bsettings.Setup(options.config_file) |