Skip to content

Commit 94cf881

Browse files
committed
Add PHONY for variable targets
The Makefile targets that are variables, weren't set as PHONY. That means their recipes weren't run when the generated files already existed. This could cause issues, such as with an out of date controller-gen. Signed-off-by: James Slagle <jslagle@redhat.com>
1 parent b7b08af commit 94cf881

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

Makefile

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -303,6 +303,7 @@ undeploy: ## Undeploy controller from the K8s cluster specified in ~/.kube/confi
303303

304304
## Location to install dependencies to
305305
LOCALBIN ?= $(shell pwd)/bin
306+
.PHONY: $(LOCALBIN)
306307
$(LOCALBIN):
307308
mkdir -p $(LOCALBIN)
308309

@@ -327,6 +328,7 @@ OC_VERSION ?= 4.16.0
327328
KUSTOMIZE_INSTALL_SCRIPT ?= "https://raw.githubusercontent.com/kubernetes-sigs/kustomize/master/hack/install_kustomize.sh"
328329
.PHONY: kustomize
329330
kustomize: $(KUSTOMIZE) ## Download kustomize locally if necessary. If wrong version is installed, it will be removed before downloading.
331+
.PHONY: $(KUSTOMIZE)
330332
$(KUSTOMIZE): $(LOCALBIN)
331333
@if test -x $(LOCALBIN)/kustomize && ! $(LOCALBIN)/kustomize version | grep -q $(KUSTOMIZE_VERSION); then \
332334
echo "$(LOCALBIN)/kustomize version is not expected $(KUSTOMIZE_VERSION). Removing it before installing."; \
@@ -336,22 +338,26 @@ $(KUSTOMIZE): $(LOCALBIN)
336338

337339
.PHONY: controller-gen
338340
controller-gen: gowork $(CONTROLLER_GEN) ## Download controller-gen locally if necessary. If wrong version is installed, it will be overwritten.
341+
.PHONY: $(CONTROLLER_GEN)
339342
$(CONTROLLER_GEN): $(LOCALBIN)
340343
test -s $(LOCALBIN)/controller-gen && $(LOCALBIN)/controller-gen --version | grep -q $(CONTROLLER_TOOLS_VERSION) || \
341344
GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-tools/cmd/controller-gen@$(CONTROLLER_TOOLS_VERSION)
342345

343346
.PHONY: crd-to-markdown
344347
crd-to-markdown: $(CRD_MARKDOWN) ## Download crd-to-markdown locally if necessary.
348+
.PHONY: $(CRD_MARKDOWN)
345349
$(CRD_MARKDOWN): $(LOCALBIN)
346350
test -s $(LOCALBIN)/crd-to-markdown || GOBIN=$(LOCALBIN) go install github.com/clamoriniere/crd-to-markdown@$(CRD_MARKDOWN_VERSION)
347351

348352
.PHONY: envtest
349353
envtest: $(ENVTEST) ## Download envtest-setup locally if necessary.
354+
.PHONY: $(ENVTEST)
350355
$(ENVTEST): $(LOCALBIN)
351356
test -s $(LOCALBIN)/setup-envtest || GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-runtime/tools/setup-envtest@$(SETUP_ENVTEST_VERSION)
352357

353358
.PHONY: ginkgo
354359
ginkgo: $(GINKGO) ## Download ginkgo locally if necessary.
360+
.PHONY: $(GINKGO)
355361
$(GINKGO): $(LOCALBIN)
356362
test -s $(LOCALBIN)/ginkgo || GOBIN=$(LOCALBIN) go install github.com/onsi/ginkgo/v2/ginkgo
357363

@@ -361,6 +367,7 @@ kuttl-test: ## Run kuttl tests
361367

362368
.PHONY: kuttl
363369
kuttl: $(KUTTL) ## Download kubectl-kuttl locally if necessary.
370+
.PHONY: $(KUTTL)
364371
$(KUTTL): $(LOCALBIN)
365372
test -s $(LOCALBIN)/kubectl-kuttl || curl -L -o $(LOCALBIN)/kubectl-kuttl https://github.com/kudobuilder/kuttl/releases/download/v$(KUTTL_VERSION)/kubectl-kuttl_$(KUTTL_VERSION)_linux_x86_64
366373
chmod +x $(LOCALBIN)/kubectl-kuttl

0 commit comments

Comments
 (0)