Skip to content

Commit

Permalink
trim the Makefile (#4054)
Browse files Browse the repository at this point in the history
Remove sections which deal with `deps`. These are not required with the
move to Go modules, and create unnecessary diffs in go.mod and go.sum
files.

Left the rest as is, as some are used from `testing.yaml` and others
serve as a handy reference when one wants to build/test something
specific.
  • Loading branch information
easwars authored Nov 20, 2020
1 parent f6c335c commit 83af853
Showing 1 changed file with 4 additions and 26 deletions.
30 changes: 4 additions & 26 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,46 +1,28 @@
all: vet test testrace

build: deps
build:
go build google.golang.org/grpc/...

clean:
go clean -i google.golang.org/grpc/...

deps:
go get -d -v google.golang.org/grpc/...

proto:
@ if ! which protoc > /dev/null; then \
echo "error: protoc not installed" >&2; \
exit 1; \
fi
go generate google.golang.org/grpc/...

test: testdeps
test:
go test -cpu 1,4 -timeout 7m google.golang.org/grpc/...

testsubmodule: testdeps
testsubmodule:
cd security/advancedtls && go test -cpu 1,4 -timeout 7m google.golang.org/grpc/security/advancedtls/...
cd security/authorization && go test -cpu 1,4 -timeout 7m google.golang.org/grpc/security/authorization/...

testappengine: testappenginedeps
goapp test -cpu 1,4 -timeout 7m google.golang.org/grpc/...

testappenginedeps:
goapp get -d -v -t -tags 'appengine appenginevm' google.golang.org/grpc/...

testdeps:
go get -d -v -t google.golang.org/grpc/...

testrace: testdeps
testrace:
go test -race -cpu 1,4 -timeout 7m google.golang.org/grpc/...

updatedeps:
go get -d -v -u -f google.golang.org/grpc/...

updatetestdeps:
go get -d -v -t -u -f google.golang.org/grpc/...

vet: vetdeps
./vet.sh

Expand All @@ -51,14 +33,10 @@ vetdeps:
all \
build \
clean \
deps \
proto \
test \
testappengine \
testappenginedeps \
testdeps \
testrace \
updatedeps \
updatetestdeps \
vet \
vetdeps

0 comments on commit 83af853

Please sign in to comment.