Skip to content

Commit 1fd6e5e

Browse files
committed
Move artifact target under emcc build condition
Normal 'make' or 'make all' does not require downloading prebuilt ELF executables from the rv32emu-prebuilt repo. However, 'deps_emcc' always depends on the 'artifact' target, even when not using emcc, causing 'make' or 'make all' to always download prebuilt ELF executables. To prevent this, the 'artifact' target is moved under the emcc build condition. Related: #491
1 parent 563af8b commit 1fd6e5e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

mk/wasm.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
CFLAGS_emcc ?=
2-
deps_emcc := artifact
2+
deps_emcc :=
33
ASSETS := assets
44
WEB_HTML_RESOURCES := $(ASSETS)/html
55
WEB_JS_RESOURCES := $(ASSETS)/js
@@ -42,7 +42,7 @@ $(OUT)/elf_list.js: tools/gen-elf-list-js.py
4242
$(Q)tools/gen-elf-list-js.py > $@
4343

4444
# used to download all dependencies of elf executable and bundle into single wasm
45-
deps_emcc += $(OUT)/elf_list.js $(DOOM_DATA) $(QUAKE_DATA) $(TIMIDITY_DATA)
45+
deps_emcc += artifact $(OUT)/elf_list.js $(DOOM_DATA) $(QUAKE_DATA) $(TIMIDITY_DATA)
4646

4747
# check browser MAJOR version if supports TCO
4848
CHROME_MAJOR :=

0 commit comments

Comments
 (0)