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

@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

@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

@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