Skip to content

Commit

Permalink
Hide "Done compiling" messages if super shell is enabled
Browse files Browse the repository at this point in the history
Fixes sbt/sbt#3939
Ref #635

Super shell will be enabled in sbt, but it's an opt-in for Zinc, so this won't have any effect if you're using Zinc as a library.
  • Loading branch information
eed3si9n committed Mar 22, 2019
1 parent 52286d0 commit 29c4cdb
Showing 1 changed file with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ import sbt.internal.inc.JavaInterfaceUtil.EnrichOption
import sbt.internal.inc.caching.ClasspathCache
import sbt.internal.inc.javac.AnalyzingJavaCompiler
import xsbti.compile.{ ClassFileManager => XClassFileManager }
import sbt.internal.util.ConsoleAppender

/** An instance of an analyzing compiler that can run both javac + scalac. */
final class MixedAnalyzingCompiler(
Expand Down Expand Up @@ -137,8 +138,10 @@ final class MixedAnalyzingCompiler(
compileScala(); compileJava()
}

if (javaSrcs.size + scalaSrcs.size > 0)
log.info("Done compiling.")
if (javaSrcs.size + scalaSrcs.size > 0) {
if (ConsoleAppender.showProgress) log.debug("Done compiling.")
else log.info("Done compiling.")
}
}

private def putJavacOutputInJar(outputJar: File, outputDir: File): Unit = {
Expand Down

0 comments on commit 29c4cdb

Please sign in to comment.