auto merge of #14781 : alexcrichton/rust/issue-14724, r=brson

* os::pipe() now returns `IoResult<os::Pipe>`
* os::pipe() is now unsafe because it does not arrange for deallocation of file
  descriptors
* PipeStream::pair() has been added. This is a safe method to get a pair of
  pipes.
* Dealing with pipes in native process bindings have been improved to be more
  robust in the face of failure and intermittent errors. This converts a few
  fail!() situations to Err situations.

cc #13538
Closes #14724
[breaking-change]

1 files changed, 5 insertions, 5 deletions

diff --git a/src/test/run-pass/sigpipe-should-be-ignored.rs b/src/test/run-pass/sigpipe-should-be-ignored.rs
index 8e2cfa30066..8c68ef173a5 100644
--- a/src/test/run-pass/sigpipe-should-be-ignored.rs
+++ b/src/test/run-pass/sigpipe-should-be-ignored.rs
@@ -16,12 +16,12 @@ use std::io::PipeStream;
 use std::io::Command;
 
 fn test() {
-    let os::Pipe { input, out } = os::pipe();
-    let input = PipeStream::open(input);
-    let mut out = PipeStream::open(out);
-    drop(input);
+    let os::Pipe { reader, writer } = unsafe { os::pipe().unwrap() };
+    let reader = PipeStream::open(reader);
+    let mut writer = PipeStream::open(writer);
+    drop(reader);
 
-    let _ = out.write([1]);
+    let _ = writer.write([1]);
 }
 
 fn main() {