std: make a lot of items crate private by pacak · Pull Request #161612 · rust-lang/rust

pacak · 2026-08-23T17:37:36Z

Enabled warn(unreachable_pub)
Made a lot of things crate private, to the point when doing more required adding exceptions
Removed warn(unreachable_pub)

So now this commit just tightens the current visibility without trying to enforce it.

View all comments

r? clarfonthey

clarfonthey · 2026-08-23T19:32:29Z

So, looking at this, I'm going to nominate this for discussion alongside the clippy lints PR: #161328

Since they should go together, I think.

@rustbot label:I-libs-nominated

pacak · 2026-08-23T19:52:32Z

Sure.

Please no more obscure platform failures, please no more obscure platform failures, please no more obscure platform failures, please no more obscure platform failures...

rustbot · 2026-08-23T23:21:22Z

Any special-casing of Miri in the standard library requires review.

cc @rust-lang/miri

pacak · 2026-08-23T23:28:38Z

Any special-casing of Miri in the standard library requires review.

Heh. I think at that point I'll just remove the warn and allow and leave as much correct visibility tightening as possible without adding exceptions.

pacak · 2026-08-25T09:09:35Z

☔ The latest upstream changes

I was afraid of this.

clarfonthey · 2026-08-26T21:09:29Z

So, I forgot to say the verdict from our meeting yesterday, which is everyone is cool with me making the call on this as a reviewer. We do have this turned on for core and alloc, so, I think it would be fine to keep the lint enabled here but maybe explicitly allow the lint for the os and sys modules as this makes it more of a mess.

Feel free to also allow the lint for any modules you find particularly annoying. We seem to have a standard of trying to keep it on by default and just allowing it for annoying bits.

Most of them don't need to be public, but there are scenarios where thing is private on one platform but public on the other, so having a lint on all the time gets complicated. I enabled the lint, made a lot of things private to the point that dealing with the rest required adding exceptions and disabled the lint back again.

rustbot · 2026-08-26T21:42:48Z

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

pacak · 2026-08-26T23:10:07Z

It's alive!!!

clarfonthey · 2026-08-26T23:57:44Z

So, you still have a few changes inside the sys and os modules: were you hoping to keep those, or?

pacak · 2026-08-27T00:31:39Z

Right now I see them as "it was improved, but it is not enforced by clippy". I can probably go though a few iterations of whack-a-mole and enforce some more of them if you think its worth it. Probably tomorrow or something.

clarfonthey · 2026-08-27T00:37:57Z

I mostly mention it in the sense of, I figure if we're going to allow the lint for these modules, we probably shouldn't commit any of the changes for them, but I don't care too much either way. Mostly just curious if you had a compelling reason for it.

rustbot added the O-unix Operating system: Unix-like label Aug 23, 2026

rustbot assigned clarfonthey Aug 23, 2026

rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 23, 2026