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

Make QNX/NTO specific "timespec capping" public to crate::sys #121498

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

flba-eb
Copy link
Contributor

@flba-eb flba-eb commented Feb 23, 2024

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

It is used in:

- `library/std/src/sys/locks/condvar/pthread.rs`
- `library/std/src/sys/pal/unix/thread_parking/pthread.rs`
@rustbot
Copy link
Collaborator

rustbot commented Feb 23, 2024

r? @cuviper

rustbot has assigned @cuviper.
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

@rustbot rustbot added O-unix Operating system: Unix-like 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. O-neutrino OS: QNX Neutrino, a POSIX-compatible real-time operating system labels Feb 23, 2024
Copy link
Member

@Noratrieb Noratrieb left a 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?

@flba-eb
Copy link
Contributor Author

flba-eb commented Feb 23, 2024

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 😃 .

@flba-eb
Copy link
Contributor Author

flba-eb commented Feb 23, 2024

Sorry for not making this clear from the beginning

@jonathanpallant
Copy link
Contributor

Can confirm this patch allows the qnx std target to build again, and I used it to successfully create a QNX binary.

@flba-eb flba-eb marked this pull request as ready for review February 23, 2024 15:17
@flba-eb
Copy link
Contributor Author

flba-eb commented Feb 23, 2024

@Nilstrieb please r+

@Noratrieb
Copy link
Member

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 23, 2024

📌 Commit 4f66783 has been approved by Nilstrieb

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 23, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 23, 2024
…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
@bors bors merged commit 2e8177a into rust-lang:master Feb 23, 2024
11 checks passed
@rustbot rustbot added this to the 1.78.0 milestone Feb 23, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Feb 23, 2024
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`
@flba-eb flba-eb deleted the make_timespec_capping_public branch December 10, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-neutrino OS: QNX Neutrino, a POSIX-compatible real-time operating system O-unix Operating system: Unix-like S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants