Skip to content
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

bump golang.org/x/sys from 0.6.0 to v0.14.0 #1206

Closed
wants to merge 1 commit into from

Conversation

lifubang
Copy link

@lifubang lifubang commented Nov 6, 2023

As we still use golang.org/x/sys with the version 0.6.0, it makes us can't bump it to the latest version 0.14.0 in other projects depending on github.com/cilium/ebpf.

Please see opencontainers/runc#4108

@lmb
Copy link
Collaborator

lmb commented Nov 7, 2023

The correct fix is to wait for a release of x/sys which has the constant and then update your minimum version to that. Seems like there is already a CL ongoing, see the upstream Go issue.

@lmb lmb closed this Nov 7, 2023
tklauser added a commit to tklauser/ebpf that referenced this pull request Nov 9, 2023
Updating golang.org/x/sys/unix to the recently released v0.14.0 will
break the build because of the missing definition of
BPF_F_KPROBE_MULTI_RETURN, see
golang/go#63969 and
cilium#1206.

This was fixed upstream in https://go.dev/cl/540395, so manually pull in
the latest unreleased version to allow modules depending on cilium/ebpf
to update to later versions as well.

Signed-off-by: Tobias Klauser <[email protected]>
lmb pushed a commit that referenced this pull request Nov 9, 2023
Updating golang.org/x/sys/unix to the recently released v0.14.0 will
break the build because of the missing definition of
BPF_F_KPROBE_MULTI_RETURN, see
golang/go#63969 and
#1206.

This was fixed upstream in https://go.dev/cl/540395, so manually pull in
the latest unreleased version to allow modules depending on cilium/ebpf
to update to later versions as well.

Signed-off-by: Tobias Klauser <[email protected]>
@lifubang lifubang deleted the feat-golang-sys-0.14.0 branch October 15, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants