-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Make QNX/NTO specific "timespec capping" public to crate::sys #121498
Conversation
It is used in: - `library/std/src/sys/locks/condvar/pthread.rs` - `library/std/src/sys/pal/unix/thread_parking/pthread.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.
Is this a draft by accident? Or have you not tested it?
I have tested it and I think it can be merged but wanted to give @jonathanpallant and @japaric the chance to have a look at it as well 😃 . |
Sorry for not making this clear from the beginning |
Can confirm this patch allows the qnx std target to build again, and I used it to successfully create a QNX binary. |
@Nilstrieb please r+ |
@bors r+ rollup |
…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#120742 (mark `min_exhaustive_patterns` as complete) - rust-lang#121470 (Don't ICE on anonymous struct in enum variant) - rust-lang#121492 (coverage: Rename `is_closure` to `is_hole`) - rust-lang#121495 (remove repetitive words) - rust-lang#121498 (Make QNX/NTO specific "timespec capping" public to crate::sys) - rust-lang#121510 (lint-overflowing-ops: unify cases and remove redundancy) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#121498 - flba-eb:make_timespec_capping_public, r=Nilstrieb Make QNX/NTO specific "timespec capping" public to crate::sys It is used in: - `library/std/src/sys/locks/condvar/pthread.rs` - `library/std/src/sys/pal/unix/thread_parking/pthread.rs` This is probably required due to introduction of `sys/pal` and rust-lang#121177 `@rustbot` label +O-neutrino CC: `@jonathanpallant` `@japaric` `@gh-tr`
It is used in:
library/std/src/sys/locks/condvar/pthread.rs
library/std/src/sys/pal/unix/thread_parking/pthread.rs
This is probably required due to introduction of
sys/pal
and #121177@rustbot label +O-neutrino
CC: @jonathanpallant @japaric @gh-tr