dont use signal to verify daemon has been killed under windows (#2484)

lib/Client.js

@@ -493,18 +493,33 @@ Client.prototype.killDaemon = function killDaemon(fn) {
     });
   }
 
-  process.once('SIGQUIT', function() {
-    debug('Received SIGQUIT');
-    clearTimeout(timeout);
-    quit();
-  });
+  // under unix, we listen for signal (that is send by daemon to notify us that its shuting down)
+  if (process.platform !== 'win32' && process.platform !== 'win64') {
+    process.once('SIGQUIT', function() {
+      debug('Received SIGQUIT from pm2 daemon');
+      clearTimeout(timeout);
+      quit();
+    });
+  }
 
   timeout = setTimeout(function() {
-    quit();
+      quit();
   }, 3000);
 
   // Kill daemon
-  this.executeRemote('killMe', {pid : process.pid}, function() {});
+  this.executeRemote('killMe', {pid : process.pid});
+
+  // if under windows, try to ping the daemon to see if it still here
+  if (process.platform === 'win32' || process.platform === 'win64') {
+    setTimeout(function() {
+      that.pingDaemon(function(alive) {
+        if (!alive) {
+          clearTimeout(timeout);
+          return quit();
+        }
+      });
+    }, 250)
+  }
 };
 
 

lib/Daemon.js

@@ -228,12 +228,9 @@ Daemon.prototype.close = function(opts, cb) {
     that.pub_socket.close(function() {
       console.log('PUB closed');
 
-      var kill_signal = 'SIGQUIT';
-
-      if (process.platform === 'win32' || process.platform === 'win64') {
-        kill_signal = 'SIGTERM';
-      }
-      process.kill(parseInt(opts.pid), kill_signal);
+      // notify cli that the daemon is shuting down (only under unix since windows doesnt handle signals)
+      if (process.platform !== 'win32' && process.platform !== 'win64') 
+        process.kill(parseInt(opts.pid), 'SIGQUIT');
 
       console.log('PM2 successfully stopped');
       setTimeout(function() {