Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests still import nats-streaming-server and break go mod #280

Closed
asiragusa opened this issue Sep 10, 2019 · 7 comments
Closed

Tests still import nats-streaming-server and break go mod #280

asiragusa opened this issue Sep 10, 2019 · 7 comments

Comments

@asiragusa
Copy link

The old import can be found here: https://github.com/nats-io/stan.go/blob/master/stan_test.go#L35. This breaks go mod, as the old repository is archived.

@kozlovic
Copy link
Member

Yes, tests import the streaming server and that is expected. I am not sure what you are referring to as "the old import".

@kozlovic
Copy link
Member

It seems that your go.mod is still using old nats-server and nats-streaming-server releases (pre go.mod). Try to update those to the latest.
https://github.com/Jeffail/benthos/pull/275/files#diff-37aff102a57d3d7b797f152915a6dc16R54

@kozlovic
Copy link
Member

Also, note that if you are using Go 1.13, you will have problems with nats and stan repos (since their name ends in .go): golang/go#32483

@asiragusa
Copy link
Author

Ok I managed to make it work, by updating the nats-streaming-server dependency, but I had to specify the latest commit, as there is an issue with the v0.16.0 and golang,org/x/sys:

go: finding github.com/nats-io/nats-streaming-server v0.16.0
go: downloading github.com/nats-io/nats-streaming-server v0.16.0
go: extracting github.com/nats-io/nats-streaming-server v0.16.0
go: github.com/nats-io/nats-streaming-server@v0.16.0 requires
        golang.org/x/sys@v0.0.0-20190726091711-fde4db37ae7a: invalid pseudo-version: does not match version-control timestamp (2019-08-13T06:44:41Z)

A new patch release could fix this

@kozlovic
Copy link
Member

A new patch release could fix this

Correct. We are going to release a new version soon that will include nats-io/nats-streaming-server#924.

@asiragusa
Copy link
Author

Ok thanks

@kozlovic
Copy link
Member

kozlovic commented Dec 2, 2019

Closing this issue now.

@kozlovic kozlovic closed this as completed Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants