Skip to content

Commit

Permalink
fix(makefile): remove testbin
Browse files Browse the repository at this point in the history
  • Loading branch information
Thuan Vo committed Aug 3, 2022
1 parent 4c5e0bb commit ded5684
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,6 @@ vendor/
*.so
*.dylib
bin
testbin/*

# Test binary, build with `go test -c`
*.test
Expand Down
11 changes: 3 additions & 8 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -230,11 +230,6 @@ LOCALBIN ?= $(shell pwd)/bin
$(LOCALBIN):
mkdir -p $(LOCALBIN)

# Location to install test dependencies
TESTBIN ?= $(shell pwd)/testbin
$(TESTBIN):
mkdir -p $(TESTBIN)

# Download controller-gen locally if necessary
CONTROLLER_GEN = $(LOCALBIN)/controller-gen
.PHONY: controller-gen
Expand All @@ -258,11 +253,11 @@ $(ADDLICENSE): $(LOCALBIN)
GOBIN=$(LOCALBIN) go install github.com/google/addlicense@v$(ADDLICENSE_VERSION)

# Download setup-envtest locally if necessary
ENVTEST = $(TESTBIN)/setup-envtest
ENVTEST = $(LOCALBIN)/setup-envtest
.PHONY: setup-envtest
setup-envtest: $(ENVTEST)
$(ENVTEST): $(TESTBIN)
GOBIN=$(TESTBIN) go install sigs.k8s.io/controller-runtime/tools/setup-envtest@latest
$(ENVTEST): $(LOCALBIN)
GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-runtime/tools/setup-envtest@latest

# Download opm locally if necessary
OPM = $(LOCALBIN)/opm
Expand Down

0 comments on commit ded5684

Please sign in to comment.