Merge pull request microsoft#182904 from microsoft/merogge/pty-kill

meganrogge · web-flow · commit 01680658ad56 · 2023-05-18T12:41:50.000-07:00
revert change to terminal process kill signal
diff --git a/src/vs/platform/terminal/node/terminalProcess.ts b/src/vs/platform/terminal/node/terminalProcess.ts
@@ -375,7 +375,7 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess
 			if (this._ptyProcess) {
 				await this._throttleKillSpawn();
 				this._logService.trace('IPty#kill');
-				this._ptyProcess.kill(!isWindows ? 'SIGKILL' : undefined);
+				this._ptyProcess.kill();
 			}
 		} catch (ex) {
 			// Swallow, the pty has already been killed

Original file line number	Diff line number	Diff line change
`@@ -375,7 +375,7 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess`
`375`	`375`	`if (this._ptyProcess) {`
`376`	`376`	`await this._throttleKillSpawn();`
`377`	`377`	`this._logService.trace('IPty#kill');`
`378`		`- this._ptyProcess.kill(!isWindows ? 'SIGKILL' : undefined);`
	`378`	`+ this._ptyProcess.kill();`
`379`	`379`	`}`
`380`	`380`	`} catch (ex) {`
`381`	`381`	`// Swallow, the pty has already been killed`