// SAFETY: We are setting the PGID of this (child) process to detach it from the group of the parent so signals are not propagated from the parent.
unsafe{
ifsetpgid(0,to)!=0{
returnErr(io::Error::last_os_error());
}
}
Ok(())
}
}
/// Spawn the process, and contain its standard output.
/// Spawn the process, and contain its standard output.
///
///
/// # Notes
/// # Notes
@ -89,28 +69,19 @@ where U: IntoIterator<Item=V>,
letstderr=std::process::Stdio::inherit();
letstderr=std::process::Stdio::inherit();
}
}
};
};
letmutchild=Command::new(process);
letchild=child
letmutchild=matchCommand::new(process)
.args(process_args.iter_cloned().map(|x|x.into_owned()))//Do we need `into_owned()` here?
.args(process_args.iter_cloned().map(|x|x.into_owned()))//Do we need `into_owned()` here?
.args(args.into_iter())
.args(args)
.stdout(std::process::Stdio::piped())
.stdout(std::process::Stdio::piped())
.stderr(stderr)
.stderr(stderr)
.stdin(std::process::Stdio::null())
.stdin(std::process::Stdio::null())
.kill_on_drop(false);
.spawn(){
Ok(chi)=>chi,
// SAFETY: This is only calling `setpgid()`.
Err(sp)=>{
unsafe{
returnErr(Error::Spawning(sp));
child
}
.pre_exec(set_process_group(0))// NOTE: .process_group() does not exist in this version, so we hack it together outselves and call `setpgid()` directly
shutdown_tx.broadcast(Some(res.clone())).expect("Failed to communicate worker shutdown with waiters");
shutdown_tx.broadcast(Some(res.clone())).expect("Failed to communicate worker shutdown with waiters");
res
res
});
});
handle
handle
@ -543,22 +512,6 @@ pub enum Error
Unknown,
Unknown,
}
}
implError
{
/// If this error comes from an attempt to communicate with the worker that failed but can be ignored.
///
/// # Usage
/// If the worker has been gracefully shutdown, and background tasks are still running that want to update the progress, an error will be returned when those already extant tasks attempt to send the commands to do so.
/// Those tasks can ignore those errors entirely if this function returns `true`.
