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

linux musl adding PIDFD_NONBLOCK constant. #3003

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

devnexen
Copy link
Contributor

closes #3002

@rust-highfive
Copy link

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

@JohnTitor
Copy link
Member

@devnexen devnexen force-pushed the musl_pidfd_nonblock branch from 515a816 to b9dbd5e Compare November 19, 2022 09:35
@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 20, 2022

📌 Commit b9dbd5e has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Nov 20, 2022

⌛ Testing commit b9dbd5e with merge e2a3239...

bors added a commit that referenced this pull request Nov 20, 2022
linux musl adding `PIDFD_NONBLOCK` constant.

closes #3002
@bors
Copy link
Contributor

bors commented Nov 20, 2022

💔 Test failed - checks-actions

@@ -590,6 +590,9 @@ pub const EFD_NONBLOCK: ::c_int = ::O_NONBLOCK;

pub const SFD_NONBLOCK: ::c_int = ::O_NONBLOCK;

pub const SOCK_NONBLOCK: ::c_int = O_NONBLOCK;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems we already have SOCK_NONBLOCK

@devnexen devnexen force-pushed the musl_pidfd_nonblock branch from b9dbd5e to 7ba33fd Compare November 20, 2022 08:48
@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 20, 2022

📌 Commit 7ba33fd has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Nov 20, 2022

⌛ Testing commit 7ba33fd with merge 34a5773...

bors added a commit that referenced this pull request Nov 20, 2022
linux musl adding `PIDFD_NONBLOCK` constant.

closes #3002
@bors
Copy link
Contributor

bors commented Nov 20, 2022

💔 Test failed - checks-actions

@devnexen devnexen force-pushed the musl_pidfd_nonblock branch from 7ba33fd to 0993260 Compare November 20, 2022 12:08
@JohnTitor
Copy link
Member

@bors r+

1 similar comment
@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 30, 2022

📌 Commit 0993260 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Nov 30, 2022

⌛ Testing commit 0993260 with merge b8c8e13...

@bors
Copy link
Contributor

bors commented Nov 30, 2022

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

@bors bors merged commit b8c8e13 into rust-lang:master Nov 30, 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.

PIDFD_NONBLOCK missing from Linux musl toolchains
4 participants