Speed up lsof by avoiding unneeded work

Tyriar · Tyriar · commit c7e72d7fded5 · 2019-10-26T08:53:44.000-07:00
Fixes microsoft#78438
diff --git a/src/vs/workbench/contrib/terminal/node/terminalProcess.ts b/src/vs/workbench/contrib/terminal/node/terminalProcess.ts
@@ -260,7 +260,7 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess
 					return;
 				}
 				this._logService.trace('IPty#pid');
-				exec('lsof -p ' + this._ptyProcess.pid + ' | grep cwd', (error, stdout, stderr) => {
+				exec('lsof -OPl -p ' + this._ptyProcess.pid + ' | grep cwd', (error, stdout, stderr) => {
 					if (stdout !== '') {
 						resolve(stdout.substring(stdout.indexOf('/'), stdout.length - 1));
 					}

Original file line number	Diff line number	Diff line change
`@@ -260,7 +260,7 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess`
`260`	`260`	`return;`
`261`	`261`	`}`
`262`	`262`	`this._logService.trace('IPty#pid');`
`263`		`- exec('lsof -p ' + this._ptyProcess.pid + ' \| grep cwd', (error, stdout, stderr) => {`
	`263`	`+ exec('lsof -OPl -p ' + this._ptyProcess.pid + ' \| grep cwd', (error, stdout, stderr) => {`
`264`	`264`	`if (stdout !== '') {`
`265`	`265`	`resolve(stdout.substring(stdout.indexOf('/'), stdout.length - 1));`
`266`	`266`	`}`