Skip to content

Commit

Permalink
Fix (make vendor)
Browse files Browse the repository at this point in the history
(export a=b command args) does not run (command args) with a=b,
it sets $a to b, and marks variables $a $command $args as exported,
i.e. (command args) is not run.

So, before #2287 we were not actually
running (go mod tidy), and now we are not running (go mod vendor).

Fix that.

Signed-off-by: Miloslav Trmač <[email protected]>
  • Loading branch information
mtrmac authored and rhatdan committed Apr 8, 2020
1 parent ce2fc8b commit 779025e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
7 changes: 3 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -142,10 +142,9 @@ vendor-in-container:

.PHONY: vendor
vendor:
export GO111MODULE=on \
$(GO) mod vendor && \
$(GO) mod tidy && \
$(GO) mod verify
GO111MODULE=on $(GO) mod vendor
GO111MODULE=on $(GO) mod tidy
GO111MODULE=on $(GO) mod verify

.PHONY: lint
lint: install.tools
Expand Down
7 changes: 3 additions & 4 deletions tests/tools/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,9 @@ all: $(BUILDDIR)

.PHONY: vendor
vendor:
export GO111MODULE=on \
$(GO) mod tidy && \
$(GO) mod vendor && \
$(GO) mod verify
GO111MODULE=on $(GO) mod tidy
GO111MODULE=on $(GO) mod vendor
GO111MODULE=on $(GO) mod verify

define go-build
$(shell cd `pwd` && $(GO_BUILD) -o $(BUILDDIR)/$(shell basename $(1)) $(1))
Expand Down

0 comments on commit 779025e

Please sign in to comment.