From 033c7af292730fab4a4b147de0fc5b33460379e1 Mon Sep 17 00:00:00 2001 From: Zach White Date: Sun, 8 Dec 2019 16:16:01 -0800 Subject: [PATCH] Fix compiling json files --- lib/python/qmk/cli/compile.py | 2 +- lib/python/qmk/commands.py | 11 +++-------- 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/lib/python/qmk/cli/compile.py b/lib/python/qmk/cli/compile.py index 234ffb12ca..8e2d0cdbf4 100755 --- a/lib/python/qmk/cli/compile.py +++ b/lib/python/qmk/cli/compile.py @@ -37,7 +37,7 @@ def compile(cli): cli.log.info('Creating {fg_cyan}%s{style_reset_all} keymap in {fg_cyan}%s', user_keymap['keymap'], keymap_path) # Compile the keymap - command = compile_configurator_json(cli.args.filename) + command = compile_configurator_json(user_keymap) cli.log.info('Wrote keymap to {fg_cyan}%s/%s/keymap.c', keymap_path, user_keymap['keymap']) diff --git a/lib/python/qmk/commands.py b/lib/python/qmk/commands.py index f83a89578e..6067d49ae7 100644 --- a/lib/python/qmk/commands.py +++ b/lib/python/qmk/commands.py @@ -25,16 +25,14 @@ def create_make_command(keyboard, keymap, target=None): return ['make', ':'.join((keyboard, keymap, target))] -def parse_configurator_json(configurator_filename): +def parse_configurator_json(configurator_file): """Open and parse a configurator json export """ - file = open(configurator_filename) - user_keymap = json.load(file) - file.close() + user_keymap = json.load(configurator_file) return user_keymap -def compile_configurator_json(configurator_filename, bootloader=None): +def compile_configurator_json(user_keymap, bootloader=None): """Convert a configurator export JSON file into a C file Args: @@ -47,9 +45,6 @@ def compile_configurator_json(configurator_filename, bootloader=None): Returns: A command to run to compile and flash the C file. """ - # Parse the configurator json - user_keymap = parse_configurator_json(configurator_filename) - # Write the keymap C file qmk.keymap.write(user_keymap['keyboard'], user_keymap['keymap'], user_keymap['layout'], user_keymap['layers'])