scripts: west_commands: fix argument handling with runners.yaml
Refactor the code to support the new runners.yaml file created by the
build system.
Compared to fishing around in the CMake cache, this makes it trivial
to put all the command line arguments to a runner-based command on
equal footing, regardless of if they're defined in the runners package
proper or defined in run_common.py.
This allows board.cmake files to do things like this:
board_set_runner_args(foo
--bin-file=${PROJECT_BINARY_DIR}/my-signed.bin)
While at it, make some other cleanups:
- Stop using the obsolete and deprecated west.cmake module while we're
here in favor of the zcmake.py module which was added to Zephyr a long
time ago. Yikes. I had forgotten this was still here.
- Stop using west.util's wrap function in favor of raw use of
textwrap. The west function splits on hyphens, which is breaking
runner names like "em-starterkit".
- Clean up the --context output a bit
Fixes: #22563
Signed-off-by: Martí Bolívar <marti.bolivar@nordicsemi.no>
diff --git a/scripts/west_commands/flash.py b/scripts/west_commands/flash.py
index 26523fe..d758581 100644
--- a/scripts/west_commands/flash.py
+++ b/scripts/west_commands/flash.py
@@ -1,12 +1,11 @@
# Copyright (c) 2018 Open Source Foundries Limited.
# Copyright 2019 Foundries.io
+# Copyright (c) 2020 Nordic Semiconductor ASA
#
# SPDX-License-Identifier: Apache-2.0
'''west "flash" command'''
-from textwrap import dedent
-
from west.commands import WestCommand
from run_common import desc_common, add_parser_common, do_run_common
@@ -19,14 +18,13 @@
'flash',
# Keep this in sync with the string in west-commands.yml.
'flash and run a binary on a board',
- dedent('''
- Connects to the board and reprograms it with a new binary\n\n''') +
+ 'Permanently reprogram a board with a new binary.\n' +
desc_common('flash'),
accepts_unknown_args=True)
+ self.runner_key = 'flash-runner' # in runners.yaml
def do_add_parser(self, parser_adder):
- return add_parser_common(parser_adder, self)
+ return add_parser_common(self, parser_adder)
def do_run(self, my_args, runner_args):
- do_run_common(self, my_args, runner_args,
- 'ZEPHYR_BOARD_FLASH_RUNNER')
+ do_run_common(self, my_args, runner_args)
