main.mk: bail out if no image type was specified

e.g., `make distro/icewm' instead of `make distro/icewm.iso' would be "successful" as there's a corresponding target indeed but the "success" would just mean building the configuration without running the actual build. Thanks cas@ for hitting this issue and reporting it. PS: note that the initial flagless implementation turned out to produce false positives for e.g. `make distclean'.

main.mk: bail out if no image type was specified
f1479275 · Michael Shigorin · 3cb4d9ae · f1479275
Commit f1479275 authored Apr 02, 2015 by Michael Shigorin
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

main.mk main.mk +6 -0

No files found.
--- a/main.mk
+++ b/main.mk
@@ -10,12 +10,18 @@
 IMAGE_TARGET := $(firstword $(MAKECMDGOALS))#	ve/generic.tar.gz
 ifeq (./,$(dir $(IMAGE_TARGET)))#		convenience fallback
 IMAGE_TARGET := distro/$(IMAGE_TARGET)#		for omitted "distro/"
+IMAGE_GUESS  := 1
+else
+IMAGE_GUESS  :=
 endif
 IMAGE_CONF    := $(firstword $(subst ., ,$(IMAGE_TARGET)))# ve/generic
 IMAGE_CLASS   := $(firstword $(subst /, ,$(IMAGE_TARGET)))# ve
 IMAGE_FILE    := $(lastword  $(subst /, ,$(IMAGE_TARGET)))# generic.tar.gz
 IMAGE_NAME    := $(firstword $(subst ., ,$(IMAGE_FILE)))#   generic
 IMAGE_TYPE    := $(subst $(IMAGE_NAME).,,$(IMAGE_FILE))#    tar.gz
+ifeq ($(IMAGE_NAME),$(IMAGE_TYPE)$(IMAGE_GUESS))
+$(warning no image type/extension specified, run make help?)
+endif

 # readjustable
 ifeq (1,$(NUM_TARGETS))