Move intransitive warning to update. Ref #2127

main/src/main/scala/sbt/Defaults.scala

@@ -1442,6 +1442,7 @@ object Classpaths {
     val logicalClock = LogicalClock(st.hashCode)
     val depDir = dependencyCacheDirectory.value
     val uc0 = updateConfiguration.value
+    val ms = publishMavenStyle.value
     // Normally, log would capture log messages at all levels.
     // Ivy logs are treated specially using sbt.UpdateConfiguration.logging.
     // This code bumps up the sbt.UpdateConfiguration.logging to Full when logLevel is Debug.
@@ -1457,17 +1458,17 @@ object Classpaths {
     cachedUpdate(s.cacheDirectory / updateCacheName.value, show, ivyModule.value, uc, transform,
       skip = (skip in update).value, force = isRoot || forceUpdateByTime, depsUpdated = depsUpdated,
       uwConfig = uwConfig, logicalClock = logicalClock, depDir = Some(depDir),
-      ewo = ewo, log = s.log)
+      ewo = ewo, mavenStyle = ms, log = s.log)
   }
   @deprecated("Use cachedUpdate with the variant that takes unresolvedHandler instead.", "0.13.6")
   def cachedUpdate(cacheFile: File, label: String, module: IvySbt#Module, config: UpdateConfiguration,
     transform: UpdateReport => UpdateReport, skip: Boolean, force: Boolean, depsUpdated: Boolean, log: Logger): UpdateReport =
     cachedUpdate(cacheFile, label, module, config, transform, skip, force, depsUpdated,
-      UnresolvedWarningConfiguration(), LogicalClock.unknown, None, EvictionWarningOptions.empty, log)
+      UnresolvedWarningConfiguration(), LogicalClock.unknown, None, EvictionWarningOptions.empty, true, log)
   private[sbt] def cachedUpdate(cacheFile: File, label: String, module: IvySbt#Module, config: UpdateConfiguration,
     transform: UpdateReport => UpdateReport, skip: Boolean, force: Boolean, depsUpdated: Boolean,
     uwConfig: UnresolvedWarningConfiguration, logicalClock: LogicalClock, depDir: Option[File],
-    ewo: EvictionWarningOptions, log: Logger): UpdateReport =
+    ewo: EvictionWarningOptions, mavenStyle: Boolean, log: Logger): UpdateReport =
     {
       implicit val updateCache = updateIC
       type In = IvyConfiguration :+: ModuleSettings :+: UpdateConfiguration :+: HNil
@@ -1486,6 +1487,7 @@ object Classpaths {
           val ew = EvictionWarning(module, ewo, result, log)
           ew.lines foreach { log.warn(_) }
           ew.infoAllTheThings foreach { log.info(_) }
+          val cw = CompatibilityWarning(module, mavenStyle, log)
           result
       }
       def uptodate(inChanged: Boolean, out: UpdateReport): Boolean =

notes/0.13.10.markdown

@@ -73,7 +73,7 @@
 ### Improvements
 
 - Scala version used by the build is updated to 2.10.6. [#2311][2311] by [@eed3si9n][@eed3si9n]
-- `makePom` warns when it sees intransitive dependencies, which do not translate to Maven. [#2127][2127] by [@jsuereth][@jsuereth]
+- If `publishMavenStyle` is `true`, `update` task warns when it sees intransitive dependencies, which do not translate to Maven. [#2127][2127] by [@jsuereth][@jsuereth]
 - Adds `Def.settings`, which facilitates mixing settings with seq of settings. See below.
 - sbt Serialization is updated to 0.1.2. [2117][#2117] by [@dwijnand][@dwijnand]
 - Hides the stack trace on compilation error in build definition. [#2071][2071]/[#2091][2091] by [@Duhemm][@Duhemm]

sbt/src/sbt-test/dependency-management/pom-type/build.sbt

@@ -1,19 +1,22 @@
-scalaVersion := "2.10.2"
-
-libraryDependencies += "org.scala-sbt" %% "sbinary" % "0.4.1" withSources() withJavadoc()
-libraryDependencies += "org.scala-sbt" % "io" % "0.13.8" intransitive()
-
 lazy val checkPom = taskKey[Unit]("check pom to ensure no <type> sections are generated")
 
-checkPom := {
+lazy val root = (project in file(".")).
-	val pomFile = makePom.value
+  settings(
-	val pom = xml.XML.loadFile(pomFile)
+    scalaVersion := "2.10.6",
-	val tpe = pom \\ "type"
+    libraryDependencies += "org.scala-tools.sbinary" %% "sbinary" % "0.4.1" withSources() withJavadoc(),
-	if(tpe.nonEmpty) {
+    libraryDependencies += "org.scala-sbt" % "io" % "0.13.8" intransitive(),
-		sys.error("Expected no <type> sections, got: " + tpe + " in \n\n" + pom)
+    checkPom := {
-	}
+      val pomFile = makePom.value
-	val dir = (streams in makePom).value.cacheDirectory / "out"
+      val pom = xml.XML.loadFile(pomFile)
-	val lines = IO.readLines(dir)
+      val tpe = pom \\ "type"
-	val hasError = lines exists { line => line contains "Translating intransitive dependency "}
+      if(tpe.nonEmpty) {
-	assert(hasError, s"Failed to detect intransitive dependencies, got: ${lines.mkString("\n")}")
+        sys.error("Expected no <type> sections, got: " + tpe + " in \n\n" + pom)
-}
+      }
+      val ur = update.value
+      val dir = (streams in update).value.cacheDirectory / "out"
+      val lines = IO.readLines(dir)
+      val hasError = lines exists { line => line contains "Found intransitive dependency "}
+      assert(hasError, s"Failed to detect intransitive dependencies, got: ${lines.mkString("\n")}")
+    },
+    resolvers += Resolver.typesafeIvyRepo("releases")
+  )