From 382c662c825416f875c082154e63a35532dab2e3 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Wed, 17 Jul 2024 16:56:57 +0100 Subject: qconfig: Move arg checking to the top of main() Check for 'test' as one of the possible operations for this tool, moving the check above the implementation. Signed-off-by: Simon Glass --- tools/qconfig.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/tools/qconfig.py b/tools/qconfig.py index 6d263ce30db..9052aedf57d 100755 --- a/tools/qconfig.py +++ b/tools/qconfig.py @@ -1524,6 +1524,10 @@ doc/develop/moveconfig.rst for documentation.''' def main(): """Main program""" parser, args = parse_args() + if not any((args.force_sync, args.build_db, args.imply, args.find, + args.scan_source, args.test)): + parser.print_usage() + sys.exit(1) if args.test: sys.argv = [sys.argv[0]] fail, _ = doctest.testmod() @@ -1534,11 +1538,6 @@ def main(): col = terminal.Color(terminal.COLOR_NEVER if args.nocolour else terminal.COLOR_IF_TERMINAL) - if not any((args.force_sync, args.build_db, args.imply, args.find, - args.scan_source)): - parser.print_usage() - sys.exit(1) - if args.scan_source: do_scan_source(os.getcwd(), args.update) return 0 -- cgit v1.2.3