From 102e54829d8297f9a1488c2fb4ad73610594eb9a Mon Sep 17 00:00:00 2001 From: Bo-Yi Wu Date: Tue, 15 Aug 2017 14:02:45 +0800 Subject: [PATCH] refactor(Makefile): allow overriding default go program --- Makefile | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/Makefile b/Makefile index 76aeffc..6cde98c 100644 --- a/Makefile +++ b/Makefile @@ -2,6 +2,7 @@ DIST := dist EXECUTABLE := drone-ssh +$(GO) ?= go # for dockerhub DEPLOY_ACCOUNT := appleboy @@ -9,7 +10,7 @@ DEPLOY_IMAGE := $(EXECUTABLE) GOFMT ?= gofmt "-s" TARGETS ?= linux darwin windows -PACKAGES ?= $(shell go list ./... | grep -v /vendor/) +PACKAGES ?= $(shell $(GO) list ./... | grep -v /vendor/) GOFILES := $(shell find . -name "*.go" -type f -not -path "./vendor/*") SOURCES ?= $(shell find . -name "*.go" -type f) TAGS ?= @@ -32,7 +33,6 @@ all: build .PHONY: fmt-check fmt-check: - # get all go files and run go fmt on them @diff=$$($(GOFMT) -d $(GOFILES)); \ if [ -n "$$diff" ]; then \ echo "Please run 'make fmt' and commit the result:"; \ @@ -43,7 +43,7 @@ fmt-check: .PHONY: test-vendor test-vendor: @hash govendor > /dev/null 2>&1; if [ $$? -ne 0 ]; then \ - go get -u github.com/kardianos/govendor; \ + $(GO) get -u github.com/kardianos/govendor; \ fi govendor list +unused | tee "$(TMPDIR)/wc-gitea-unused" [ $$(cat "$(TMPDIR)/wc-gitea-unused" | wc -l) -eq 0 ] || echo "Warning: /!\\ Some vendor are not used /!\\" @@ -57,39 +57,39 @@ fmt: $(GOFMT) -w $(GOFILES) vet: - go vet $(PACKAGES) + $(GO) vet $(PACKAGES) errcheck: @hash errcheck > /dev/null 2>&1; if [ $$? -ne 0 ]; then \ - go get -u github.com/kisielk/errcheck; \ + $(GO) get -u github.com/kisielk/errcheck; \ fi errcheck $(PACKAGES) lint: @hash golint > /dev/null 2>&1; if [ $$? -ne 0 ]; then \ - go get -u github.com/golang/lint/golint; \ + $(GO) get -u github.com/golang/lint/golint; \ fi for PKG in $(PACKAGES); do golint -set_exit_status $$PKG || exit 1; done; unconvert: @hash unconvert > /dev/null 2>&1; if [ $$? -ne 0 ]; then \ - go get -u github.com/mdempsky/unconvert; \ + $(GO) get -u github.com/mdempsky/unconvert; \ fi for PKG in $(PACKAGES); do unconvert -v $$PKG || exit 1; done; test: fmt-check - for PKG in $(PACKAGES); do go test -v -cover -coverprofile $$GOPATH/src/$$PKG/coverage.txt $$PKG || exit 1; done; + for PKG in $(PACKAGES); do $(GO) test -v -cover -coverprofile $$GOPATH/src/$$PKG/coverage.txt $$PKG || exit 1; done; html: - go tool cover -html=coverage.txt + $(GO) tool cover -html=coverage.txt install: $(SOURCES) - go install -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' + $(GO) install -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' build: $(EXECUTABLE) $(EXECUTABLE): $(SOURCES) - go build -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' -o $@ + $(GO) build -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' -o $@ release: release-dirs release-build release-copy release-check @@ -98,7 +98,7 @@ release-dirs: release-build: @hash gox > /dev/null 2>&1; if [ $$? -ne 0 ]; then \ - go get -u github.com/mitchellh/gox; \ + $(GO) get -u github.com/mitchellh/gox; \ fi gox -os="$(TARGETS)" -arch="amd64 386" -tags="$(TAGS)" -ldflags="-s -w $(LDFLAGS)" -output="$(DIST)/binaries/$(EXECUTABLE)-$(VERSION)-{{.OS}}-{{.Arch}}" @@ -110,7 +110,7 @@ release-check: # for docker. static_build: - CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' -o $(DEPLOY_IMAGE) + CGO_ENABLED=0 GOOS=linux GOARCH=amd64 $(GO) build -a -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' -o $(DEPLOY_IMAGE) docker_image: docker build -t $(DEPLOY_ACCOUNT)/$(DEPLOY_IMAGE) . @@ -133,7 +133,7 @@ coverage: ./.codecov -f .cover/coverage.txt clean: - go clean -x -i ./... + $(GO) clean -x -i ./... rm -rf coverage.txt $(EXECUTABLE) $(DIST) vendor ssh-server: