domi/forgejo-domi
1
0
Fork 0
forked from forgejo/forgejo
Code Pull requests Activity

Combine Drone release steps (#9338)

Browse source 
* Combine Drone release steps

Fixes missing JS/CSS because drone did unwanted parallelization of the
js/css task and the generate task. Combined the tasks into one and made
'make release' work standalone.

Fixes: https://github.com/go-gitea/gitea/issues/9324
Fixes: https://github.com/go-gitea/gitea/issues/9362

* move js/css to generate dependencies

* remove unneccessary go-all target

* remove go target as well
This commit is contained in:
silverwind 2019-12-15 19:06:36 +01:00 • committed by techknowlogick
parent 60b31c8f01
commit 7217b703e9
2 changed files with 6 additions and 34 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Makefile
View file

@ -128,7 +128,7 @@ vet:
$(GO) vet $(PACKAGES)
.PHONY: generate
generate:
generate: js css
GO111MODULE=on $(GO) generate -mod=vendor $(PACKAGES)
.PHONY: generate-swagger
@ -384,14 +384,8 @@ check: test
install: $(wildcard *.go)
$(GO) install -v -tags '$(TAGS)' -ldflags '-s -w $(LDFLAGS)'
.PHONY: go
go: go-check $(EXECUTABLE)
.PHONY: go-all
go-all: go-check generate go
.PHONY: build
build: js css go-all
build: go-check generate $(EXECUTABLE)
$(EXECUTABLE): $(GO_SOURCES)
GO111MODULE=on $(GO) build -mod=vendor $(GOFLAGS) $(EXTRA_GOFLAGS) -tags '$(TAGS)' -ldflags '-s -w $(LDFLAGS)' -o $@