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

kqueue: Ignore EPIPE coming out of kevent #583

Closed
wants to merge 1 commit into from

Conversation

alexcrichton
Copy link
Contributor

This commit updates mio to ignore EPIPE coming out of kevent. It
turns out that older versions of OSX (10.10 and 10.11 have been
confirmed) will return an EPIPE when registering one half of a pipe
where the other half has been closed. On other platforms this is an ok
operation which just returns that the pipe is readable/writable (to
return an error/eof), so this brings the OSX behavior in line by
ignoring the error and moving to the next event.

Closes #582

This commit updates mio to ignore `EPIPE` coming out of `kevent`. It
turns out that older versions of OSX (10.10 and 10.11 have been
confirmed) will return an `EPIPE` when registering one half of a pipe
where the other half has been closed. On other platforms this is an ok
operation which just returns that the pipe is readable/writable (to
return an error/eof), so this brings the OSX behavior in line by
ignoring the error and moving to the next event.

Closes tokio-rs#582
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Apr 4, 2017
I've tracked down what I believe is the last spurious sccache failure on rust-lang#40240
to behavior in mio (tokio-rs/mio#583), and this commit updates the binaries to
a version which has that fix incorporated.
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Apr 5, 2017
I've tracked down what I believe is the last spurious sccache failure on rust-lang#40240
to behavior in mio (tokio-rs/mio#583), and this commit updates the binaries to
a version which has that fix incorporated.
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 5, 2017
…sxcv

travis: Update sccache binaries

I've tracked down what I believe is the last spurious sccache failure on rust-lang#40240
to behavior in mio (tokio-rs/mio#583), and this commit updates the binaries to
a version which has that fix incorporated.
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 5, 2017
…sxcv

travis: Update sccache binaries

I've tracked down what I believe is the last spurious sccache failure on rust-lang#40240
to behavior in mio (tokio-rs/mio#583), and this commit updates the binaries to
a version which has that fix incorporated.
@carllerche
Copy link
Member

Closed by #584

@carllerche carllerche closed this Apr 7, 2017
@alexcrichton alexcrichton deleted the ignore-epipe-osx branch April 28, 2017 15:43
alexcrichton added a commit to mozilla/sccache that referenced this pull request Apr 28, 2017
Contains tokio-rs/mio#583 to fix tokio-rs/mio#582, transitively fixing
flakiness discovered in rust-lang/rust#40240.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

suspicion: EPIPE coming out of register on OSX
2 participants