js.mk: Don't minify without minifier

JS_MINIFY_FILTER_IN can be defined to nothing, in which case
minifying breaks, so don't minify if there's no filter. As an
additional benifit, defining it to the empty string in local.mk
allows to use Vim's quickfix window for syntax errors, because
there's no intermediate file created.

Signed-off-by: Jan Lindemann <jan@janware.com>
This commit is contained in:
Jan Lindemann 2025-12-11 17:23:17 +01:00
commit fc017b624b

View file

@ -23,8 +23,12 @@ all: $(JS_GENERATED)
clean: minify.clean clean: minify.clean
%.min.js: %.js %.min.js: %.js
ifneq ($(JS_MINIFY_FILTER_IN),)
cat $< | $(JS_MINIFY_FILTER_IN) > $@.filtered cat $< | $(JS_MINIFY_FILTER_IN) > $@.filtered
$(JS_MINIFY) $@.filtered > $@.tmp $(JS_MINIFY) $@.filtered > $@.tmp
else
$(JS_MINIFY) $< > $@.tmp
endif
mv $@.tmp $@ mv $@.tmp $@
minify.clean: minify.clean: