diff options
author | 2022-10-23 00:41:56 +0800 | |
---|---|---|
committer | 2022-10-22 09:41:56 -0700 | |
commit | 55c42f166332b143649b5770ae9f9c48104c1ea0 (patch) | |
tree | 02d326f96f9bf1e46146bd85ee7edaf75ca30d73 /src/bun.js | |
parent | 85808700c8a89f82cfce55330a2c8c592a5623d2 (diff) | |
download | bun-55c42f166332b143649b5770ae9f9c48104c1ea0.tar.gz bun-55c42f166332b143649b5770ae9f9c48104c1ea0.tar.zst bun-55c42f166332b143649b5770ae9f9c48104c1ea0.zip |
Fix spawn exitcode (#1371)
Diffstat (limited to 'src/bun.js')
-rw-r--r-- | src/bun.js/api/bun/subprocess.zig | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/bun.js/api/bun/subprocess.zig b/src/bun.js/api/bun/subprocess.zig index 638c8ffef..467a3a842 100644 --- a/src/bun.js/api/bun/subprocess.zig +++ b/src/bun.js/api/bun/subprocess.zig @@ -1177,7 +1177,7 @@ pub const Subprocess = struct { subprocess.finalize(); const sync_value = JSC.JSValue.createEmptyObject(globalThis, 4); - sync_value.put(globalThis, JSC.ZigString.static("exitCode"), JSValue.jsNumber(@intCast(i32, exitCode) * -1)); + sync_value.put(globalThis, JSC.ZigString.static("exitCode"), JSValue.jsNumber(@intCast(i32, exitCode))); sync_value.put(globalThis, JSC.ZigString.static("stdout"), stdout); sync_value.put(globalThis, JSC.ZigString.static("stderr"), stderr); sync_value.put(globalThis, JSC.ZigString.static("success"), JSValue.jsBoolean(exitCode == 0)); @@ -1203,6 +1203,13 @@ pub const Subprocess = struct { }, .result => |status| { this.exit_code = @truncate(u8, status.status); + // Do WEXITSTATUS macro check: https://linux.die.net/man/3/waitpid + // https://code.woboq.org/gtk/include/bits/waitstatus.h.html + const w_if_exited = (status.status & 0x7f) == 0; + if (w_if_exited) { + const w_exit_status = (status.status & 0xff00) >> 8; + this.exit_code = @truncate(u8, w_exit_status); + } }, } |