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

[0.2] Deprecate FreeBSD's CAP_UNUSED* and CAP_ALL* constants #4183

Closed
wants to merge 1 commit into from

Conversation

asomers
Copy link
Contributor

@asomers asomers commented Dec 8, 2024

They aren't stable across OS versions, and they don't have any legitimate use in applications.

Description

Deprecate constants that are unstable and not useful to our consumers

Sources

This commit, for example, changed some of these constants in FreeBSD 15.

freebsd/freebsd-src@b165e9e

Checklist

  • Relevant tests in libc-test/semver have been updated
  • No placeholder or unstable values like *LAST or *MAX are
    included (see #3131)
  • Tested locally (cd libc-test && cargo test --target mytarget);
    especially relevant for platforms that may not be checked in CI

They aren't stable across OS versions, and they don't have any
legitimate use in applications.
@rustbot
Copy link
Collaborator

rustbot commented Dec 8, 2024

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@asomers
Copy link
Contributor Author

asomers commented Dec 8, 2024

I audited all of the consumers of the capsicum-rs crate. None of them use these constants, and they all build OK with a patched libc. For the main branch, I think we can simply delete these constants.

@asomers
Copy link
Contributor Author

asomers commented Dec 8, 2024

CI is failing because this branch needs #4177 .

asomers added a commit to asomers/libc that referenced this pull request Dec 8, 2024
They aren't stable across OS versions and don't have any legitimate use
in applications.

See rust-lang#4183 for the corresponding
change to the libc-0.2 branch.
Copy link
Contributor

@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking into this. I’ll probably have to pick it and merge together with the other fix.

@tgross35 tgross35 changed the title Deprecate FreeBSD's CAP_UNUSED* and CAP_ALL* constants [0.2] Deprecate FreeBSD's CAP_UNUSED* and CAP_ALL* constants Dec 9, 2024
@tgross35 tgross35 mentioned this pull request Dec 9, 2024
@tgross35
Copy link
Contributor

tgross35 commented Dec 9, 2024

Applied via #4187. Not sure why GH didn't close this PR but this should be all set.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants