diff --git a/ivy/src/main/scala/sbt/CompatibilityWarning.scala b/ivy/src/main/scala/sbt/CompatibilityWarning.scala new file mode 100644 index 000000000..7c7a7c1fb --- /dev/null +++ b/ivy/src/main/scala/sbt/CompatibilityWarning.scala @@ -0,0 +1,25 @@ +package sbt + +private[sbt] object CompatibilityWarning { + def apply(module: IvySbt#Module, mavenStyle: Boolean, log: Logger): Unit = { + if (mavenStyle) { + processIntransitive(module, log) + } + } + + def processIntransitive(module: IvySbt#Module, log: Logger): Unit = { + val directDependencies: Seq[ModuleID] = module.moduleSettings match { + case x: InlineConfiguration => x.dependencies + case x: InlineConfigurationWithExcludes => x.dependencies + case _ => Seq() + } + directDependencies foreach { m => + if (!m.isTransitive) { + log.warn( + s"""Found intransitive dependency ($m), but maven does not support intransitive dependencies. + | Use exclusions instead so transitive dependencies will be correctly excluded in dependent projects. + """.stripMargin) + } else () + } + } +} diff --git a/ivy/src/main/scala/sbt/MakePom.scala b/ivy/src/main/scala/sbt/MakePom.scala index e2018612f..742b7b8b4 100644 --- a/ivy/src/main/scala/sbt/MakePom.scala +++ b/ivy/src/main/scala/sbt/MakePom.scala @@ -235,30 +235,21 @@ class MakePom(val log: Logger) { def makeDependency(dependency: DependencyDescriptor, includeTypes: Set[String]): NodeSeq = makeDependency(dependency, includeTypes, Nil) - def makeDependency(dependency: DependencyDescriptor, includeTypes: Set[String], excludes: Seq[ExcludeRule]): NodeSeq = { - def warnIntransitve(): Unit = - if (!dependency.isTransitive) - log.warn( - s"""Translating intransitive dependency (${dependency.getDependencyId}) into pom.xml, but maven does not support intransitive dependencies. - | Please use exclusions instead so transitive dependencies will be correctly excluded in dependent projects. - """.stripMargin) - else () - val artifacts = dependency.getAllDependencyArtifacts - val includeArtifacts = artifacts.filter(d => includeTypes(d.getType)) - if (artifacts.isEmpty) { - val configs = dependency.getModuleConfigurations - if (!configs.forall(Set("sources", "docs"))) { - warnIntransitve() - val (scope, optional) = getScopeAndOptional(dependency.getModuleConfigurations) - makeDependencyElem(dependency, scope, optional, None, None, excludes) - } else NodeSeq.Empty - } else if (includeArtifacts.isEmpty) { - NodeSeq.Empty - } else { - warnIntransitve() - NodeSeq.fromSeq(artifacts.flatMap(a => makeDependencyElem(dependency, a, excludes))) + def makeDependency(dependency: DependencyDescriptor, includeTypes: Set[String], excludes: Seq[ExcludeRule]): NodeSeq = + { + val artifacts = dependency.getAllDependencyArtifacts + val includeArtifacts = artifacts.filter(d => includeTypes(d.getType)) + if (artifacts.isEmpty) { + val configs = dependency.getModuleConfigurations + if (configs.filterNot(Set("sources", "docs")).nonEmpty) { + val (scope, optional) = getScopeAndOptional(dependency.getModuleConfigurations) + makeDependencyElem(dependency, scope, optional, None, None, excludes) + } else NodeSeq.Empty + } else if (includeArtifacts.isEmpty) + NodeSeq.Empty + else + NodeSeq.fromSeq(artifacts.flatMap(a => makeDependencyElem(dependency, a, excludes))) } - } @deprecated("Use `makeDependencyElem` variant which takes excludes", "0.13.9") def makeDependencyElem(dependency: DependencyDescriptor, artifact: DependencyArtifactDescriptor): Option[Elem] = diff --git a/main/src/main/scala/sbt/Defaults.scala b/main/src/main/scala/sbt/Defaults.scala index 9c68b0e23..c713eabf7 100755 --- a/main/src/main/scala/sbt/Defaults.scala +++ b/main/src/main/scala/sbt/Defaults.scala @@ -1384,6 +1384,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. @@ -1399,17 +1400,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 @@ -1428,6 +1429,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 = diff --git a/notes/0.13.10.markdown b/notes/0.13.10.markdown index bac68befa..d78f348e9 100644 --- a/notes/0.13.10.markdown +++ b/notes/0.13.10.markdown @@ -75,7 +75,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. - Adds configurable compiler bridge. See below. - sbt Serialization is updated to 0.1.2. [2117][#2117] by [@dwijnand][@dwijnand] diff --git a/sbt/src/sbt-test/dependency-management/pom-type/build.sbt b/sbt/src/sbt-test/dependency-management/pom-type/build.sbt index 79af2c7af..b83e9fbbb 100644 --- a/sbt/src/sbt-test/dependency-management/pom-type/build.sbt +++ b/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 sections are generated") -checkPom := { - val pomFile = makePom.value - val pom = xml.XML.loadFile(pomFile) - val tpe = pom \\ "type" - if(tpe.nonEmpty) { - sys.error("Expected no sections, got: " + tpe + " in \n\n" + pom) - } - val dir = (streams in makePom).value.cacheDirectory / "out" - val lines = IO.readLines(dir) - val hasError = lines exists { line => line contains "Translating intransitive dependency "} - assert(hasError, s"Failed to detect intransitive dependencies, got: ${lines.mkString("\n")}") -} \ No newline at end of file +lazy val root = (project in file(".")). + settings( + scalaVersion := "2.10.6", + libraryDependencies += "org.scala-tools.sbinary" %% "sbinary" % "0.4.1" withSources() withJavadoc(), + libraryDependencies += "org.scala-sbt" % "io" % "0.13.8" intransitive(), + checkPom := { + val pomFile = makePom.value + val pom = xml.XML.loadFile(pomFile) + val tpe = pom \\ "type" + if(tpe.nonEmpty) { + sys.error("Expected no 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") + ) diff --git a/sbt/src/sbt-test/dependency-management/pom-type/test b/sbt/src/sbt-test/dependency-management/pom-type/test index 0e9f0f7b6..8d831496e 100644 --- a/sbt/src/sbt-test/dependency-management/pom-type/test +++ b/sbt/src/sbt-test/dependency-management/pom-type/test @@ -1 +1 @@ -> checkPom \ No newline at end of file +> checkPom