Skip to content

Commit

Permalink
rustdoc: Use unix_sigpipe instead of `rustc_driver::set_sigpipe_han…
Browse files Browse the repository at this point in the history
…dler`

Start using `unix_sigpipe` instead of
`rustc_driver::set_sigpipe_handler`.

After this has been merged, we can completely remove
`rustc_driver::set_sigpipe_handler`.

Verification of this change
---------------------------

1. Remove `#[unix_sigpipe = "sig_dfl"]`
1. Run `./x.py build`
1. Run `./build/aarch64-apple-darwin/stage1/bin/rustdoc --help | false`
1. Observe ICE
1. Add back `#[unix_sigpipe = "sig_dfl"]`
1. Run `./x.py build`
1. Run `./build/aarch64-apple-darwin/stage1/bin/rustdoc --help | false`
1. Observe ICE fixed
  • Loading branch information
Enselic committed Oct 24, 2022
1 parent c07a6a9 commit 5e624bf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/librustdoc/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,6 @@ pub fn main() {
}
}

rustc_driver::set_sigpipe_handler();
rustc_driver::install_ice_hook();

// When using CI artifacts (with `download_stage1 = true`), tracing is unconditionally built
Expand Down
3 changes: 3 additions & 0 deletions src/tools/rustdoc/main.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
#![feature(unix_sigpipe)]

#[unix_sigpipe = "sig_dfl"]
fn main() {
rustdoc::main()
}

0 comments on commit 5e624bf

Please sign in to comment.