diff options
Diffstat (limited to 'lib/python/qmk/cli/flash.py')
-rw-r--r-- | lib/python/qmk/cli/flash.py | 26 |
1 files changed, 21 insertions, 5 deletions
diff --git a/lib/python/qmk/cli/flash.py b/lib/python/qmk/cli/flash.py index d720d42e7..b3827e800 100644 --- a/lib/python/qmk/cli/flash.py +++ b/lib/python/qmk/cli/flash.py | |||
@@ -3,7 +3,6 @@ | |||
3 | You can compile a keymap already in the repo or using a QMK Configurator export. | 3 | You can compile a keymap already in the repo or using a QMK Configurator export. |
4 | A bootloader must be specified. | 4 | A bootloader must be specified. |
5 | """ | 5 | """ |
6 | import subprocess | ||
7 | from argparse import FileType | 6 | from argparse import FileType |
8 | 7 | ||
9 | from milc import cli | 8 | from milc import cli |
@@ -37,6 +36,9 @@ def print_bootloader_help(): | |||
37 | @cli.argument('-km', '--keymap', help='The keymap to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') | 36 | @cli.argument('-km', '--keymap', help='The keymap to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') |
38 | @cli.argument('-kb', '--keyboard', help='The keyboard to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') | 37 | @cli.argument('-kb', '--keyboard', help='The keyboard to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') |
39 | @cli.argument('-n', '--dry-run', arg_only=True, action='store_true', help="Don't actually build, just show the make command to be run.") | 38 | @cli.argument('-n', '--dry-run', arg_only=True, action='store_true', help="Don't actually build, just show the make command to be run.") |
39 | @cli.argument('-j', '--parallel', type=int, default=1, help="Set the number of parallel make jobs to run.") | ||
40 | @cli.argument('-e', '--env', arg_only=True, action='append', default=[], help="Set a variable to be passed to make. May be passed multiple times.") | ||
41 | @cli.argument('-c', '--clean', arg_only=True, action='store_true', help="Remove object files before compiling.") | ||
40 | @cli.subcommand('QMK Flash.') | 42 | @cli.subcommand('QMK Flash.') |
41 | @automagic_keyboard | 43 | @automagic_keyboard |
42 | @automagic_keymap | 44 | @automagic_keymap |
@@ -50,6 +52,20 @@ def flash(cli): | |||
50 | 52 | ||
51 | If bootloader is omitted the make system will use the configured bootloader for that keyboard. | 53 | If bootloader is omitted the make system will use the configured bootloader for that keyboard. |
52 | """ | 54 | """ |
55 | if cli.args.clean and not cli.args.filename and not cli.args.dry_run: | ||
56 | command = create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, 'clean') | ||
57 | cli.run(command, capture_output=False) | ||
58 | |||
59 | # Build the environment vars | ||
60 | envs = {} | ||
61 | for env in cli.args.env: | ||
62 | if '=' in env: | ||
63 | key, value = env.split('=', 1) | ||
64 | envs[key] = value | ||
65 | else: | ||
66 | cli.log.warning('Invalid environment variable: %s', env) | ||
67 | |||
68 | # Determine the compile command | ||
53 | command = '' | 69 | command = '' |
54 | 70 | ||
55 | if cli.args.bootloaders: | 71 | if cli.args.bootloaders: |
@@ -60,16 +76,16 @@ def flash(cli): | |||
60 | 76 | ||
61 | if cli.args.filename: | 77 | if cli.args.filename: |
62 | # Handle compiling a configurator JSON | 78 | # Handle compiling a configurator JSON |
63 | user_keymap = parse_configurator_json(cli.args.filename) | 79 | user_keymap = parse_configurator_json(cli.args.filename, parallel=cli.config.flash.parallel) |
64 | keymap_path = qmk.path.keymap(user_keymap['keyboard']) | 80 | keymap_path = qmk.path.keymap(user_keymap['keyboard']) |
65 | command = compile_configurator_json(user_keymap, cli.args.bootloader) | 81 | command = compile_configurator_json(user_keymap, cli.args.bootloader, **envs) |
66 | 82 | ||
67 | cli.log.info('Wrote keymap to {fg_cyan}%s/%s/keymap.c', keymap_path, user_keymap['keymap']) | 83 | cli.log.info('Wrote keymap to {fg_cyan}%s/%s/keymap.c', keymap_path, user_keymap['keymap']) |
68 | 84 | ||
69 | else: | 85 | else: |
70 | if cli.config.flash.keyboard and cli.config.flash.keymap: | 86 | if cli.config.flash.keyboard and cli.config.flash.keymap: |
71 | # Generate the make command for a specific keyboard/keymap. | 87 | # Generate the make command for a specific keyboard/keymap. |
72 | command = create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, cli.args.bootloader) | 88 | command = create_make_command(cli.config.flash.keyboard, cli.config.flash.keymap, cli.args.bootloader, parallel=cli.config.flash.parallel, **envs) |
73 | 89 | ||
74 | elif not cli.config.flash.keyboard: | 90 | elif not cli.config.flash.keyboard: |
75 | cli.log.error('Could not determine keyboard!') | 91 | cli.log.error('Could not determine keyboard!') |
@@ -81,7 +97,7 @@ def flash(cli): | |||
81 | cli.log.info('Compiling keymap with {fg_cyan}%s', ' '.join(command)) | 97 | cli.log.info('Compiling keymap with {fg_cyan}%s', ' '.join(command)) |
82 | if not cli.args.dry_run: | 98 | if not cli.args.dry_run: |
83 | cli.echo('\n') | 99 | cli.echo('\n') |
84 | compile = subprocess.run(command) | 100 | compile = cli.run(command, capture_output=False, text=True) |
85 | return compile.returncode | 101 | return compile.returncode |
86 | 102 | ||
87 | else: | 103 | else: |