summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoel Challis <git@zvecr.com>2021-05-09 11:57:49 +0100
committerGitHub <noreply@github.com>2021-05-09 12:57:49 +0200
commit7725d813c9cf1a47863e325457b13a4542984eda (patch)
tree4b51c7e3b6c10328467207bfe4683c3bcff78526
parent60a39f4f5d192625ea03daf8cd26bc89aa1a91c2 (diff)
Allow MAKE environment override for 'qmk clean' (#12473)
-rw-r--r--lib/python/qmk/cli/clean.py8
-rw-r--r--lib/python/qmk/commands.py34
2 files changed, 30 insertions, 12 deletions
diff --git a/lib/python/qmk/cli/clean.py b/lib/python/qmk/cli/clean.py
index ec6501b760..9096529fde 100644
--- a/lib/python/qmk/cli/clean.py
+++ b/lib/python/qmk/cli/clean.py
@@ -1,16 +1,12 @@
"""Clean the QMK firmware folder of build artifacts.
"""
-from qmk.commands import run
+from qmk.commands import run, create_make_target
from milc import cli
-import shutil
-
@cli.argument('-a', '--all', arg_only=True, action='store_true', help='Remove *.hex and *.bin files in the QMK root as well.')
@cli.subcommand('Clean the QMK firmware folder of build artifacts.')
def clean(cli):
"""Runs `make clean` (or `make distclean` if --all is passed)
"""
- make_cmd = 'gmake' if shutil.which('gmake') else 'make'
-
- run([make_cmd, 'distclean' if cli.args.all else 'clean'])
+ run(create_make_target('distclean' if cli.args.all else 'clean'))
diff --git a/lib/python/qmk/commands.py b/lib/python/qmk/commands.py
index d742f67560..97774001a3 100644
--- a/lib/python/qmk/commands.py
+++ b/lib/python/qmk/commands.py
@@ -29,6 +29,33 @@ def _find_make():
return make_cmd
+def create_make_target(target, parallel=1, **env_vars):
+ """Create a make command
+
+ Args:
+
+ target
+ Usually a make rule, such as 'clean' or 'all'.
+
+ parallel
+ The number of make jobs to run in parallel
+
+ **env_vars
+ Environment variables to be passed to make.
+
+ Returns:
+
+ A command that can be run to make the specified keyboard and keymap
+ """
+ env = []
+ make_cmd = _find_make()
+
+ for key, value in env_vars.items():
+ env.append(f'{key}={value}')
+
+ return [make_cmd, '-j', str(parallel), *env, target]
+
+
def create_make_command(keyboard, keymap, target=None, parallel=1, **env_vars):
"""Create a make compile command
@@ -53,17 +80,12 @@ def create_make_command(keyboard, keymap, target=None, parallel=1, **env_vars):
A command that can be run to make the specified keyboard and keymap
"""
- env = []
make_args = [keyboard, keymap]
- make_cmd = _find_make()
if target:
make_args.append(target)
- for key, value in env_vars.items():
- env.append(f'{key}={value}')
-
- return [make_cmd, '-j', str(parallel), *env, ':'.join(make_args)]
+ return create_make_target(':'.join(make_args), parallel, **env_vars)
def get_git_version(repo_dir='.', check_dir='.'):