avoid printing useless stacktraces when subprocess runs fail

1 files changed, 8 insertions, 4 deletions

diff --git a/scalalib/src/mill/scalalib/JavaModule.scala b/scalalib/src/mill/scalalib/JavaModule.scala
index 3134851e..0a396a53 100644
--- a/scalalib/src/mill/scalalib/JavaModule.scala
+++ b/scalalib/src/mill/scalalib/JavaModule.scala
@@ -250,14 +250,16 @@ trait JavaModule extends mill.Module with TaskModule { outer =>
   }
 
   def run(args: String*) = T.command{
-    Jvm.interactiveSubprocess(
+    try Result.Success(Jvm.interactiveSubprocess(
       finalMainClass(),
       runClasspath().map(_.path),
       forkArgs(),
       forkEnv(),
       args,
       workingDir = ammonite.ops.pwd
-    )
+    )) catch { case e: InteractiveShelloutException =>
+       Result.Failure("subprocess failed")
+    }
   }
 
 
@@ -270,14 +272,16 @@ trait JavaModule extends mill.Module with TaskModule { outer =>
   }
 
   def runMain(mainClass: String, args: String*) = T.command{
-    Jvm.interactiveSubprocess(
+    try Result.Success(Jvm.interactiveSubprocess(
       mainClass,
       runClasspath().map(_.path),
       forkArgs(),
       forkEnv(),
       args,
       workingDir = ammonite.ops.pwd
-    )
+    )) catch { case e: InteractiveShelloutException =>
+      Result.Failure("subprocess failed")
+    }
   }
 
   // publish artifact with name "mill_2.12.4" instead of "mill_2.12"