-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Nodemon not restarting when using node-java #573
Comments
+1 |
Firstly, I've no clue what On Mon, 28 Sep 2015 18:15 Adam Urban [email protected] wrote:
|
+1 happens with Some clues what might be happening: Using I managed to correct the behavior in my case by adding: process.once('SIGUSR2', function() {
process.kill(process.pid, 'SIGUSR2')
}) This needs to be run after I import Also, I tried to quit my code on receiving process.once('SIGUSR2', function() {
process.exit() // status code 0 or 1 ?
}) but such ending of the process does not cause the |
seems that SIGUSR2 is never called when node-java is there |
@dannyn-mirth I would suggest that this is closed by #1061. As @tomaskulich correctly pointed out the issue is with the signal being passed to node-java. If you use the |
@dannyn-mirth i switched to pm2 (with pm2-dev) for projects with node-java - works absolutly fine - tried every signal from nodemon without luck... cuz node-java seems to need the signal multiple times or sth like dis cya |
This issue has been automatically marked as idle and stale because it hasn't had any recent activity. It will be automtically closed if no further activity occurs. If you think this is wrong, or the problem still persists, just pop a reply in the comments and @remy will (try!) to follow up. |
I posted this issue over at the node-java project: joeferner/node-java#226
In a nutshell, when using node-java, nodemon doesn't start the server after it detects file changes. I thought this was a node-java issue but when I tried out supervisor, it was able to restart the server as it should.
The text was updated successfully, but these errors were encountered: