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/link: external linking on darwin strips symbols starting with "l" #33808
Comments
I think we already add leading _ to symbol names for plugins, for exactly the reason mentioned in go-delve/delve#1635 (comment) : dlsym needs it. We may want to do this for other build modes as well. |
Change https://golang.org/cl/194381 mentions this issue: |
Fixes #33808 Change-Id: If1f30bc80004093ffdf9121768464dfb3a6e1f63 Reviewed-on: https://go-review.googlesource.com/c/go/+/194381 Run-TryBot: Jeremy Faller <jeremy@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Given that the change was reverted this issue should be reopened, I think. |
Totally right. I'm on vacation till Monday, but will fix whatever caused the trybots unhappiness first thing. |
Change https://golang.org/cl/195619 mentions this issue: |
Apparently external linking hasn't ever worked on darwin_386. I'll exclude that in the test, and resubmit. |
Change https://golang.org/cl/196217 mentions this issue: |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
What did you expect to see?
I expected that log.Fatal would still be in my executable.
What did you see instead?
Symbols starting with an l have been stripped.
Oringally reported HERE
The text was updated successfully, but these errors were encountered: