You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I actually don't want to have to figure out if a crypto/tls internal failure is due to a replaced chacha20poly1305.
But of course the same is true for x-repo interdependencies in general: if someone is reporting an issue against, say, x/tools or x/net, it would be really helpful to know the exact version in which they found the issue, and whether any of the dependencies involved have been locally modified.
I propose that we request go list -m all alongside go env in the issue template.
This is probably good to do independent of the outcome of #30241.
Some people will want to remove proprietary or internal modules from the output of go list -m all, or substitute in foo or corp.example or similar. The template could attempt to briefly mention that is fine to do. Alternatively, could instead just wait to see if it confuses anyone in practice.
If you are not running with module mode enabled, go list -m all currently reports:
go list -m: not using modules
That is probably fine and understandable enough to not confuse people.
This seems reasonable, @bcmills want to send a CL for this? We can try it and adjust based on experience.
dmitshur
added
NeedsFix
The path to resolution is known, but the work has not been done.
Community
and removed
NeedsDecision
Feedback is required from experts, contributors, and/or the community before a change can be made.
labels
Jun 28, 2022
As @FiloSottile notes in #30241 (comment):
But of course the same is true for
x
-repo interdependencies in general: if someone is reporting an issue against, say,x/tools
orx/net
, it would be really helpful to know the exact version in which they found the issue, and whether any of the dependencies involved have been locally modified.I propose that we request
go list -m all
alongsidego env
in the issue template.(CC @thepudds @myitcv @bradfitz)
The text was updated successfully, but these errors were encountered: