From c07543133a092039c7eae0cead6c9ee3badcc7dc Mon Sep 17 00:00:00 2001 From: Zach White Date: Sat, 2 Jan 2021 09:27:35 -0800 Subject: [PATCH] Return the make exit code for qmk compile and flash (#11402) --- lib/python/qmk/cli/compile.py | 3 ++- lib/python/qmk/cli/flash.py | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/python/qmk/cli/compile.py b/lib/python/qmk/cli/compile.py index 341f365f8..daee597d8 100755 --- a/lib/python/qmk/cli/compile.py +++ b/lib/python/qmk/cli/compile.py @@ -48,7 +48,8 @@ def compile(cli): cli.log.info('Compiling keymap with {fg_cyan}%s', ' '.join(command)) if not cli.args.dry_run: cli.echo('\n') - subprocess.run(command) + compile = subprocess.run(command) + return compile.returncode else: cli.log.error('You must supply a configurator export, both `--keyboard` and `--keymap`, or be in a directory for a keyboard or keymap.') diff --git a/lib/python/qmk/cli/flash.py b/lib/python/qmk/cli/flash.py index cefb9ca31..d720d42e7 100644 --- a/lib/python/qmk/cli/flash.py +++ b/lib/python/qmk/cli/flash.py @@ -81,7 +81,8 @@ def flash(cli): cli.log.info('Compiling keymap with {fg_cyan}%s', ' '.join(command)) if not cli.args.dry_run: cli.echo('\n') - subprocess.run(command) + compile = subprocess.run(command) + return compile.returncode else: cli.log.error('You must supply a configurator export, both `--keyboard` and `--keymap`, or be in a directory for a keyboard or keymap.')