]> git.donarmstrong.com Git - qmk_firmware.git/commitdiff
Fix compiling json files
authorZach White <skullydazed@gmail.com>
Mon, 9 Dec 2019 00:16:01 +0000 (16:16 -0800)
committerskullydazed <skullydazed@users.noreply.github.com>
Mon, 9 Dec 2019 00:40:03 +0000 (16:40 -0800)
lib/python/qmk/cli/compile.py
lib/python/qmk/commands.py

index 234ffb12ca9a2bcd04a6da0eccd2a296b9124908..8e2d0cdbf4fb1007628d65a74495d9e7d4d27df7 100755 (executable)
@@ -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'])
 
index f83a89578e2735e6e3d6cc6c79b5cd3aa6818e38..6067d49ae7c8ff7cb9042527dba4037e738d83c9 100644 (file)
@@ -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'])