Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Commit

Permalink
Remove native dep targets
Browse files Browse the repository at this point in the history
They don't work because GOPATH is redefined in the Makefile
So people will just have to rely on the docker, or install dep themselves.
  • Loading branch information
carolynvs-msft committed Jan 24, 2018
1 parent c2a3760 commit bb93bd3
Showing 1 changed file with 2 additions and 19 deletions.
21 changes: 2 additions & 19 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -371,25 +371,8 @@ release-push-%:
$(MAKE) ARCH=$* build
$(MAKE) ARCH=$* push

# Dependency management via dep (https://github.com/golang/dep)
PHONHY: check-dep get-dep verify-vendor test-dep
check-dep:
@if [ -z "$$(which dep)" ]; then \
echo 'Missing `dep` client which is required for development'; \
exit 2; \
else \
dep version; \
fi

get-dep:
# Install the latest release of dep
go get -d -u github.com/golang/dep
cd $(go env GOPATH)/src/github.com/golang/dep && \
DEP_TAG=$$(git describe --abbrev=0 --tags) && \
git checkout $$DEP_TAG && \
go install -ldflags="-X main.version=$$DEP_TAG" ./cmd/dep; \
git checkout master # Make go get happy by switching back to master

# Dependency management via dep (https://golang.github.io/dep)
PHONHY: verify-vendor test-dep
verify-vendor: .init
# Verify that vendor/ is in sync with Gopkg.lock
$(DOCKER_CMD) $(BUILD_DIR)/verify-vendor.sh
Expand Down

0 comments on commit bb93bd3

Please sign in to comment.