From 7f108ae997bd3ffd8a129a5553d8e8f216dfba69 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Thu, 5 Apr 2018 11:14:24 +0200 Subject: [PATCH] Makefile: Remove `deps` from scripts' dependencies This is useless because `app` already depends on `deps`. Furthermore, `deps` is a phony target, and thus always considered outdated. As a consequence, the scripts' recipe need to always be executed again. Therefore, adding `deps` again as a dependency just increases the time to build for nothing. --- deps/rabbitmq_cli/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deps/rabbitmq_cli/Makefile b/deps/rabbitmq_cli/Makefile index 5a1463e9c3..e2d8728a99 100644 --- a/deps/rabbitmq_cli/Makefile +++ b/deps/rabbitmq_cli/Makefile @@ -81,7 +81,7 @@ rabbitmqctl_srcs := mix.exs \ # it's missing. Another way to do it is to use `mix local.hex` but it # can't be integrated in an alias and doing it from the Makefile isn't # practical. -$(ACTUAL_ESCRIPTS): $(rabbitmqctl_srcs) deps +$(ACTUAL_ESCRIPTS): $(rabbitmqctl_srcs) $(gen_verbose) echo y | mix make_all $(LINKED_ESCRIPTS):