-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
x/tools: fix dep install for x/tools? #23981
Comments
Is there an open dep bug about this? |
Sounds like golang/dep#1272 or possibly golang/dep#899 |
Timed out in state WaitingForInfo. Closing. (I am just a bot, though. Please speak up if this is a mistake or you have the requested information.) |
Lol this is still an issue. Just because I didn't respond to a not-question doesn't mean that this issue is magically resolved. So dep appears to be a nascent state, unable to install even basic Go utliities, due to dep's requirement that all repositories feature Go code at the top directory level. I am proposing a workaround: Declare the utility version in a top level |
I think we are waiting to see whether dep is going to fix the problem. If dep makes a definitive decision that they will not support this kind of repo--which seems like an odd choice to me--then we can consider changing the repo here. |
Top-level directories that don't have Go code in them are completely fine. We should not turn inadequacies in tools into changes forced into every repo in existence. Even tools we own like Dep. |
dep is currently unable to resolve Go's x/tools due to an issue with directories that lack Go source files. Could we add a
version.go
or similar nonce file at the top of golang.org/x/tools, as a workaround?The text was updated successfully, but these errors were encountered: