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

Update go.sum to remove invalid dependency #143

Closed
wants to merge 1 commit into from
Closed

Update go.sum to remove invalid dependency #143

wants to merge 1 commit into from

Conversation

llimllib
Copy link

Now that github.com/golang/lint has become golang.org/x/lint, that dependency causes my build to break with: go: github.com/golang/[email protected]: parsing go.mod: unexpected module path "golang.org/x/lint"

This commit just removes that line from go.sum

Now that `github.com/golang/lint` has become `golang.org/x/lint`, that dependency causes my build to break with: `go: github.com/golang/[email protected]: parsing go.mod: unexpected module path "golang.org/x/lint"`

This commit just removes that line from go.sum
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 27, 2019
jeanbza added a commit to jeanbza/go-genproto that referenced this pull request Feb 27, 2019
Updates go.mod to include new version of gRPC, which removes its
dependency on the old github.com/golang/lint in favour of the new
golang.org/x/lint.

Relevant: googleapis#143
@jeanbza jeanbza mentioned this pull request Feb 27, 2019
@jeanbza
Copy link
Member

jeanbza commented Feb 27, 2019

Looks like the dependency is coming in through gRPC:

$ GO111MODULE=on go mod graph
[...]
google.golang.org/genproto google.golang.org/[email protected]
[...]
google.golang.org/[email protected] github.com/golang/[email protected]
[...]

I've created #144 to do the needful.

@jeanbza jeanbza closed this Feb 27, 2019
@llimllib
Copy link
Author

ah thanks, sorry for my incomplete research

@llimllib llimllib deleted the remove-invalid-dep branch February 27, 2019 21:32
jeanbza added a commit that referenced this pull request Feb 27, 2019
Updates go.mod to include new version of gRPC, which removes its
dependency on the old github.com/golang/lint in favour of the new
golang.org/x/lint.

Relevant: #143
@jeanbza
Copy link
Member

jeanbza commented Feb 27, 2019

Not a problem at all, thanks for reporting this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants