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

Add pthread_sigqueue. #2873

Merged
merged 1 commit into from
Aug 15, 2022
Merged

Add pthread_sigqueue. #2873

merged 1 commit into from
Aug 15, 2022

Conversation

pirocks
Copy link
Contributor

@pirocks pirocks commented Aug 13, 2022

This resurrects #991.

Should be a pretty straightforward addition of pthread_sigqueue.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.

Please see the contribution instructions for more information.

@pirocks pirocks changed the title Add pthread_sigqueue. Draft: Add pthread_sigqueue. Aug 13, 2022
@pirocks
Copy link
Contributor Author

pirocks commented Aug 13, 2022

Drafted while figure out how to fix CI.

@pirocks pirocks changed the title Draft: Add pthread_sigqueue. Add pthread_sigqueue. Aug 13, 2022
pthread_sigqueue is a gnu libc extension, however I have a use case for it,
hence why I'm adding it.
@pirocks
Copy link
Contributor Author

pirocks commented Aug 13, 2022

Should be good to go now.

@JohnTitor
Copy link
Member

Thanks for the comment on the test, looks sensible to me. @bors r+

@bors
Copy link
Contributor

bors commented Aug 15, 2022

📌 Commit f967994 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Aug 15, 2022

⌛ Testing commit f967994 with merge 08c0f2c...

@bors
Copy link
Contributor

bors commented Aug 15, 2022

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

@bors bors merged commit 08c0f2c into rust-lang:master Aug 15, 2022
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.

5 participants