Remove some make dependencies

- Remove docker build from default build path
- Full docker build is still done by CI
- Remove build-dep on test and linter to give the developer the
freedom to just the stuff they want (make all, or just make,
still performs make build test)

Signed-off-by: Marcus brandenburger <[email protected]>

Author: mbrandenburger

.github/workflows/test.yml

@@ -31,7 +31,7 @@ jobs:
           DOCKER_QUIET_BUILD: 1
         run: |
             DOCKER_BUILD_OPTS="--build-arg UBUNTU_VERSION=${{ matrix.os-version }} --build-arg UBUNTU_NAME=${{ matrix.os-name }}" \
-            make -C utils/docker run DOCKER_DEV_OPTIONAL_CMD='env IS_CI_RUNNING=true \
+            make -C utils/docker build run DOCKER_DEV_OPTIONAL_CMD='env IS_CI_RUNNING=true \
                                                                   DOCKER_QUIET_BUILD=1 \
                                                                   DOCKER_BUILDKIT=0 \
                                                                   DOCKER_BUILD_OPTS="--build-arg UBUNTU_VERSION=${{ matrix.os-version }} --build-arg UBUNTU_NAME=${{ matrix.os-name }}" \

Makefile

@@ -27,7 +27,7 @@ license:
 	@echo "License: Running licence checks.."
 	@scripts/check_license.sh
 
-linter: gotools build
+linter: gotools
 	@echo "LINT: Running code checks for Go files..."
 	@cd $$(/bin/pwd) && ./scripts/golinter.sh
 	@echo "LINT: Running code checks for Cpp/header files..."

build.mk

@@ -18,7 +18,6 @@ GO := $(GO_CMD) $(GOFLAGS)
 all: build test ci_report checks # keep checks last as license test is brittle ...
 
 .PHONY: test
-test: build
 
 .PHONY: ci_report
 

utils/Makefile

@@ -5,7 +5,7 @@
 TOP = ..
 include $(TOP)/build.mk
 
-SUB_DIRS = docker fabric
+SUB_DIRS = fabric
 
 all build test clean clobber:
 	$(foreach DIR, $(SUB_DIRS), $(MAKE) -C $(DIR) $@ || exit ;)