diff options
author | QMK Bot <hello@qmk.fm> | 2021-05-09 10:58:43 +0000 |
---|---|---|
committer | QMK Bot <hello@qmk.fm> | 2021-05-09 10:58:43 +0000 |
commit | a0c95cb07e3b9874a7a92669ea2695825a436cfd (patch) | |
tree | e97ecbc6f37431de497bec6552623adf421fe66d | |
parent | 7a25dcacffcadf541da5107a35856b66e770bcaf (diff) | |
parent | 7725d813c9cf1a47863e325457b13a4542984eda (diff) | |
download | qmk_firmware-a0c95cb07e3b9874a7a92669ea2695825a436cfd.tar.gz qmk_firmware-a0c95cb07e3b9874a7a92669ea2695825a436cfd.zip |
Merge remote-tracking branch 'origin/master' into develop
-rw-r--r-- | lib/python/qmk/cli/clean.py | 8 | ||||
-rw-r--r-- | lib/python/qmk/commands.py | 34 |
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 510abffdce..ffd1502f70 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='.'): |