summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnout Vandecappelle <arnout@mind.be>2017-06-30 19:00:33 +0200
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>2017-07-01 10:10:22 +0200
commit503439f99a8b1a7f762ed184082b0c97800ffae5 (patch)
treeb42caaf88f08a23406afb7c5b6911cd7d5cf471f
parent10ce9c6f8d3567c2e423907d3c5b008a97c9cdfa (diff)
downloadbuildroot-503439f99a8b1a7f762ed184082b0c97800ffae5.tar.gz
buildroot-503439f99a8b1a7f762ed184082b0c97800ffae5.zip
Makefile: unconfigured "make toolchain" should run report error
As reported by Alessandro Power on StackOverflow [1], the behaviour of "make toolchain" in an unconfigured tree is misleading. When .config doesn't exist, we don't read in the package .mk files, so "make <package>" doesn't work: $ make busybox make: *** No rule to make target 'busybox'. Stop. However, for "linux" and "toolchain", the corresponding file (or actually directory) already exists. So instead, we get: $ make linux make: Nothing to be done for 'linux'. This is confusing, because it looks as if the build succeeded. The obvious solution is to make linux and toolchain PHONY targets when .config doesn't exist. However, that actually does the reverse, because then a rule _does_ exist for them and since they don't have dependencies, make will consider them to be ready. Therefore, we also have to provide an explicit rule for them, and explicitly error out. Thise behaviour is still different from other packages, but at least it is much less confusing. [1] https://stackoverflow.com/questions/44521150 Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Cc: Luca Ceresoli <luca@lucaceresoli.net> Tested-by: Luca Ceresoli <luca@lucaceresoli.net> Acked-by: Luca Ceresoli <luca@lucaceresoli.net> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-rw-r--r--Makefile8
1 files changed, 8 insertions, 0 deletions
diff --git a/Makefile b/Makefile
index 88d98e0405..470aa837ce 100644
--- a/Makefile
+++ b/Makefile
@@ -813,6 +813,14 @@ else # ifeq ($(BR2_HAVE_DOT_CONFIG),y)
all: menuconfig
+# Some subdirectories are also package names. To avoid that "make linux"
+# on an unconfigured tree produces "Nothing to be done", add an explicit
+# rule for it.
+.PHONY: linux toolchain
+linux toolchain:
+ $(error Please configure Buildroot first (e.g. "make menuconfig"))
+ @exit 1
+
endif # ifeq ($(BR2_HAVE_DOT_CONFIG),y)
# configuration
OpenPOWER on IntegriCloud