-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Reuse modules on hermit
#84521
Reuse modules on hermit
#84521
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
|
pub mod io; | ||
pub mod memchr; | ||
pub mod mutex; | ||
pub mod net; | ||
pub mod os; | ||
#[path = "../unix/path.rs"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I'm a bit worried that in theory people changing the unix path.rs will not expect it to also get picked up on other platforms. It does seem pretty unlikely to get changed, but maybe we can at least add a comment to that file that it's also used by other platforms?
Modulo path.rs being reused this seems fine -- curious to hear @m-ou-se if you have thoughts as you've done some recent work on the sys/ files with the mutex stuff if that's a real concern |
That
We could maybe move it out of |
In that case, @bors r+ Seems good to potentially move it out but it's probably a common pattern and the sys stuff doesn't change much outside additions or refactors. |
📌 Commit 36e9382 has been approved by |
Rollup of 5 pull requests Successful merges: - rust-lang#84132 (Ignore nonstandard lldb version strings in compiletest) - rust-lang#84521 (Reuse modules on `hermit`) - rust-lang#84563 (Update backtrace to 0.3.57) - rust-lang#84610 (Update Clippy) - rust-lang#84613 (move representability checks to rustc_ty_utils) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Looking around
Not sure what to do about those last two, I would like to move them out of
Of these I personally think option 4 would be the best way forward, maybe with a comment in the |
Reuse the following modules on
hermit
:unix::path
(contents identical)unsupported::io
(contents identical)unsupported::thread_local_key
(contents functionally identical, only changes are the panic error messages)@rustbot label: +T-libs-impl