Mirrors/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell synced 2024-12-25 12:23:09 +00:00
Code Issues Projects Releases Packages Wiki Activity

Temporary workaround for BSD WEXITSTATUS libc bug

Browse source 
The libc crate has a bug on BSD where WEXITSTATUS is not an 8-bit
value, causing assertion failures.

Any libc higher than our 0.2.155 would increase our MSRV, see libc
commit 5ddbdc29f (Bump MSRV to 1.71, 2024-01-07), so we want to
woraround this anyway.  It's probably not worth using a patched
version of libc since it's just one line.

While at it, tighten some types I guess.

Upstream fix: https://github.com/rust-lang/libc/pull/4213

Closes #10919
This commit is contained in:
Johannes Altmanninger 2024-12-19 07:57:03 +01:00
parent 5de6f4bb3d
commit c1b460525c
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/proc.rs
View file

@ -233,14 +233,14 @@ impl ProcStatus {
}
/// Return the exit code, given that we normal exited.
pub fn exit_code(&self) -> libc::c_int {
pub fn exit_code(&self) -> u8 {
assert!(self.normal_exited(), "Process is not normal exited");
WEXITSTATUS(self.status())
u8::try_from(WEXITSTATUS(self.status()) & 0xff).unwrap() // Workaround for libc bug
}
/// Return if this status represents success.
pub fn is_success(&self) -> bool {
self.normal_exited() && self.exit_code() == EXIT_SUCCESS
self.normal_exited() && self.exit_code() == u8::try_from(EXIT_SUCCESS).unwrap()
}
/// Return the value appropriate to populate $status.
@ -248,7 +248,7 @@ impl ProcStatus {
if self.signal_exited() {
128 + self.signal_code()
} else if self.normal_exited() {
self.exit_code()
i32::from(self.exit_code())
} else {
panic!("Process is not exited")
}