diff options
| author | Joel Challis <git@zvecr.com> | 2023-01-19 00:24:13 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-19 00:24:13 +0000 |
| commit | a1f253cbef309906c989cfb3e123ee8fb3172bb4 (patch) | |
| tree | 5e9094d4aeea4b4134f5e8f770da2f887e82eb0e /lib/python/qmk/cli/flash.py | |
| parent | baecc69da5c6fc9961392f77e5a7d8d2e095b69a (diff) | |
`qmk compile`/`qmk flash` - Validate keymap argument (#19530)
Diffstat (limited to 'lib/python/qmk/cli/flash.py')
| -rw-r--r-- | lib/python/qmk/cli/flash.py | 17 |
1 files changed, 16 insertions, 1 deletions
diff --git a/lib/python/qmk/cli/flash.py b/lib/python/qmk/cli/flash.py index 52defb5f0d..8724f26889 100644 --- a/lib/python/qmk/cli/flash.py +++ b/lib/python/qmk/cli/flash.py | |||
| @@ -11,10 +11,20 @@ import qmk.path | |||
| 11 | from qmk.decorators import automagic_keyboard, automagic_keymap | 11 | from qmk.decorators import automagic_keyboard, automagic_keymap |
| 12 | from qmk.commands import compile_configurator_json, create_make_command, parse_configurator_json, build_environment | 12 | from qmk.commands import compile_configurator_json, create_make_command, parse_configurator_json, build_environment |
| 13 | from qmk.keyboard import keyboard_completer, keyboard_folder | 13 | from qmk.keyboard import keyboard_completer, keyboard_folder |
| 14 | from qmk.keymap import keymap_completer | 14 | from qmk.keymap import keymap_completer, locate_keymap |
| 15 | from qmk.flashers import flasher | 15 | from qmk.flashers import flasher |
| 16 | 16 | ||
| 17 | 17 | ||
| 18 | def _is_keymap_target(keyboard, keymap): | ||
| 19 | if keymap == 'all': | ||
| 20 | return True | ||
| 21 | |||
| 22 | if locate_keymap(keyboard, keymap): | ||
| 23 | return True | ||
| 24 | |||
| 25 | return False | ||
| 26 | |||
| 27 | |||
| 18 | def _list_bootloaders(): | 28 | def _list_bootloaders(): |
| 19 | """Prints the available bootloaders listed in docs.qmk.fm. | 29 | """Prints the available bootloaders listed in docs.qmk.fm. |
| 20 | """ | 30 | """ |
| @@ -98,6 +108,11 @@ def flash(cli): | |||
| 98 | 108 | ||
| 99 | elif cli.config.flash.keyboard and cli.config.flash.keymap: | 109 | elif cli.config.flash.keyboard and cli.config.flash.keymap: |
| 100 | # Generate the make command for a specific keyboard/keymap. | 110 | # Generate the make command for a specific keyboard/keymap. |
| 111 | if not _is_keymap_target(cli.config.flash.keyboard, cli.config.flash.keymap): | ||
| 112 | cli.log.error('Invalid keymap argument.') | ||
| 113 | cli.print_help() | ||
| 114 | return False | ||
| 115 | |||
| 101 | if cli.args.clean: | 116 | if cli.args.clean: |
| 102 | commands.append(create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, 'clean', **envs)) | 117 | commands.append(create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, 'clean', **envs)) |
| 103 | commands.append(create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, cli.args.bootloader, parallel=cli.config.flash.parallel, **envs)) | 118 | commands.append(create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, cli.args.bootloader, parallel=cli.config.flash.parallel, **envs)) |
