From 5ab93bc8efb0caebf525f4065126a5847adf46aa Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 7 Jan 2019 17:32:56 +0100 Subject: [PATCH] Fix some missing targets in "make help" The `make help` output was missing some targets (`fmt`, `test-unit`, `test-e2e`). Note that some targets are still hidden (`test-e2e-experimental`, `test-e2e-non-experimental`, `test-e2e-connhelper-ssh`) as they're likely not usually run separate from `test-e2e`. Before this patch: make -f docker.Makefile help binary build the CLI build alias for binary clean clean build artifacts cross build the CLI for macOS and Windows binary-windows build the CLI for Windows binary-osx build the CLI for macOS dev start a build container in interactive mode for in-container development shell alias for dev lint run linters vendor download dependencies (vendor/) listed in vendor.conf dynbinary build the CLI dynamically linked authors generate AUTHORS file from git history manpages generate man pages from go source and markdown yamldocs generate documentation YAML files consumed by docs repo shellcheck run shellcheck validation help print this help With this patch applied: make -f docker.Makefile help binary build the CLI build alias for binary clean clean build artifacts test-unit run unit tests (using go test) cross build the CLI for macOS and Windows binary-windows build the CLI for Windows binary-osx build the CLI for macOS dev start a build container in interactive mode for in-container development shell alias for dev lint run linters fmt run gofmt vendor download dependencies (vendor/) listed in vendor.conf dynbinary build the CLI dynamically linked authors generate AUTHORS file from git history manpages generate man pages from go source and markdown yamldocs generate documentation YAML files consumed by docs repo shellcheck run shellcheck validation test-e2e run all e2e tests help print this help Signed-off-by: Sebastiaan van Stijn --- Makefile | 2 +- docker.Makefile | 16 ++++++++-------- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/Makefile b/Makefile index 5e4fad15b4..09fbfb202c 100644 --- a/Makefile +++ b/Makefile @@ -73,7 +73,7 @@ shellcheck: ## run shellcheck validation .PHONY: help help: ## print this help - @awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z_-]+:.*?## / {sub("\\\\n",sprintf("\n%22c"," "), $$2);printf "\033[36m%-20s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST) + @awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z0-9_-]+:.*?## / {sub("\\\\n",sprintf("\n%22c"," "), $$2);printf "\033[36m%-20s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST) cli/compose/schema/bindata.go: cli/compose/schema/data/*.json diff --git a/docker.Makefile b/docker.Makefile index 8c45a5a688..c8f6de2172 100644 --- a/docker.Makefile +++ b/docker.Makefile @@ -62,7 +62,7 @@ clean: build_docker_image ## clean build artifacts docker volume rm -f $(CACHE_VOLUME_NAME) .PHONY: test-unit -test-unit: build_docker_image # run unit tests (using go test) +test-unit: build_docker_image ## run unit tests (using go test) docker run --rm $(ENVVARS) $(MOUNTS) $(DEV_DOCKER_IMAGE_NAME) make test-unit .PHONY: test ## run unit and e2e tests @@ -93,7 +93,7 @@ lint: build_linter_image ## run linters docker run -ti --rm $(ENVVARS) $(MOUNTS) $(LINTER_IMAGE_NAME) .PHONY: fmt -fmt: +fmt: ## run gofmt docker run --rm $(ENVVARS) $(MOUNTS) $(DEV_DOCKER_IMAGE_NAME) make fmt .PHONY: vendor @@ -119,21 +119,21 @@ yamldocs: build_docker_image ## generate documentation YAML files consumed by do shellcheck: build_shell_validate_image ## run shellcheck validation docker run -ti --rm $(ENVVARS) $(MOUNTS) $(VALIDATE_IMAGE_NAME) make shellcheck -.PHONY: test-e2e ## run e2e tests -test-e2e: test-e2e-non-experimental test-e2e-experimental test-e2e-connhelper-ssh +.PHONY: test-e2e +test-e2e: test-e2e-non-experimental test-e2e-experimental test-e2e-connhelper-ssh ## run all e2e tests .PHONY: test-e2e-experimental -test-e2e-experimental: build_e2e_image +test-e2e-experimental: build_e2e_image # run experimental e2e tests docker run --rm -v /var/run/docker.sock:/var/run/docker.sock -e DOCKERD_EXPERIMENTAL=1 $(E2E_IMAGE_NAME) .PHONY: test-e2e-non-experimental -test-e2e-non-experimental: build_e2e_image +test-e2e-non-experimental: build_e2e_image # run non-experimental e2e tests docker run --rm -v /var/run/docker.sock:/var/run/docker.sock $(E2E_IMAGE_NAME) .PHONY: test-e2e-connhelper-ssh -test-e2e-connhelper-ssh: build_e2e_image +test-e2e-connhelper-ssh: build_e2e_image # run experimental SSH-connection helper e2e tests docker run --rm -v /var/run/docker.sock:/var/run/docker.sock -e DOCKERD_EXPERIMENTAL=1 -e TEST_CONNHELPER=ssh $(E2E_IMAGE_NAME) .PHONY: help help: ## print this help - @awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z_-]+:.*?## / {sub("\\\\n",sprintf("\n%22c"," "), $$2);printf "\033[36m%-20s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST) + @awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z0-9_-]+:.*?## / {sub("\\\\n",sprintf("\n%22c"," "), $$2);printf "\033[36m%-20s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST)