JS-2148 Don't kill -STOP under Windows

sos-berlin · Aug 31, 2024 · 1c1aa3f · 1c1aa3f
1 parent 0a15879
commit 1c1aa3f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/js7-launcher/src/main/scala/js7/launcher/processkiller/ProcessKiller.scala b/js7-launcher/src/main/scala/js7/launcher/processkiller/ProcessKiller.scala
@@ -8,6 +8,7 @@ import js7.base.io.process.Processes.*
 import js7.base.io.process.{JavaProcess, Js7Process, Pid, ReturnCode}
 import js7.base.log.Logger
 import js7.base.log.Logger.syntax.*
+import js7.base.system.OperatingSystem.isUnix
 import js7.base.thread.IOExecutor.env.interruptibleVirtualThread
 import js7.base.time.ScalaTime.*
 import js7.base.utils.Assertions.assertThat
@@ -130,8 +131,9 @@ private[launcher] trait ProcessKiller[P <: Pid | Js7Process]:
           (isAlive(process) ? process.toPid.number) ++: descendants.map(_.pid)
 
         // Try to stop all processes, then kill them
-        trySendStopSignal(stopPids) *>
-          sigkillAll(processToDescendants)
+        IO.whenA(isUnix):
+          trySendStopSignal(stopPids)
+        *> sigkillAll(processToDescendants)
 
   /** Return a Seq of distinct ProcessHandles */
   private def descendantsOf(processHandles: Seq[ProcessHandle]): Seq[ProcessHandle] =