Merge pull request #477 from coursier/topic/enable-sbt-plugin-hack-by-default

Enable sbt plugin hack in Maven repo paths by default

cli/src/main/scala-2.11/coursier/cli/Helper.scala

@@ -134,11 +134,10 @@ class Helper(
 
     var repos = (if (common.noDefault) Nil else defaultRepositories) ++ repos0
 
-    if (common.sbtPluginHack)
-      repos = repos.map {
-        case m: MavenRepository => m.copy(sbtAttrStub = true)
-        case other => other
-      }
+    repos = repos.map {
+      case m: MavenRepository => m.copy(sbtAttrStub = common.sbtPluginHack)
+      case other => other
+    }
 
     if (common.dropInfoAttr)
       repos = repos.map {

cli/src/main/scala-2.11/coursier/cli/Options.scala

@@ -37,7 +37,7 @@ final case class CommonOptions(
   @Help("Do not add default repositories (~/.ivy2/local, and Central)")
     noDefault: Boolean = false,
   @Help("Modify names in Maven repository paths for SBT plugins")
-    sbtPluginHack: Boolean = false,
+    sbtPluginHack: Boolean = true,
   @Help("Drop module attributes starting with 'info.' - these are sometimes used by projects built with SBT")
     dropInfoAttr: Boolean = false,
   @Help("Force module version")

core/shared/src/main/scala/coursier/maven/MavenRepository.scala

@@ -63,7 +63,7 @@ final case class MavenRepository(
   root: String,
   changing: Option[Boolean] = None,
   /** Hackish hack for sbt plugins mainly - what this does really sucks */
-  sbtAttrStub: Boolean = false,
+  sbtAttrStub: Boolean = true,
   authentication: Option[Authentication] = None
 ) extends Repository {
 

sbt-coursier/src/main/scala-2.10/coursier/FromSbt.scala

@@ -156,7 +156,6 @@ object FromSbt {
       Some(
         MavenRepository(
           root0,
-          sbtAttrStub = true,
           authentication = authentication
         )
       )