[8406] ProcessBuilderImpl catches exceptions thrown on background thread

[8406] Adds more caught exceptions in Simple ProcessBuilder#run

src/library/scala/sys/process/ProcessBuilderImpl.scala

@@ -68,7 +68,17 @@ private[process] trait ProcessBuilderImpl {
   /** Represents a simple command without any redirection or combination. */
   private[process] class Simple(p: JProcessBuilder) extends AbstractBuilder {
     override def run(io: ProcessIO): Process = {
-      val process = p.start() // start the external process
+      val process = try {
+        p.start() // start the external process
+      } catch {
+        case _: IndexOutOfBoundsException
+             | _: IOException
+             | _: NullPointerException
+             | _: SecurityException
+             | _: UnsupportedOperationException
+        => return FailedProcess
+      }
+
       import io._
 
       // spawn threads that process the input, output, and error streams using the functions defined in `io`

src/library/scala/sys/process/ProcessImpl.scala

@@ -255,4 +255,10 @@ private[process] trait ProcessImpl {
     override def exitValue() = if (success.get) 0 else 1   // thread.join()
     override def destroy()   = thread.interrupt()
   }
+
+  private[process] object FailedProcess extends Process {
+    override def isAlive() = false
+    override def exitValue() = 1
+    override def destroy(): Unit = ()
+  }
 }

test/junit/scala/sys/process/ProcessBuilderTest.scala

@@ -0,0 +1,17 @@
+package scala.sys.process
+
+import org.junit.runner.RunWith
+import org.junit.runners.JUnit4
+import org.junit.Test
+
+@RunWith(classOf[JUnit4])
+class ProcessBuilderTest {
+
+  @Test
+  def t8406: Unit = {
+    import java.io.ByteArrayInputStream
+    import scala.util.Try
+    assert(Try(("does-not-exist" #< new ByteArrayInputStream("foo".getBytes("utf-8"))).lazyLines).isSuccess)
+    assert(Try(("does-not-exist" #< new ByteArrayInputStream("foo".getBytes("utf-8"))).lazyLines.toList).isFailure)
+  }
+}