-
Notifications
You must be signed in to change notification settings - Fork 273
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
Failure to go get because of moved gcimporter15 #58
Comments
Same problem |
For anyone looking to move forward with this, you can patch from my fork: serussell/goreporter@3514eb452. It'll get you going (compiles & runs), but it's a quick hack and I haven't vetted the solution for correctness as I don't entirely understand why the original code was doing what it was doing in the way it was doing it (it seemed needlessly complex). |
@ednp @eric00liu Thank you very much for your feedback on this issue. I did not have version control on goreporter.I have solved it with godep, and later I will check if the dependency can be updated. @serussell Great fix. |
same here in go1.11 I solved it with go1.10 and the corresponding version of golang.org/x/tools. The problem is that gcimporter15 was removed, and I found that this package was still in go1.10. But golint relies on go/gcexportdata, which is added in go1.8, so I think go1.8~1.10 should be ok |
how to solve this problem?? I have same problem |
Go the same problem. Go version go1.16.3 linux/amd64 https://pkg.go.dev/golang.org/x/tools/go/gcimporter15
|
It appears goreporter relies on gcimporter15, which has been moved:
https://go-review.googlesource.com/c/tools/+/105977
https://godoc.org/golang.org/x/tools/go
Code location that imports it:
https://github.com/360EntSecGroup-Skylar/goreporter/blob/master/linters/simplecode/lint/lint.go#L23
Example error when running
go get -u github.com/360EntSecGroup-Skylar/goreporter
:The text was updated successfully, but these errors were encountered: