Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrated Terminal createNew command throws an error on Insiders #11894

Closed
zet4 opened this issue Sep 12, 2016 · 3 comments
Closed

Integrated Terminal createNew command throws an error on Insiders #11894

zet4 opened this issue Sep 12, 2016 · 3 comments
Assignees
Labels
api bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority terminal Integrated terminal issues verified Verification succeeded

Comments

@zet4
Copy link

zet4 commented Sep 12, 2016

  • VSCode Version: 1.6.0-insider 52bceb1
  • OS Version: Windows 10 Pro

Steps to Reproduce:

  1. Open the terminal (using Ctrl + ` or otherwise.)
  2. Kill the terminal.
  3. Attempt to open it again, notice that cmd does not start again.

Notes:

This behavior was noticed originally while trying to re-run a python script using Python extension, further testing showed the issue is only with nightly as it doesn't occur on stable.

Console:

Upon closing first terminal:

index.js:23 [uncaught exception]: Error: IPC channel is already disconnectedonError @ index.js:23(anonymous function) @ index.js:90emitOne @ events.js:90emit @ events.js:182window.onerror @ C:\Program Files (x86)\Microsoft VS Code Insiders\resources\electron.asar\renderer\init.js:107i.globals.onerror @ errorTelemetry.ts:64
index.js:26 Error: IPC channel is already disconnected
    at ChildProcess.target.disconnect (internal/child_process.js:660)
    at Pipe.channel.onread (internal/child_process.js:453)onError @ index.js:26(anonymous function) @ index.js:90emitOne @ events.js:90emit @ events.js:182window.onerror @ C:\Program Files (x86)\Microsoft VS Code Insiders\resources\electron.asar\renderer\init.js:107i.globals.onerror @ errorTelemetry.ts:64
events.js:154 Uncaught Error: IPC channel is already disconnectedtarget.disconnect @ internal/child_process.js:660channel.onread @ internal/child_process.js:453

Further attempts to open terminal:

threadService.ts:217 [Extension Host] WARNING: Promise with no error callback:62
threadService.ts:217 [Extension Host] Object {exception: null, error: Object, promise: Object, id: 62}
shell.ts:424 Cannot set property 'name' of undefined: TypeError: Cannot set property 'name' of undefined
    at t.createNew (file:///C:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:96:15100)
    at t.$createTerminal (file:///C:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:105:7710)
    at t.e.handle (file:///C:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:104:4494)
    at s (file:///C:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:50:9560)
    at f (file:///C:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:50:10237)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)e.onUnexpectedError @ shell.ts:424(anonymous function) @ shell.ts:337e.onUnexpectedError @ errors.ts:68r @ errors.ts:83t.onUnexpectedExtHostError @ mainThreadErrors.ts:13e.handle @ abstractThreadService.ts:34s @ ipcRemoteCom.ts:269f @ ipcRemoteCom.ts:226_combinedTickCallback @ internal/process/next_tick.js:67_tickCallback @ internal/process/next_tick.js:98
@ramya-rao-a ramya-rao-a added the terminal Integrated terminal issues label Sep 12, 2016
@Tyriar
Copy link
Member

Tyriar commented Sep 12, 2016

/cc @DonJayamanne

@Tyriar Tyriar added this to the September 2016 milestone Sep 12, 2016
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug api important Issue identified as high-priority labels Sep 12, 2016
@Tyriar Tyriar changed the title Integrated Terminal fails to launch after first run. Integrated Terminal createNew command throws an error on Insiders Sep 12, 2016
@Tyriar
Copy link
Member

Tyriar commented Sep 12, 2016

Repro'd myself on Insiders using ctrl+shift+backtick, likely caused by #11700 /cc @octref - should be fixed in tomorrow's Insiders build.

@Tyriar
Copy link
Member

Tyriar commented Sep 12, 2016

Will be fixed when #11839 is merged.

@Tyriar Tyriar closed this as completed Sep 13, 2016
@sandy081 sandy081 added the verified Verification succeeded label Sep 30, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority terminal Integrated terminal issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants