Skip to content

Commit 9130ebd

Browse files
committed
Ensure pty host only adds --inspect when a port is set
Fixes microsoft#185033
1 parent e209a1c commit 9130ebd

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

src/vs/platform/terminal/electron-main/electronPtyHostStarter.ts

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,10 @@ export class ElectronPtyHostStarter implements IPtyHostStarter {
4242
this.utilityProcess = new UtilityProcess(this._logService, NullTelemetryService, this._lifecycleMainService);
4343

4444
const inspectParams = parsePtyHostDebugPort(this._environmentService.args, this._environmentService.isBuilt);
45-
let execArgv: string[] | undefined = undefined;
46-
if (inspectParams) {
47-
execArgv = ['--nolazy'];
48-
if (inspectParams.break) {
49-
execArgv.push(`--inspect-brk=${inspectParams.port}`);
50-
} else if (!inspectParams.break) {
51-
execArgv.push(`--inspect=${inspectParams.port}`);
52-
}
53-
}
45+
const execArgv = inspectParams.port ? [
46+
'--nolazy',
47+
`--inspect${inspectParams.break ? '-brk' : ''}=${inspectParams.port}`
48+
] : undefined;
5449

5550
this.utilityProcess.start({
5651
type: 'ptyHost',

0 commit comments

Comments
 (0)