Skip to content

openbsd: make WIFCONTINUED() safe#2035

Merged
bors merged 1 commit into
rust-lang:masterfrom
semarie:openbsd-safesignal
Jan 18, 2021
Merged

openbsd: make WIFCONTINUED() safe#2035
bors merged 1 commit into
rust-lang:masterfrom
semarie:openbsd-safesignal

Conversation

@semarie

@semarie semarie commented Jan 16, 2021

Copy link
Copy Markdown
Contributor

it seems it was a missed part of #1891

Found it because rustc -nightly started to use libc::WIFCONTINUED() recently in a safe context and the build fail to build on OpenBSD.

Fixes #2001

@rust-highfive

Copy link
Copy Markdown

r? @JohnTitor

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

@JohnTitor JohnTitor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for catching this!

@JohnTitor

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Jan 18, 2021

Copy link
Copy Markdown
Contributor

📌 Commit cf0793a has been approved by JohnTitor

@bors

bors commented Jan 18, 2021

Copy link
Copy Markdown
Contributor

⌛ Testing commit cf0793a with merge 7f11cdb...

@bors

bors commented Jan 18, 2021

Copy link
Copy Markdown
Contributor

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

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.

WIFCONTINUED is unsafe on OpenBSD

4 participants