From 7fead66244bd428e611849b63c891ae2e5ee369c Mon Sep 17 00:00:00 2001 From: Grzegorz Kossakowski Date: Thu, 10 Apr 2014 15:24:07 +0200 Subject: [PATCH] Pick the right overload in EvaluateTask.runTask The deprecated method should forward to the other overloaded alternative but it recursed instead. This kind of mistake would be easily caught by linter warning about unused `newConfig` local variable. I hope we'll get there some day. Fixes #1251 --- main/src/main/scala/sbt/EvaluateTask.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main/src/main/scala/sbt/EvaluateTask.scala b/main/src/main/scala/sbt/EvaluateTask.scala index c7df90b96..0ea0a4f9f 100644 --- a/main/src/main/scala/sbt/EvaluateTask.scala +++ b/main/src/main/scala/sbt/EvaluateTask.scala @@ -292,7 +292,7 @@ object EvaluateTask def runTask[T](root: Task[T], state: State, streams: Streams, triggers: Triggers[Task], config: EvaluateConfig)(implicit taskToNode: NodeView[Task]): (State, Result[T]) = { val newConfig = EvaluateTaskConfig(config) - runTask(root, state, streams, triggers, config)(taskToNode) + runTask(root, state, streams, triggers, newConfig)(taskToNode) } def runTask[T](root: Task[T], state: State, streams: Streams, triggers: Triggers[Task], config: EvaluateTaskConfig)(implicit taskToNode: NodeView[Task]): (State, Result[T]) = {