mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-15 05:12:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

polyfills: fix Bun.spawn stdin cleanup

Browse Source
This commit is contained in:
jhmaster2000
2023-11-30 23:57:10 -03:00
parent 19cee77e8b
commit 2d002297ce
2 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/bun-polyfills/src/global/process.ts
View File

@@ -16,7 +16,7 @@ if (typeof process === 'object' && process !== null) {
process.versions.c_ares = '0e7a5dee0fbb04080750cf6eabbe89d8bae87faa' satisfies Process['versions'][string];
process.versions.zig = '0.12.0-dev.1604+caae40c21' satisfies Process['versions'][string];
process.versions.bun = '1.0.13' satisfies Process['versions'][string];
Reflect.set(process, 'revision', 'ee140933258ea9bf72cfa0fdc1cd998c15680bc7' satisfies Process['revision']);
Reflect.set(process, 'revision', '19cee77e8b063917b04d1fb01b929890a4670741' satisfies Process['revision']);
/** @end_generated_code */
// Doesn't work on Windows sadly

11
packages/bun-polyfills/src/modules/bun.ts
View File

@@ -48,7 +48,7 @@ export const main = path.resolve(process.cwd(), process.argv[1] ?? 'repl') satis
//? These are automatically updated on build by tools/updateversions.ts, do not edit manually.
export const version = '1.0.13' satisfies typeof Bun.version;
export const revision = 'ee140933258ea9bf72cfa0fdc1cd998c15680bc7' satisfies typeof Bun.revision;
export const revision = '19cee77e8b063917b04d1fb01b929890a4670741' satisfies typeof Bun.revision;
export const gc = (globalThis.gc ? (() => (globalThis.gc!(), process.memoryUsage().heapUsed)) : (() => {
const err = new Error('[bun-polyfills] Garbage collection polyfills are only available when Node.js is ran with the --expose-gc flag.');
@@ -270,8 +270,7 @@ export const spawn = ((...args) => {
if (opts.stdio[0] && typeof opts.stdio[0] !== 'string') {
stdinSrc = opts.stdio[0];
stdio[0] = 'pipe';
}
} else stdio[0] = opts.stdio[0];
const subp = chp.spawn(cmd, argv, {
cwd: opts.cwd ?? process.cwd(),
// why is this set to (string | number) on env values...
@@ -299,12 +298,12 @@ export const spawn = ((...args) => {
let internalStdinStream: streams.Writable;
if (subpAsNode.stdin) {
const wstream = subpAsNode.stdin;
Reflect.set(wstream, 'destroy', function (this: NodeJS.WritableStream, err?: Error) {
internalStdinStream = wstream;
(<Mutable<Subprocess>>subp).stdin = new FileSink(wstream);
Reflect.set(subp.stdin as FileSink, 'destroy', function (this: NodeJS.WritableStream, err?: Error) {
void this.end(); /* if it fails its already closed */
return this;
});
internalStdinStream = wstream;
(<Mutable<Subprocess>>subp).stdin = new FileSink(wstream);
}
Object.defineProperty(subp, 'readable', { get(this: Subprocess) { return this.stdout; } });