Update go version to 1.7.0

[riyazdf]

Bumps to go 1.7.0.

We can replace golang.org/x/net/context with context in a follow-up PR because we'll have to also bump grpc to a version that also uses context

Closes #928

Signed-off-by: Riyaz Faizullabhoy [email protected]

[riyazdf]

yubikey hardware tests are failing because our jenkins infrastructure has go1.6, will bump the go version there and re-run :)

[riyazdf]

Installed Go 1.7 on our yubikey jenkins infrastructure and we're green!

[cyli]

That is awesome! LGTM! Thanks for keeping us up to date!

(I am strangely locally seeing some vet errors on go1.7:

server/handlers/default_test.go:419: literal copies lock value from *metaStore: github.com/docker/notary/server/storage.MemStorage contains sync.Mutex
server/handlers/default_test.go:461: literal copies lock value from *metaStore: github.com/docker/notary/server/storage.MemStorage contains sync.Mutex

But if the CI passes on 1.7 my local state must be broken.)

[riyazdf]

@cyli hm yeah that's weird. I just tried running govet against this PR on our yubikey test infrastructure and didn't run into any errors. Not sure if other folks are seeing this or what's going on 😕

[endophage]

LGTM!

[HuKeping]

@cyli I tried this PR manually but didn't get those go vet warnings.

[cyli]

Thanks! It must be some kind of weird local setting or inconsistency I
have.

On Friday, September 2, 2016, HuKeping [email protected] wrote:

@cyli https://github.com/cyli I tried this PR manually but didn't get
those go vet warnings.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#936 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANn4P6SKDCaNrmh3ArJRlmY-9rLPTdoks5qmAQtgaJpZM4JyAUg
.