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

Downloading sdkv4 #251

Closed
paologallinaharbur opened this issue Dec 17, 2020 · 4 comments
Closed

Downloading sdkv4 #251

paologallinaharbur opened this issue Dec 17, 2020 · 4 comments
Labels
bug Categorizes issue or PR as related to a bug.

Comments

@paologallinaharbur
Copy link
Member

SDKv4 has been released but we had issues importing it

$go get github.com/newrelic/infra-integrations-sdk/v4@v4.0.0
go get github.com/newrelic/infra-integrations-sdk/v4@v4.0.0: github.com/newrelic/infra-integrations-sdk@v4.0.0: invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v4

$go get github.com/newrelic/infra-integrations-sdk@v4.0.0
go get github.com/newrelic/infra-integrations-sdk@v4.0.0: github.com/newrelic/infra-integrations-sdk@v4.0.0: invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v4

If I run simply $go get github.com/newrelic/infra-integrations-sdk I get version:
github.com/newrelic/infra-integrations-sdk v3.6.5+incompatible // indirect

Should we add in the go.mod the version of the package?
something like this

golang/go#35732 (comment)

@paologallinaharbur paologallinaharbur added the bug Categorizes issue or PR as related to a bug. label Dec 17, 2020
@carlosroman
Copy link
Contributor

So we did something bad and delete the old V4 tag as realised that the way go mod was implemented was in correct. We recreated the tag with a fix added to it. You now have to use go get github.com/newrelic/infra-integrations-sdk/v4.

@carlosroman
Copy link
Contributor

But yes, fix was to add the version to the package as you can see from the PR #252.

@paologallinaharbur
Copy link
Member Author

Awesome thanks, we can close the issue I believe

@paologallinaharbur
Copy link
Member Author

$ go get github.com/newrelic/infra-integrations-sdk/v4
go: github.com/newrelic/infra-integrations-sdk/v4 upgrade => v4.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants