Avoid unnecessary dependency downloading by providing fetches per cache policy (#494)

* Avoid unnecessary dependency downloading by providing fetches per cache policy; add ticker logging when they are downloading

* Fix GenIdeaTests by making the Log context Option[]al

* Add some comments

* Rebase and resolve

1 files changed, 9 insertions, 3 deletions

diff --git a/main/core/src/eval/Evaluator.scala b/main/core/src/eval/Evaluator.scala
index dbaf9433..71fe85a9 100644
--- a/main/core/src/eval/Evaluator.scala
+++ b/main/core/src/eval/Evaluator.scala
@@ -251,7 +251,7 @@ case class Evaluator(home: os.Path,
 
     val nonEvaluatedTargets = group.indexed.filterNot(results.contains)
 
-    maybeTargetLabel.foreach { targetLabel =>
+    val tickerPrefix = maybeTargetLabel.map { targetLabel =>
       val inputResults = for {
         target <- nonEvaluatedTargets
         item <- target.inputs.filterNot(group.contains)
@@ -259,10 +259,16 @@ case class Evaluator(home: os.Path,
 
       val logRun = inputResults.forall(_.isInstanceOf[Result.Success[_]])
 
-      if(logRun) { log.ticker(s"[$counterMsg] $targetLabel ") }
+      val prefix = s"[$counterMsg] $targetLabel "
+      if(logRun) log.ticker(prefix)
+      prefix + "| "
     }
 
-    val multiLogger = resolveLogger(paths.map(_.log))
+    val multiLogger = new ProxyLogger(resolveLogger(paths.map(_.log))) {
+      override def ticker(s: String): Unit = {
+        super.ticker(tickerPrefix.getOrElse("")+s)
+      }
+    }
     var usedDest = Option.empty[(Task[_], Array[StackTraceElement])]
     for (task <- nonEvaluatedTargets) {
       newEvaluated.append(task)