Skip to content

Commit

Permalink
Invoke Go consistently and remove GO15VENDOREXPERIMENT from container
Browse files Browse the repository at this point in the history
  • Loading branch information
Alfonso Acosta committed Feb 25, 2016
1 parent 02808e3 commit db62f8c
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ shell: $(SCOPE_BACKEND_BUILD_UPTODATE)
/bin/bash

tests: $(SCOPE_BACKEND_BUILD_UPTODATE)
./tools/test -no-go-get
env -u GOGC $(GO_ENVS) ./tools/test -no-go-get

lint: $(SCOPE_BACKEND_BUILD_UPTODATE)
./tools/lint .
Expand Down
1 change: 0 additions & 1 deletion backend/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
FROM golang:1.5.3
ENV GO15VENDOREXPERIMENT 1
RUN apt-get update && apt-get install -y libpcap-dev python-requests time
RUN go clean -i net && go install -tags netgo std && go install -race -tags netgo std
RUN go get -tags netgo \
Expand Down
8 changes: 4 additions & 4 deletions experimental/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ DIRS=$(shell find . -maxdepth 2 -name *.go | xargs -n1 dirname | sort -u)
TARGETS=$(join $(patsubst %,%/,$(DIRS)),$(patsubst ./%,%,$(DIRS)))
BUILD_IN_CONTAINER=true
RM=--rm

GO ?= env GO15VENDOREXPERIMENT=1 GOGC=off go
all: $(TARGETS)

ifeq ($(BUILD_IN_CONTAINER),true)
Expand All @@ -13,11 +13,11 @@ $(TARGETS):
weaveworks/scope-backend-build -C experimental $@
else
$(TARGETS):
go build -ldflags "-extldflags \"-static\"" -tags netgo -o $@ ./$(@D)
$(GO) build -ldflags "-extldflags \"-static\"" -tags netgo -o $@ ./$(@D)
endif

test:
go test ./...
$(GO) test ./...

clean:
go clean ./...
$(GO) clean ./...

0 comments on commit db62f8c

Please sign in to comment.