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

Stray processes left when closing the main window #7527

Closed
semihalf-kardach-stanislaw opened this issue Jun 10, 2016 · 4 comments
Closed

Stray processes left when closing the main window #7527

semihalf-kardach-stanislaw opened this issue Jun 10, 2016 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug
Milestone

Comments

@semihalf-kardach-stanislaw

Steps to Reproduce:

  1. In terminal run code.
  2. (sometimes optional) enable built-in terminal and run ls in it
  3. Close the window
  4. Result:
kda@toster ~ $ code
# launch terminal, do something in it and close the window
kda@toster ~/workspace/tmp $ code
kda@toster ~/workspace/tmp $ pgrep -la code
31925 /usr/share/code/code
31928 /usr/share/code/code --type=zygote --no-sandbox
31999 /usr/share/code/code /usr/share/code/resources/app/out/bootstrap --type=SharedProcess
32179 /usr/share/code/code ./terminalProcess

I've captured the strace for the code process with:
strace -f -o vscode.strace.log code > vscode.stdout.log 2> vscode.stderr.log

Result is attached.
vscode.strace.log.zip

@alexdima
Copy link
Member

@joaomoreno for the shared process, @Tyriar for the terminal process

@joaomoreno joaomoreno added the bug Issue identified by VS Code Team member as probable bug label Jun 10, 2016
@Tyriar
Copy link
Member

Tyriar commented Jun 10, 2016

Terminal process already fixed in #7458

@Tyriar Tyriar removed their assignment Jun 10, 2016
@joaomoreno
Copy link
Member

This is strange. The shared process checks for its parent. And now so does the terminal process. If that doesn't work as we think it does... both are broken.

@joaomoreno joaomoreno added this to the June 2016 milestone Jun 13, 2016
joaomoreno added a commit that referenced this issue Jun 15, 2016
@joaomoreno joaomoreno modified the milestones: Backlog, June 2016 Jun 17, 2016
@joaomoreno
Copy link
Member

This should be fixed in the later insiders: https://code.visualstudio.com/insiders

@semihalf-kardach-stanislaw Let us know if otherwise.

@joaomoreno joaomoreno assigned Tyriar and unassigned joaomoreno Jul 6, 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
bug Issue identified by VS Code Team member as probable bug
Projects
None yet
Development

No branches or pull requests

4 participants