zombie process either by process.kill or pty.destroy

[hayeah]

zombie process either by process.kill or pty.destroy

[hhuuggoo]

I think the problem is chjj/tty.js#111, basically you need to signal(SIGCHLD, SIG_INT), though I have no idea how to do that in node.js, process.on seems to require a function as the second parameter.

the following code works, however it seems to be a bit overkill

Terminal.prototype.kill = function(sig) {
  try {
    process.kill(this.pid, sig || 'SIGTERM');
    waitpid(this.pid);
  } catch(e) {
    ;
  }
};

where waitpid is https://www.npmjs.org/package/waitpid

[chjj]

Yes, it's most likely due to the non-handling of SIGCHLD (which is what a zombie is: a child process whos exit never got read by waitpid after a sigchld). Adding our own SIGCHLD handler may conflict with the libuv SIGCHLD handler. It's still in the works. I may just merge it to master and use it for a while to see if there are any major problems.

SIGCHLD discussion: #37

[moul]

Any update on this ?

[EloB]

+1

[howardroark]

👍 :)