mirror of
				https://github.com/appleboy/drone-ssh.git
				synced 2025-10-29 00:51:15 +08:00 
			
		
		
		
	refactor(Makefile): allow overriding default go program (#95)
* refactor(Makefile): allow overriding default go program * fix typo Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
This commit is contained in:
		
							parent
							
								
									bf269615ce
								
							
						
					
					
						commit
						5fbd22f265
					
				
							
								
								
									
										28
									
								
								Makefile
									
									
									
									
									
								
							
							
						
						
									
										28
									
								
								Makefile
									
									
									
									
									
								
							| @ -2,6 +2,7 @@ | |||||||
| 
 | 
 | ||||||
| DIST := dist | DIST := dist | ||||||
| EXECUTABLE := drone-ssh | EXECUTABLE := drone-ssh | ||||||
|  | GO ?= go | ||||||
| 
 | 
 | ||||||
| # for dockerhub
 | # for dockerhub
 | ||||||
| DEPLOY_ACCOUNT := appleboy | DEPLOY_ACCOUNT := appleboy | ||||||
| @ -9,7 +10,7 @@ DEPLOY_IMAGE := $(EXECUTABLE) | |||||||
| GOFMT ?= gofmt "-s" | GOFMT ?= gofmt "-s" | ||||||
| 
 | 
 | ||||||
| TARGETS ?= linux darwin windows | 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/*") | GOFILES := $(shell find . -name "*.go" -type f -not -path "./vendor/*") | ||||||
| SOURCES ?= $(shell find . -name "*.go" -type f) | SOURCES ?= $(shell find . -name "*.go" -type f) | ||||||
| TAGS ?= | TAGS ?= | ||||||
| @ -32,7 +33,6 @@ all: build | |||||||
| 
 | 
 | ||||||
| .PHONY: fmt-check | .PHONY: fmt-check | ||||||
| fmt-check: | fmt-check: | ||||||
| 	# get all go files and run go fmt on them |  | ||||||
| 	@diff=$$($(GOFMT) -d $(GOFILES)); \
 | 	@diff=$$($(GOFMT) -d $(GOFILES)); \
 | ||||||
| 	if [ -n "$$diff" ]; then \
 | 	if [ -n "$$diff" ]; then \
 | ||||||
| 		echo "Please run 'make fmt' and commit the result:"; \
 | 		echo "Please run 'make fmt' and commit the result:"; \
 | ||||||
| @ -43,7 +43,7 @@ fmt-check: | |||||||
| .PHONY: test-vendor | .PHONY: test-vendor | ||||||
| test-vendor: | test-vendor: | ||||||
| 	@hash govendor > /dev/null 2>&1; if [ $$? -ne 0 ]; then \
 | 	@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 | 	fi | ||||||
| 	govendor list +unused | tee "$(TMPDIR)/wc-gitea-unused" | 	govendor list +unused | tee "$(TMPDIR)/wc-gitea-unused" | ||||||
| 	[ $$(cat "$(TMPDIR)/wc-gitea-unused" | wc -l) -eq 0 ] || echo "Warning: /!\\ Some vendor are not used /!\\" | 	[ $$(cat "$(TMPDIR)/wc-gitea-unused" | wc -l) -eq 0 ] || echo "Warning: /!\\ Some vendor are not used /!\\" | ||||||
| @ -57,39 +57,39 @@ fmt: | |||||||
| 	$(GOFMT) -w $(GOFILES) | 	$(GOFMT) -w $(GOFILES) | ||||||
| 
 | 
 | ||||||
| vet: | vet: | ||||||
| 	go vet $(PACKAGES) | 	$(GO) vet $(PACKAGES) | ||||||
| 
 | 
 | ||||||
| errcheck: | errcheck: | ||||||
| 	@hash errcheck > /dev/null 2>&1; if [ $$? -ne 0 ]; then \
 | 	@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 | 	fi | ||||||
| 	errcheck $(PACKAGES) | 	errcheck $(PACKAGES) | ||||||
| 
 | 
 | ||||||
| lint: | lint: | ||||||
| 	@hash golint > /dev/null 2>&1; if [ $$? -ne 0 ]; then \
 | 	@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 | 	fi | ||||||
| 	for PKG in $(PACKAGES); do golint -set_exit_status $$PKG || exit 1; done; | 	for PKG in $(PACKAGES); do golint -set_exit_status $$PKG || exit 1; done; | ||||||
| 
 | 
 | ||||||
| unconvert: | unconvert: | ||||||
| 	@hash unconvert > /dev/null 2>&1; if [ $$? -ne 0 ]; then \
 | 	@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 | 	fi | ||||||
| 	for PKG in $(PACKAGES); do unconvert -v $$PKG || exit 1; done; | 	for PKG in $(PACKAGES); do unconvert -v $$PKG || exit 1; done; | ||||||
| 
 | 
 | ||||||
| test: fmt-check | 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: | html: | ||||||
| 	go tool cover -html=coverage.txt | 	$(GO) tool cover -html=coverage.txt | ||||||
| 
 | 
 | ||||||
| install: $(SOURCES) | install: $(SOURCES) | ||||||
| 	go install -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' | 	$(GO) install -v -tags '$(TAGS)' -ldflags '$(EXTLDFLAGS)-s -w $(LDFLAGS)' | ||||||
| 
 | 
 | ||||||
| build: $(EXECUTABLE) | build: $(EXECUTABLE) | ||||||
| 
 | 
 | ||||||
| $(EXECUTABLE): $(SOURCES) | $(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 | release: release-dirs release-build release-copy release-check | ||||||
| 
 | 
 | ||||||
| @ -98,7 +98,7 @@ release-dirs: | |||||||
| 
 | 
 | ||||||
| release-build: | release-build: | ||||||
| 	@hash gox > /dev/null 2>&1; if [ $$? -ne 0 ]; then \
 | 	@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 | 	fi | ||||||
| 	gox -os="$(TARGETS)" -arch="amd64 386" -tags="$(TAGS)" -ldflags="-s -w $(LDFLAGS)" -output="$(DIST)/binaries/$(EXECUTABLE)-$(VERSION)-{{.OS}}-{{.Arch}}" | 	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.
 | # for docker.
 | ||||||
| static_build: | 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_image: | ||||||
| 	docker build -t $(DEPLOY_ACCOUNT)/$(DEPLOY_IMAGE) . | 	docker build -t $(DEPLOY_ACCOUNT)/$(DEPLOY_IMAGE) . | ||||||
| @ -133,7 +133,7 @@ coverage: | |||||||
| 	./.codecov -f .cover/coverage.txt | 	./.codecov -f .cover/coverage.txt | ||||||
| 
 | 
 | ||||||
| clean: | clean: | ||||||
| 	go clean -x -i ./... | 	$(GO) clean -x -i ./... | ||||||
| 	rm -rf coverage.txt $(EXECUTABLE) $(DIST) vendor | 	rm -rf coverage.txt $(EXECUTABLE) $(DIST) vendor | ||||||
| 
 | 
 | ||||||
| ssh-server: | ssh-server: | ||||||
|  | |||||||
		Loading…
	
		Reference in New Issue
	
	Block a user
	 Bo-Yi Wu
						Bo-Yi Wu