From cc564a7692e2f6b768905d0f42f05810db6d7fd8 Mon Sep 17 00:00:00 2001 From: Daniel Nephin Date: Tue, 2 May 2017 16:40:29 -0400 Subject: [PATCH] Verify that vendored files are correct Signed-off-by: Daniel Nephin Upstream-commit: 31851fbe3ccd646a930fd379782ac8a64a869af9 Component: cli --- components/cli/Makefile | 10 ++++++++++ components/cli/circle.yml | 12 +++++++++--- components/cli/dockerfiles/Dockerfile.ci | 5 +---- 3 files changed, 20 insertions(+), 7 deletions(-) diff --git a/components/cli/Makefile b/components/cli/Makefile index dadad7c584..f9ccd585d4 100644 --- a/components/cli/Makefile +++ b/components/cli/Makefile @@ -25,3 +25,13 @@ cross: clean @gox -output build/docker-{{.OS}}-{{.Arch}} \ -osarch="linux/arm linux/amd64 darwin/amd64 windows/amd64" \ github.com/docker/cli/cmd/docker + +vendor: vendor.conf + @vndr 2> /dev/null + @if [ "`git status --porcelain -- vendor 2>/dev/nul`" ]; then \ + echo; echo "vendoring is wrong. These files were changed:"; \ + echo; git status --porcelain -- vendor 2>/dev/nul; \ + echo; exit 1; \ + fi; + +ci: cross test diff --git a/components/cli/circle.yml b/components/cli/circle.yml index 29fd13c3ba..e6fedd44d4 100644 --- a/components/cli/circle.yml +++ b/components/cli/circle.yml @@ -7,11 +7,17 @@ jobs: steps: - checkout - setup_remote_docker - - run: - name: "Lint" + - run: + name: "Lint" command: | docker build -f dockerfiles/Dockerfile.lint --tag cli-linter . docker run cli-linter - run: name: "Build and Unit Test" - command: docker build -f dockerfiles/Dockerfile.ci . + command: | + docker build -f dockerfiles/Dockerfile.ci --tag cli-builder . + docker run cli-builder + - run: + name: "Vendor" + command: | + docker run cli-builder make vendor diff --git a/components/cli/dockerfiles/Dockerfile.ci b/components/cli/dockerfiles/Dockerfile.ci index 29918badab..0dd14ee192 100644 --- a/components/cli/dockerfiles/Dockerfile.ci +++ b/components/cli/dockerfiles/Dockerfile.ci @@ -24,7 +24,4 @@ COPY . /go/src/github.com/docker/cli ENV CGO_ENABLED=0 WORKDIR /go/src/github.com/docker/cli -RUN make cross -RUN make test - -CMD sh +CMD ["make", "ci"]