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
cmd/go: Make go get -u update existing packages, or add another flag #13332
Comments
The golang/lint bugs are often due to the move of x/tools from code.google.com to github. There is, of course, a general class of problems here to do with the use of vanity import paths, but the specific case of the Go subrepos is a particularly notable instance of that class, and I wonder if it would be worth explicitly checking for them in |
The wording is "By default, get uses..." (emphasis mine).
So |
Note that it only updates the particular checkout of a package (e.g. approximately |
@dsymonds hm, that seems like an oversight. We should probably print an error in that case. |
Didn't f3a98de address the issue of vanity paths changing where they point? |
I'm guessing f3a98de only checked the named package, not its dependencies. |
"go get -u" does update existing packages. Maybe there is something wrong with something else. If so, please file a separate bug. |
There is a constant stream (golang/lint#173, golang/lint#169, golang/lint#168, golang/lint#167, golang/lint#165 this list goes on and on) of user for golint that have outdated packages. This reports make it apparent that the
-u
flag ofgo get
is kind of flawed. As the documentation for it statesThe wording "by default" would suggest that there is another flag which does update existing packages but it seems that this is still missing.
I am happy to work on this, but would first like to discuss if this should be another flag, and if so, what its name should be.
The text was updated successfully, but these errors were encountered: