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: discard stderr when querying vcs tools #9523
Comments
An even safer option might be to bail when stderr from the vcs command is non-empty. |
The problem is in:
... which merges stderr and stdout. |
I'd like to fix this, but it's too late to be making fundamental changes like Possibly Brad's fix for #11381 will make this easier. |
Missed this again. |
I think unless there are more instances of this problem, we can just leave this alone. Git seems not to have this particular problem, and I can only assume that the hg codereview extension is dying out. Reopen if you have a case that you believe is still compelling. Thanks. |
My copy of
rsc.io/x86
contained anhgrc
that pointed to a copy ofcodereview.py
that has moved.hg
commands onrsc.io/x86
thus yielded warnings. Thego
command interpreted the warning as part of the output, leading to a spurious import path violation:The fix is probably to only use stdout, not stdout+stderr. I have confirmed that the warning came on stderr and the useful output on stdout.
The text was updated successfully, but these errors were encountered: