Clean up variables needed 'make flash' target to not get pre-calculated

for all targets
pull/1144/head
Jesse Vincent 3 years ago
parent 2a1fb9014d
commit d1911c8395
No known key found for this signature in database
GPG Key ID: 122F5DF7108E4046

@ -99,20 +99,6 @@ endif
# Flashing related config
ifneq ($(FQBN),)
ifeq ($(KALEIDOSCOPE_DEVICE_PORT),)
KALEIDOSCOPE_DEVICE_PORT := $(shell $(ARDUINO_CLI) board list --format=text | grep $(FQBN) |cut -d' ' -f 1)
endif
endif
flashing_instructions := $(call _arduino_prop,build.flashing_instructions)
ifeq ($(flashing_instructions),)
flashing_instructions := "If your keyboard needs you to do something to put it in flashing mode, do that now."
endif
unescaped_flashing_instructions := $(shell printf $(flashing_instructions) )
DEFAULT_GOAL: compile DEFAULT_GOAL: compile
@ -137,7 +123,6 @@ size-map: ${ELF_FILE_PATH}
$(call _arduino_prop,compiler.size-map.flags) \ $(call _arduino_prop,compiler.size-map.flags) \
"${ELF_FILE_PATH}" "${ELF_FILE_PATH}"
flash: ${HEX_FILE_PATH}
${ELF_FILE_PATH}: compile ${ELF_FILE_PATH}: compile
${HEX_FILE_PATH}: compile ${HEX_FILE_PATH}: compile
@ -203,32 +188,19 @@ endif
#TODO (arduino team) I'd love to do this with their json output #TODO (arduino team) I'd love to do this with their json output
#but it's short some of the data we kind of need #but it's short some of the data we kind of need
.PHONY: ensure-device-port-defined flashing_instructions = $(call _arduino_prop,build.flashing_instructions)
ensure-device-port-defined: kaleidoscope-hardware-configured flash: ${HEX_FILE_PATH}
@if [ -z $(KALEIDOSCOPE_DEVICE_PORT) ]; then \ ifneq ($(flashing_instructions),)
echo "ERROR: Unable to detect keyboard serial port.";\ $(info $(shell printf $(flashing_instructions)))
echo ;\ else
echo "Arduino should autodetect it, but you could also set";\ $(info If your keyboard needs you to do something to put it in flashing mode, do that now.)
echo "KALEIDOSCOPE_DEVICE_PORT to your keyboard's serial port.";\
echo ;\
exit -1;fi
ifneq ($(FQBN),)
fqbn_arg = --fqbn $(FQBN)
endif
ifneq ($(KALEIDOSCOPE_DEVICE_PORT),)
port_arg = --port $(KALEIDOSCOPE_DEVICE_PORT)
endif endif
flash: compile
$(info $(unescaped_flashing_instructions))
$(info ) $(info )
$(info When you're ready to proceed, press 'Enter'.) $(info When you're ready to proceed, press 'Enter'.)
$(info ) $(info )
@$(shell read _) @$(shell read _)
$(QUIET) $(ARDUINO_CLI) upload $(fqbn_arg) \ $(QUIET) $(ARDUINO_CLI) upload --fqbn $(FQBN) \
$(shell $(ARDUINO_CLI) board list --format=text | grep $(FQBN) |cut -d' ' -f 1 | xargs -n 1 echo "--port" ) \
--input-dir "${OUTPUT_PATH}" \ --input-dir "${OUTPUT_PATH}" \
$(port_arg) $(ARDUINO_VERBOSE) $(ARDUINO_VERBOSE)

Loading…
Cancel
Save