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

In v1.3.0, cannot kill child process when using control-c in Windows #80

Open
wlingke opened this issue Oct 28, 2014 · 10 comments
Open

Comments

@wlingke
Copy link

wlingke commented Oct 28, 2014

I am working in a Windows environment. In 1.2.3, when you hit control-c to exit the forever-monitor process, it also killed the child process. However, in 1.3.0, when you hit control-c, you only kill the forever-monitor process but the child process remains.

@wlingke wlingke changed the title 1.3.0 cannot kill child process when using control-c in Windows In v1.3.0, cannot kill child process when using control-c in Windows Oct 28, 2014
@wlingke
Copy link
Author

wlingke commented Oct 28, 2014

Setting killTree: true also doesn't solve this issue

@matthewrobb
Copy link

Any changes with this?

@wlingke
Copy link
Author

wlingke commented Mar 13, 2015

haven't heard anything... staying on 1.2.3 til otherwise

@matthewrobb
Copy link

@wlingke Do you do anything to get the stdout piped to the main proc? Using inherit breaks with the older versions.

@wlingke
Copy link
Author

wlingke commented Mar 13, 2015

The stdout pipes normally to the main process for me. Not sure what you
mean by using inherit

Best,
Lingke

Lingke Wang
Stanford GSB '16

On Fri, Mar 13, 2015 at 8:20 AM, Matthew Robb notifications@github.com
wrote:

@wlingke https://github.com/wlingke Do you do anything to get the
stdout piped to the main proc? Using inherit breaks with the older versions.


Reply to this email directly or view it on GitHub
#80 (comment)
.

@wlingke
Copy link
Author

wlingke commented Aug 5, 2015

any luck on this issue? This still persists in the latest forever monitor

@CaleyD
Copy link

CaleyD commented Sep 10, 2015

I ran into the same problem. It looks like killTree was explicitly disabled for Windows.
See 8dd9ba1 and #52

@wlingke
Copy link
Author

wlingke commented Sep 10, 2015

good job on finding issue. I'm not sure why it was disabled. The command used to work in 1.2.x

@v3xx3d
Copy link

v3xx3d commented Feb 6, 2016

Does anyone know how to actually fix this? Just disabling it for windows doesn't seem to resolve anything other than hiding a known issue.

@boysmovie
Copy link

What I did was to setup the SIGINT event and when Control-C is triggered just call child.stop()

process.on('SIGINT', function() {
    child.stop();
    process.exit();
});

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants