Terminate pipe processes instead of sending them SIGINTs

Author: kmvanbrunt

cmd2/cmd2.py

@@ -1655,8 +1655,8 @@ def sigint_handler(self, signum: int, frame) -> None:
         :param frame
         """
         try:
-            # Forward the sigint to the current pipe process
-            self.pipe_proc_reader.send_sigint()
+            # Terminate the current pipe process
+            self.pipe_proc_reader.terminate()
         except AttributeError:
             # Ignore since self.pipe_proc_reader was None
             pass
@@ -1920,7 +1920,7 @@ def _redirect_output(self, statement: Statement) -> Tuple[bool, RedirectionSaved
             # We want Popen to raise an exception if it fails to open the process.  Thus we don't set shell to True.
             try:
                 # Set options to not forward signals to the pipe process. If a Ctrl-C event occurs,
-                # our sigint handler will forward it to the most recent pipe process. This makes sure
+                # our sigint handler terminate the most recent pipe process. This makes sure
                 # pipe processes close in the right order (most recent first).
                 if sys.platform == 'win32':
                     creationflags = subprocess.CREATE_NEW_PROCESS_GROUP

cmd2/utils.py

@@ -404,10 +404,10 @@ def __init__(self, proc: subprocess.Popen, stdout: Union[StdSim, BinaryIO, TextI
         if self._proc.stderr is not None:
             self._err_thread.start()
 
-    def send_sigint(self) -> None:
-        """Send a SIGINT to the process"""
+    def terminate(self) -> None:
+        """Terminate the process"""
         import signal
-        self._proc.send_signal(signal.SIGINT)
+        self._proc.terminate()
 
     def wait(self) -> None:
         """Wait for the process to finish"""