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

Commit

Permalink
CI: Do not build, but do update, repos when pulling them
Browse files Browse the repository at this point in the history
Without a '-d' on the 'go get', the script ties to build the repos,
and for repos that do not work under 'go build', this leaves errors
in the CI logs. We don't need a build at this stage as we invoke the
repo specific builds later on.
Also, some CIs do not clean out their build trees between builds, and
thus have repos already present - so add an 'update -u' flag as well.

Fixes: #887

Signed-off-by: Graham Whaley <[email protected]>
  • Loading branch information
Graham Whaley committed Feb 5, 2018
1 parent 402b386 commit 2aa4720
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .ci/jenkins_job_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,11 @@ then
go get github.com/mattn/goveralls
fi

# Get the repository and move to the correct commit
go get ${cc_repo} || true
# Get or update the repository, but do build ie.
go get -d -u ${cc_repo} || true
pushd ${GOPATH}/src/${cc_repo}

# And move to the correct commit
pr_number=

[ "${ghprbPullId}" ] && [ "${ghprbTargetBranch}" ] && pr_number="${ghprbPullId}"
Expand Down

0 comments on commit 2aa4720

Please sign in to comment.