-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
dockerfile: add delve to binaries-for-test #5497
base: master
Are you sure you want to change the base?
Conversation
112b447
to
7b369d8
Compare
7b369d8
to
a684664
Compare
Does not seem supported on
Seems we need to enable CGO: https://github.com/go-delve/delve/blob/b16e12fde736f7cad60cfe4d029ba9595ca8de0c/pkg/proc/native/support_sentinel_cgo_freebsd.go#L5 See also go-delve/delve#1480 |
How much would we loose if we'd skip this for freeBSD? (mostly considering if it's worth adding CGO only for that platform) |
Yes I think I will skip freebsd, enabling CGO doesn't seem to work, some headers look missing:
|
a684664
to
b9d503b
Compare
Signed-off-by: CrazyMax <[email protected]>
b9d503b
to
fe1583a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the add.
exit 0 | ||
fi | ||
xx-go install "github.com/go-delve/delve/cmd/dlv@${DELVE_VERSION}" | ||
if ! xx-info is-cross; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you change the GOBIN
solution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can't install cross-compiled binaries when GOBIN is set: https://github.com/golang/go/blob/84e58c84fdbe4681b4cbfeb615aaba3437c8ac85/src/cmd/go/internal/work/build.go#L771-L772
follow-up #5291 (comment)