auto merge of #16342 : alexcrichton/rust/issue-16341, r=huonw
Now that rustdoc is spawning a child task, the program won't exit with a default error code if the main task fails (because it never fails). This commit forces the main task to wait for a child task in order to correctly propagate failure. Closes #16341
This commit is contained in:
commit
beda30e7ae
@ -120,9 +120,16 @@ pub fn main() {
|
||||
// So, in summary, it is unknown why this is necessary, what it is
|
||||
// preventing, or what the actual bug is. In the meantime, this allows
|
||||
// --test to work on windows, which seems good, right? Fun times.
|
||||
let (tx, rx) = channel();
|
||||
spawn(proc() {
|
||||
std::os::set_exit_status(main_args(std::os::args().as_slice()));
|
||||
tx.send(());
|
||||
});
|
||||
|
||||
// If the task failed, set an error'd exit status
|
||||
if rx.recv_opt().is_err() {
|
||||
std::os::set_exit_status(std::rt::DEFAULT_ERROR_CODE);
|
||||
}
|
||||
}
|
||||
|
||||
pub fn opts() -> Vec<getopts::OptGroup> {
|
||||
|
Loading…
Reference in New Issue
Block a user