Fix overwrite default to true in deprecated method calls.

We cannot break existing users, but we can deprecate the improper usage. This is part #2 of the workaround for #1156. This ensures that users will stop using the legacy methods after 0.13.2 is out.
sbt · Mar 21, 2014 · 7e77d16 · 7e77d16
1 parent 3e9a7c7
commit 7e77d16
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/main/src/main/scala/sbt/Defaults.scala b/main/src/main/scala/sbt/Defaults.scala
@@ -1213,8 +1213,9 @@ object Classpaths
 
 	def deliverConfig(outputDirectory: File, status: String = "release", logging: UpdateLogging.Value = UpdateLogging.DownloadOnly) =
 	    new DeliverConfiguration(deliverPattern(outputDirectory), status, None, logging)
+	@deprecated("0.13.2", "Previous semantics allowed overwriting cached files, which was unsafe.  Please specify overwrite parameter.")
 	def publishConfig(artifacts: Map[Artifact, File], ivyFile: Option[File], checksums: Seq[String], resolverName: String, logging: UpdateLogging.Value): PublishConfiguration =
-		publishConfig(artifacts, ivyFile, checksums, resolverName, logging, overwrite = false)
+		publishConfig(artifacts, ivyFile, checksums, resolverName, logging, overwrite = true)
 	def publishConfig(artifacts: Map[Artifact, File], ivyFile: Option[File], checksums: Seq[String], resolverName: String = "local", logging: UpdateLogging.Value = UpdateLogging.DownloadOnly, overwrite: Boolean = false) =
 		new PublishConfiguration(ivyFile, resolverName, artifacts, checksums, logging, overwrite)