coniks-sys / coniks-go

A CONIKS implementation in Golang
http://coniks.org
Other
116 stars 30 forks source link

Run dep ensure #206

Closed corbym closed 6 years ago

corbym commented 6 years ago

You can run dep ensure in travis by running go get -u github.com/golang/dep/cmd/dep before dep ensure.

corbym commented 6 years ago

Looks like running dep ensure might be causing some tests to fail in travis on the Go 1.8 build?

Works on 1,9 and tip

vqhuy commented 6 years ago

Thanks @corbym for this PR!

masomel commented 6 years ago

Based on the travis logs, go get fails for a few external dependencies. Looking into it.

EDIT: These are the packages that are causing problems:

I suspect dep might be downloading a different version for these packages in go1.8?

corbym commented 6 years ago

It's possible you may need to branch for 1.8. it may be that the dependencies are no longer supporting 1.8.

The Twitter dependency is on master, for example. You won't be able to support master and a 1.8 compatible version of the dep.

vqhuy commented 6 years ago

So I tried to run dep ensure locally with go 1.8, and there was no issue as reported in Travis. You can try yourself with Docker:

# Dockerfile
FROM golang:1.8
WORKDIR /go/src/app
COPY . .
RUN go get -u github.com/golang/dep/cmd/dep
RUN dep ensure

I would wait for the "official" dep support of Travis. Feel free to re-open this PR if you find out something else.

mattcorbyeaglen commented 6 years ago

... but it's your build that's failing, not dep ensure. Run your build in the docker container after a dep ensure, that should duplicate the issue.

Just running dep ensure won't show anything.

masomel commented 6 years ago

@corbym The odd thing is that our build does not fail in Travis for go1.8 on any of our other branches, so it would seem that our build failure is somehow related to introducing dep into the Travis build flow.

You're definitely right, though, we should be running the build locally after running dep ensure. Will investigate further.

corbym commented 6 years ago

@masomel if I get time I will try to build your project under 1.8 in a docker container and run dep ensure in the same manner as the build tries to.

Sorry about the account switch above btw, I have two github accounts and sometimes I forget which is logged in!

vqhuy commented 6 years ago

Sorry, I thought the issue was in dep ensure step, it was late. The issue is reproducible in my docker container. The odd thing is go get in go 1.8 downloads dependencies to go/src directory, while it's not the case for go 1.9.

corbym commented 6 years ago

@c633 @masomel looks like you have mystery on your hands.. I'll leave it to you guys if that's ok?

masomel commented 6 years ago

Thanks again @corbym for bringing this to our attention :)