Skip to content

Add pthread_sigqueue.#2873

Merged
bors merged 1 commit into
rust-lang:masterfrom
pirocks:master
Aug 15, 2022
Merged

Add pthread_sigqueue.#2873
bors merged 1 commit into
rust-lang:masterfrom
pirocks:master

Conversation

@pirocks

@pirocks pirocks commented Aug 13, 2022

Copy link
Copy Markdown
Contributor

This resurrects #991.

Should be a pretty straightforward addition of pthread_sigqueue.

@rust-highfive

Copy link
Copy Markdown

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

pirocks commented Aug 13, 2022

Copy link
Copy Markdown
Contributor Author

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

pirocks commented Aug 13, 2022

Copy link
Copy Markdown
Contributor Author

Should be good to go now.

@JohnTitor

Copy link
Copy Markdown
Member

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

@bors

bors commented Aug 15, 2022

Copy link
Copy Markdown
Contributor

📌 Commit f967994 has been approved by JohnTitor

It is now in the queue for this repository.

@bors

bors commented Aug 15, 2022

Copy link
Copy Markdown
Contributor

⌛ Testing commit f967994 with merge 08c0f2c...

@bors

bors commented Aug 15, 2022

Copy link
Copy Markdown
Contributor

☀️ 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