Merge pull request #225 from berdario/terminate_subprocesses

Terminate all subprocesses

src/watchdog/tricks/__init__.py

@@ -17,6 +17,7 @@
 # limitations under the License.
 
 
+import os
 import signal
 import subprocess
 import time
@@ -131,7 +132,7 @@ def __init__(self, command, patterns=None, ignore_patterns=None,
                  kill_after=10):
         super(AutoRestartTrick, self).__init__(
             patterns, ignore_patterns, ignore_directories)
-        self.command = command
+        self.command = ['setsid'] + command
         self.stop_signal = stop_signal
         self.kill_after = kill_after
         self.process = None
@@ -143,7 +144,7 @@ def stop(self):
         if self.process is None:
             return
         try:
-            self.process.send_signal(self.stop_signal)
+            os.killpg(os.getpgid(self.process.pid), self.stop_signal)
         except OSError:
             # Process is already gone
             pass
@@ -155,7 +156,7 @@ def stop(self):
                 time.sleep(0.25)
             else:
                 try:
-                    self.process.kill()
+                    os.killpg(os.getpgid(self.process.pid), 9)
                 except OSError:
                     # Process is already gone
                     pass