Story #11720
closed
- Description updated (diff)
- Status changed from New to In Progress
Procedure for updating dependencies:
- Ensure
$GOPATH
points somewhere (maybe ~/go
)
go get github.com/kardianos/govendor
- Either
- add a symlink to a checked-out arvados tree:
mkdir -p ${GOPATH}/src/git.curoverse.com && ln -sfT ${WORKSPACE} ${GOPATH}/src/git.curoverse.com/arvados.git
- ...or check out the arvados tree the normal Go way:
go get git.curoverse.com/arvados.git
cd ${GOPATH}/src/git.curoverse.com/arvados.git && ${GOPATH}/bin/govendor update +v
"+v" means all vendor packages. You can also update a single dependency, etc. See https://github.com/kardianos/govendor
Now $GOPATH/src/git.coroverse.com/arvados.git/vendor/vendor.json is updated with new version specs. Run tests, commit, push.
- Target version set to 2017-05-24 sprint
- Assigned To set to Tom Clegg
in build/run-tests.sh
cd "$GOPATH/src/git.curoverse.com/arvados.git" && \
"$GOPATH/bin/govendor" sync -v || \
fatal "govendor sync failed"
shouldn't be using GOROOT ? I had the impression GOPATH is a colon/comma separated values of the path, that in 95% of the time is only onve value,
but I could be wrong and this changed in later versions of Go
Nico César wrote:
shouldn't be using GOROOT ? I had the impression GOPATH is a colon/comma separated values of the path, that in 95% of the time is only onve value,
That's true in general, but in run-tests.sh we set GOPATH to {tempdir}/GOPATH so I don't think we need to support the multi-value case.
(GOROOT is where stdlib and the go tools themselves live)
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:b22ff6242148a9b37baef6c74701fa85a8765adf.
Also available in: Atom
PDF