-
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
dist: strip debug information in release version #5450
Labels
Comments
Stripping binaries is not tested and not guaranteed to work by anyone. It might lead to strange bugs, and previous attempts have resulted in crashes and buggy executables. See issue #5200 for example. So it is a dangerous change and it should not be done so late in the Go 1.1 release process. Also end users encounter bugs, so they are required to debug. |
In fact, as a Go end user. I prefer to strip more debug info in the my Go built binary. https://golang.org/issue/3467 |
I think, it just like Go's compiler did not allow 'variable unused'. If every package packer treated 3MB(at least) is nothing. Then a linux distribution may added 300MB and waste much energy in the earth. My opinion is: if it is useful, then keep it. If the Go developers think it won't be use, then remove it is better. |
please see https://golang.org/issue/5200?c=21 we still have at least two bugs concerning stripped binaries, so don't do that, it's not guaranteed to work. |
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: