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

Fix/remove deprecated DragonFly items #2522

Merged
merged 1 commit into from
Dec 5, 2021

Conversation

rtzoeller
Copy link
Contributor

These items were recently deprecated on DragonFly, either because the platform does not define them or because they were out of date.

These items were recently deprecated on DragonFly, either because
the platform does not define them or because they were out of date.
@rust-highfive
Copy link

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

@rtzoeller
Copy link
Contributor Author

I'm not sure how long the maintainers prefer to wait after deprecating items before fixing them, but I thought I'd put a PR up now and let the maintainers merge it when they felt it was appropriate.

My hope is that it can be merged without waiting too many releases, since DragonFly is a tier 3 target for rust.

@JohnTitor
Copy link
Member

I think about a month is enough, maybe? Looking at the download stats on https://crates.io/crates/libc, some of the active users have updated but right now is a bit early, I feel.

@rtzoeller rtzoeller mentioned this pull request Nov 16, 2021
asomers added a commit to nix-rust/nix that referenced this pull request Nov 16, 2021
libc::stat is deprecated on DragonflyBSD in libc.  But there isn't any
alternative yet, so Nix must simply suppress the warnings.  It's used in
too many places to suppress each one individually, so just suppress all
deprecation warnings globally until it's properly fixed.

rust-lang/libc#2522
@rtzoeller
Copy link
Contributor Author

@JohnTitor it's been a little under a month (~4 weeks), but based on the download stats it looks like we're likely reaching the point of diminishing returns. I don't see any open PRs calling for a new release (suggesting the next one is at least a few days away); are we at a point where this can be merged?

@JohnTitor
Copy link
Member

Okay, let's assume most active users should've got the deprecation note, @bors r+

@bors
Copy link
Contributor

bors commented Dec 5, 2021

📌 Commit 45cd3dc has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Dec 5, 2021

⌛ Testing commit 45cd3dc with merge b76911e...

@bors
Copy link
Contributor

bors commented Dec 5, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing b76911e to master...

@bors bors merged commit b76911e into rust-lang:master Dec 5, 2021
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.

4 participants