Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

vendor: switch to moby/sys/capability #2428

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Conversation

kolyshkin
Copy link
Contributor

github.com/moby/sys/capability is a fork of the (no longer maintained) github.com/syndtr/gocapability package.

For changes since the fork took place, see https://github.com/moby/sys/blob/main/capability/CHANGELOG.md

Related:

github.com/moby/sys/capability is a fork of the (no longer maintained) github.com/syndtr/gocapability package.

For changes since the fork took place, see https://github.com/moby/sys/blob/main/capability/CHANGELOG.md

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@rhatdan
Copy link
Member

rhatdan commented Oct 1, 2024

LGTM
But it is still pulling in
github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635 // indirect

@vrothberg
Copy link
Member

github.com/syndtr/gocapability v0.0.0-20200815063812-42c35b437635 // indirect

Probably from containers/common?

@rhatdan rhatdan enabled auto-merge October 1, 2024 10:41
@rhatdan rhatdan disabled auto-merge October 1, 2024 10:42
@rhatdan rhatdan merged commit f240ce0 into containers:main Oct 1, 2024
23 checks passed
@mtrmac
Copy link
Contributor

mtrmac commented Oct 1, 2024

It’s both c/common and c/storage, but they have also been migrated. On the next update the old version will be removed, so that’s fine; no need to do anything extra right now.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants