diff options
| author | Nick Brassel <nick@tzarc.org> | 2023-06-21 08:41:49 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-06-21 08:41:49 +1000 |
| commit | ffdc70bf0a6fecff024ece2a5b7b4c0c88dd6ee3 (patch) | |
| tree | 5b4f6b347fd5902c6db24019173e5706a84b9bba /builddefs/build_keyboard.mk | |
| parent | 45dc1ccffedd92c543997cc153efbaf058a46616 (diff) | |
Consolidate `KEYBOARD_OUTPUT`+`KEYMAP_OUTPUT`=>`INTERMEDIATE_OUTPUT` (#21272)
Diffstat (limited to 'builddefs/build_keyboard.mk')
| -rw-r--r-- | builddefs/build_keyboard.mk | 67 |
1 files changed, 31 insertions, 36 deletions
diff --git a/builddefs/build_keyboard.mk b/builddefs/build_keyboard.mk index 0425d3fc9d..32e41e7d5b 100644 --- a/builddefs/build_keyboard.mk +++ b/builddefs/build_keyboard.mk | |||
| @@ -27,7 +27,6 @@ QMK_BIN ?= qmk | |||
| 27 | # Set the filename for the final firmware binary | 27 | # Set the filename for the final firmware binary |
| 28 | KEYBOARD_FILESAFE := $(subst /,_,$(KEYBOARD)) | 28 | KEYBOARD_FILESAFE := $(subst /,_,$(KEYBOARD)) |
| 29 | TARGET ?= $(KEYBOARD_FILESAFE)_$(KEYMAP) | 29 | TARGET ?= $(KEYBOARD_FILESAFE)_$(KEYMAP) |
| 30 | KEYBOARD_OUTPUT := $(BUILD_DIR)/obj_$(KEYBOARD_FILESAFE) | ||
| 31 | 30 | ||
| 32 | ifeq ($(strip $(DUMP_CI_METADATA)),yes) | 31 | ifeq ($(strip $(DUMP_CI_METADATA)),yes) |
| 33 | $(info CI Metadata: KEYBOARD=$(KEYBOARD)) | 32 | $(info CI Metadata: KEYBOARD=$(KEYBOARD)) |
| @@ -44,7 +43,7 @@ endif | |||
| 44 | # Object files and generated keymap directory | 43 | # Object files and generated keymap directory |
| 45 | # To put object files in current directory, use a dot (.), do NOT make | 44 | # To put object files in current directory, use a dot (.), do NOT make |
| 46 | # this an empty or blank macro! | 45 | # this an empty or blank macro! |
| 47 | KEYMAP_OUTPUT := $(BUILD_DIR)/obj_$(TARGET) | 46 | INTERMEDIATE_OUTPUT := $(BUILD_DIR)/obj_$(TARGET) |
| 48 | 47 | ||
| 49 | ifdef SKIP_VERSION | 48 | ifdef SKIP_VERSION |
| 50 | OPT_DEFS += -DSKIP_VERSION | 49 | OPT_DEFS += -DSKIP_VERSION |
| @@ -60,7 +59,7 @@ VERSION_H_FLAGS += --skip-git | |||
| 60 | endif | 59 | endif |
| 61 | 60 | ||
| 62 | # Generate the board's version.h file. | 61 | # Generate the board's version.h file. |
| 63 | $(shell $(QMK_BIN) generate-version-h $(VERSION_H_FLAGS) -q -o $(KEYMAP_OUTPUT)/src/version.h) | 62 | $(shell $(QMK_BIN) generate-version-h $(VERSION_H_FLAGS) -q -o $(INTERMEDIATE_OUTPUT)/src/version.h) |
| 64 | 63 | ||
| 65 | # Determine which subfolders exist. | 64 | # Determine which subfolders exist. |
| 66 | KEYBOARD_FOLDER_PATH_1 := $(KEYBOARD) | 65 | KEYBOARD_FOLDER_PATH_1 := $(KEYBOARD) |
| @@ -121,7 +120,7 @@ MAIN_KEYMAP_PATH_4 := $(KEYBOARD_PATH_4)/keymaps/$(KEYMAP) | |||
| 121 | MAIN_KEYMAP_PATH_5 := $(KEYBOARD_PATH_5)/keymaps/$(KEYMAP) | 120 | MAIN_KEYMAP_PATH_5 := $(KEYBOARD_PATH_5)/keymaps/$(KEYMAP) |
| 122 | 121 | ||
| 123 | # Pull in rules from info.json | 122 | # Pull in rules from info.json |
| 124 | INFO_RULES_MK = $(shell $(QMK_BIN) generate-rules-mk --quiet --escape --keyboard $(KEYBOARD) --output $(KEYBOARD_OUTPUT)/src/info_rules.mk) | 123 | INFO_RULES_MK = $(shell $(QMK_BIN) generate-rules-mk --quiet --escape --keyboard $(KEYBOARD) --output $(INTERMEDIATE_OUTPUT)/src/info_rules.mk) |
| 125 | include $(INFO_RULES_MK) | 124 | include $(INFO_RULES_MK) |
| 126 | 125 | ||
| 127 | # Check for keymap.json first, so we can regenerate keymap.c | 126 | # Check for keymap.json first, so we can regenerate keymap.c |
| @@ -161,28 +160,28 @@ endif | |||
| 161 | 160 | ||
| 162 | # Have we found a keymap.json? | 161 | # Have we found a keymap.json? |
| 163 | ifneq ("$(wildcard $(KEYMAP_JSON))", "") | 162 | ifneq ("$(wildcard $(KEYMAP_JSON))", "") |
| 164 | KEYMAP_C := $(KEYMAP_OUTPUT)/src/keymap.c | 163 | KEYMAP_C := $(INTERMEDIATE_OUTPUT)/src/keymap.c |
| 165 | KEYMAP_H := $(KEYMAP_OUTPUT)/src/config.h | 164 | KEYMAP_H := $(INTERMEDIATE_OUTPUT)/src/config.h |
| 166 | 165 | ||
| 167 | # Load the keymap-level rules.mk if exists | 166 | # Load the keymap-level rules.mk if exists |
| 168 | -include $(KEYMAP_PATH)/rules.mk | 167 | -include $(KEYMAP_PATH)/rules.mk |
| 169 | 168 | ||
| 170 | # Load any rules.mk content from keymap.json | 169 | # Load any rules.mk content from keymap.json |
| 171 | INFO_RULES_MK = $(shell $(QMK_BIN) generate-rules-mk --quiet --escape --output $(KEYMAP_OUTPUT)/src/rules.mk $(KEYMAP_JSON)) | 170 | INFO_RULES_MK = $(shell $(QMK_BIN) generate-rules-mk --quiet --escape --output $(INTERMEDIATE_OUTPUT)/src/rules.mk $(KEYMAP_JSON)) |
| 172 | include $(INFO_RULES_MK) | 171 | include $(INFO_RULES_MK) |
| 173 | 172 | ||
| 174 | # Add rules to generate the keymap files - indentation here is important | 173 | # Add rules to generate the keymap files - indentation here is important |
| 175 | $(KEYMAP_OUTPUT)/src/keymap.c: $(KEYMAP_JSON) | 174 | $(INTERMEDIATE_OUTPUT)/src/keymap.c: $(KEYMAP_JSON) |
| 176 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 175 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 177 | $(eval CMD=$(QMK_BIN) json2c --quiet --output $(KEYMAP_C) $(KEYMAP_JSON)) | 176 | $(eval CMD=$(QMK_BIN) json2c --quiet --output $(KEYMAP_C) $(KEYMAP_JSON)) |
| 178 | @$(BUILD_CMD) | 177 | @$(BUILD_CMD) |
| 179 | 178 | ||
| 180 | $(KEYMAP_OUTPUT)/src/config.h: $(KEYMAP_JSON) | 179 | $(INTERMEDIATE_OUTPUT)/src/config.h: $(KEYMAP_JSON) |
| 181 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 180 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 182 | $(eval CMD=$(QMK_BIN) generate-config-h --quiet --output $(KEYMAP_H) $(KEYMAP_JSON)) | 181 | $(eval CMD=$(QMK_BIN) generate-config-h --quiet --output $(KEYMAP_H) $(KEYMAP_JSON)) |
| 183 | @$(BUILD_CMD) | 182 | @$(BUILD_CMD) |
| 184 | 183 | ||
| 185 | generated-files: $(KEYMAP_OUTPUT)/src/config.h $(KEYMAP_OUTPUT)/src/keymap.c | 184 | generated-files: $(INTERMEDIATE_OUTPUT)/src/config.h $(INTERMEDIATE_OUTPUT)/src/keymap.c |
| 186 | 185 | ||
| 187 | endif | 186 | endif |
| 188 | 187 | ||
| @@ -323,34 +322,34 @@ ifneq ("$(wildcard $(KEYBOARD_PATH_5)/info.json)","") | |||
| 323 | INFO_JSON_FILES += $(KEYBOARD_PATH_5)/info.json | 322 | INFO_JSON_FILES += $(KEYBOARD_PATH_5)/info.json |
| 324 | endif | 323 | endif |
| 325 | 324 | ||
| 326 | CONFIG_H += $(KEYBOARD_OUTPUT)/src/info_config.h | 325 | CONFIG_H += $(INTERMEDIATE_OUTPUT)/src/info_config.h |
| 327 | KEYBOARD_SRC += $(KEYBOARD_OUTPUT)/src/default_keyboard.c | 326 | KEYBOARD_SRC += $(INTERMEDIATE_OUTPUT)/src/default_keyboard.c |
| 328 | 327 | ||
| 329 | $(KEYBOARD_OUTPUT)/src/info_config.h: $(INFO_JSON_FILES) | 328 | $(INTERMEDIATE_OUTPUT)/src/info_config.h: $(INFO_JSON_FILES) |
| 330 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 329 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 331 | $(eval CMD=$(QMK_BIN) generate-config-h --quiet --keyboard $(KEYBOARD) --output $(KEYBOARD_OUTPUT)/src/info_config.h) | 330 | $(eval CMD=$(QMK_BIN) generate-config-h --quiet --keyboard $(KEYBOARD) --output $(INTERMEDIATE_OUTPUT)/src/info_config.h) |
| 332 | @$(BUILD_CMD) | 331 | @$(BUILD_CMD) |
| 333 | 332 | ||
| 334 | $(KEYBOARD_OUTPUT)/src/default_keyboard.c: $(INFO_JSON_FILES) | 333 | $(INTERMEDIATE_OUTPUT)/src/default_keyboard.c: $(INFO_JSON_FILES) |
| 335 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 334 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 336 | $(eval CMD=$(QMK_BIN) generate-keyboard-c --quiet --keyboard $(KEYBOARD) --output $(KEYBOARD_OUTPUT)/src/default_keyboard.c) | 335 | $(eval CMD=$(QMK_BIN) generate-keyboard-c --quiet --keyboard $(KEYBOARD) --output $(INTERMEDIATE_OUTPUT)/src/default_keyboard.c) |
| 337 | @$(BUILD_CMD) | 336 | @$(BUILD_CMD) |
| 338 | 337 | ||
| 339 | $(KEYBOARD_OUTPUT)/src/default_keyboard.h: $(INFO_JSON_FILES) | 338 | $(INTERMEDIATE_OUTPUT)/src/default_keyboard.h: $(INFO_JSON_FILES) |
| 340 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 339 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 341 | $(eval CMD=$(QMK_BIN) generate-keyboard-h --quiet --keyboard $(KEYBOARD) --include $(FOUND_KEYBOARD_H) --output $(KEYBOARD_OUTPUT)/src/default_keyboard.h) | 340 | $(eval CMD=$(QMK_BIN) generate-keyboard-h --quiet --keyboard $(KEYBOARD) --include $(FOUND_KEYBOARD_H) --output $(INTERMEDIATE_OUTPUT)/src/default_keyboard.h) |
| 342 | @$(BUILD_CMD) | 341 | @$(BUILD_CMD) |
| 343 | 342 | ||
| 344 | generated-files: $(KEYBOARD_OUTPUT)/src/info_config.h $(KEYBOARD_OUTPUT)/src/default_keyboard.c $(KEYBOARD_OUTPUT)/src/default_keyboard.h | 343 | generated-files: $(INTERMEDIATE_OUTPUT)/src/info_config.h $(INTERMEDIATE_OUTPUT)/src/default_keyboard.c $(INTERMEDIATE_OUTPUT)/src/default_keyboard.h |
| 345 | 344 | ||
| 346 | generated-files: $(KEYMAP_OUTPUT)/src/info_deps.d | 345 | generated-files: $(INTERMEDIATE_OUTPUT)/src/info_deps.d |
| 347 | 346 | ||
| 348 | $(KEYMAP_OUTPUT)/src/info_deps.d: | 347 | $(INTERMEDIATE_OUTPUT)/src/info_deps.d: |
| 349 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) | 348 | @$(SILENT) || printf "$(MSG_GENERATING) $@" | $(AWK_CMD) |
| 350 | $(eval CMD=$(QMK_BIN) generate-make-dependencies -kb $(KEYBOARD) -km $(KEYMAP) -o $(KEYMAP_OUTPUT)/src/info_deps.d) | 349 | $(eval CMD=$(QMK_BIN) generate-make-dependencies -kb $(KEYBOARD) -km $(KEYMAP) -o $(INTERMEDIATE_OUTPUT)/src/info_deps.d) |
| 351 | @$(BUILD_CMD) | 350 | @$(BUILD_CMD) |
| 352 | 351 | ||
| 353 | -include $(KEYMAP_OUTPUT)/src/info_deps.d | 352 | -include $(INTERMEDIATE_OUTPUT)/src/info_deps.d |
| 354 | 353 | ||
| 355 | .INTERMEDIATE : generated-files | 354 | .INTERMEDIATE : generated-files |
| 356 | 355 | ||
| @@ -423,8 +422,7 @@ VPATH += $(KEYMAP_PATH) | |||
| 423 | VPATH += $(USER_PATH) | 422 | VPATH += $(USER_PATH) |
| 424 | VPATH += $(KEYBOARD_PATHS) | 423 | VPATH += $(KEYBOARD_PATHS) |
| 425 | VPATH += $(COMMON_VPATH) | 424 | VPATH += $(COMMON_VPATH) |
| 426 | VPATH += $(KEYBOARD_OUTPUT)/src | 425 | VPATH += $(INTERMEDIATE_OUTPUT)/src |
| 427 | VPATH += $(KEYMAP_OUTPUT)/src | ||
| 428 | 426 | ||
| 429 | include $(BUILDDEFS_PATH)/common_features.mk | 427 | include $(BUILDDEFS_PATH)/common_features.mk |
| 430 | include $(BUILDDEFS_PATH)/generic_features.mk | 428 | include $(BUILDDEFS_PATH)/generic_features.mk |
| @@ -471,17 +469,14 @@ PROJECT_CONFIG := $(CONFIG_H) | |||
| 471 | CONFIG_H += $(POST_CONFIG_H) | 469 | CONFIG_H += $(POST_CONFIG_H) |
| 472 | ALL_CONFIGS := $(PROJECT_CONFIG) $(CONFIG_H) | 470 | ALL_CONFIGS := $(PROJECT_CONFIG) $(CONFIG_H) |
| 473 | 471 | ||
| 474 | OUTPUTS := $(KEYMAP_OUTPUT) $(KEYBOARD_OUTPUT) | 472 | OUTPUTS := $(INTERMEDIATE_OUTPUT) |
| 475 | $(KEYMAP_OUTPUT)_SRC := $(SRC) | 473 | $(INTERMEDIATE_OUTPUT)_SRC := $(SRC) $(PLATFORM_SRC) |
| 476 | $(KEYMAP_OUTPUT)_DEFS := $(OPT_DEFS) \ | 474 | $(INTERMEDIATE_OUTPUT)_DEFS := $(OPT_DEFS) \ |
| 477 | -DQMK_KEYBOARD=\"$(KEYBOARD)\" -DQMK_KEYBOARD_H=\"$(KEYBOARD_OUTPUT)/src/default_keyboard.h\" \ | 475 | -DQMK_KEYBOARD=\"$(KEYBOARD)\" -DQMK_KEYBOARD_H=\"$(INTERMEDIATE_OUTPUT)/src/default_keyboard.h\" \ |
| 478 | -DQMK_KEYMAP=\"$(KEYMAP)\" -DQMK_KEYMAP_H=\"$(KEYMAP).h\" -DQMK_KEYMAP_CONFIG_H=\"$(KEYMAP_PATH)/config.h\" | 476 | -DQMK_KEYMAP=\"$(KEYMAP)\" -DQMK_KEYMAP_H=\"$(KEYMAP).h\" -DQMK_KEYMAP_CONFIG_H=\"$(KEYMAP_PATH)/config.h\" \ |
| 479 | $(KEYMAP_OUTPUT)_INC := $(VPATH) $(EXTRAINCDIRS) | 477 | $(PROJECT_DEFS) |
| 480 | $(KEYMAP_OUTPUT)_CONFIG := $(CONFIG_H) | 478 | $(INTERMEDIATE_OUTPUT)_INC := $(VPATH) $(EXTRAINCDIRS) $(PROJECT_INC) |
| 481 | $(KEYBOARD_OUTPUT)_SRC := $(PLATFORM_SRC) | 479 | $(INTERMEDIATE_OUTPUT)_CONFIG := $(CONFIG_H) $(PROJECT_CONFIG) |
| 482 | $(KEYBOARD_OUTPUT)_DEFS := $(PROJECT_DEFS) | ||
| 483 | $(KEYBOARD_OUTPUT)_INC := $(PROJECT_INC) | ||
| 484 | $(KEYBOARD_OUTPUT)_CONFIG := $(PROJECT_CONFIG) | ||
| 485 | 480 | ||
| 486 | # Default target. | 481 | # Default target. |
| 487 | all: build check-size | 482 | all: build check-size |
