From dac2481de0cd8162519470bf4be3dd0168cbea38 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Thu, 1 Aug 2024 12:28:16 +0200 Subject: [PATCH] Makefile: dont't run "test" as part of default "all" make target The unit tests require CAP_NET_ADMIN permissions, so commonly they fail in a developer environment for valid reasons. Having the "test" target part of the default make target ("all") will fail in such an environment, which is inconvenient. Running unit tests should be a separate make target. To run tests, explicitly run `make test`. Fixes: f2dbb73a9938 ('Makefile refactor') --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 9c803860..2768e279 100644 --- a/Makefile +++ b/Makefile @@ -42,7 +42,7 @@ V ?= 0 Q = $(if $(filter 1,$V),,@) .PHONY: all -all: fmt lint build test +all: fmt lint build $(BINDIR) $(BUILDDIR) $(COVERAGE_DIR): ; $(info Creating directory $@...) @mkdir -p $@