Readd the deprecated ivy string interpolator and some fixes (#5351)

The interpolated `ivy"g:a:v"` string is heavily used outside of Mill
buildscripts, e.g. in various repository managers and search result
pages. The popular Mvnrepository Search recently added support for Mill.
https://mvnrepository.com/artifact/com.lihaoyi/mill-dist/1.0.0-RC2

![grafik](https://github.com/user-attachments/assets/374e2924-02f9-4199-a262-66e5dae3c7a2)

Unfortunately, those pages often use the now removed `ivy` interpolator.

To improved the user experience, this PR re-adds the `ivy` interpolator,
such that copy and pasting dependencies from such portals just works.
Users should instantly see the deprecation warning and learn, that they
should use `mvn` instead. But at least, the build succeeds, whereas in
the current state, it would fail.

Pull request: #5351

Author: lefou

libs/kotlinlib/src/mill/kotlinlib/package.scala

@@ -1,13 +1,11 @@
 package mill
 
 package object kotlinlib {
-  implicit class DepSyntax(ctx: StringContext) {
-    def mvn(args: Any*): Dep = Dep.parse {
-      (
-        ctx.parts.take(args.length).zip(args).flatMap { case (p, a) => Seq(p, a) } ++
-          ctx.parts.drop(args.length)
-      ).mkString
-    }
+  implicit class DepSyntax(ctx: StringContext) extends AnyVal {
+    def mvn(args: Any*): Dep = mill.scalalib.DepSyntax(ctx).mvn(args*)
+
+    @deprecated("Use `mvn` string interpolator instead.", "Mill 0.12.11")
+    def ivy(args: Any*): Dep = mill.scalalib.DepSyntax(ctx).mvn(args*)
   }
 
   type Dep = mill.scalalib.Dep

libs/scalalib/src/mill/javalib/package.scala

@@ -1,13 +1,11 @@
 package mill
 
 package object javalib extends mill.scalalib.JsonFormatters {
-  implicit class DepSyntax(ctx: StringContext) {
-    def mvn(args: Any*): Dep = Dep.parse {
-      (
-        ctx.parts.take(args.length).zip(args).flatMap { case (p, a) => Seq(p, a) } ++
-          ctx.parts.drop(args.length)
-      ).mkString
-    }
+  implicit class DepSyntax(ctx: StringContext) extends AnyVal {
+    def mvn(args: Any*): Dep = mill.scalalib.DepSyntax(ctx).mvn(args*)
+
+    @deprecated("Use `mvn` string interpolator instead.", "Mill 0.12.11")
+    def ivy(args: Any*): Dep = mill.scalalib.DepSyntax(ctx).mvn(args*)
   }
 
   val Assembly = mill.scalalib.Assembly

libs/scalalib/src/mill/scalalib/Dep.scala

@@ -11,7 +11,7 @@ case class Dep(dep: coursier.Dependency, cross: CrossVersion, force: Boolean) {
   require(
     !dep.module.name.value.contains("/") &&
       !dep.module.organization.value.contains("/") &&
-      !dep.version.contains("/"),
+      !dep.versionConstraint.asString.contains("/"),
     "Dependency coordinates must not contain `/`s"
   )
 

libs/scalalib/src/mill/scalalib/IvyDepsTreeArgs.scala renamed to libs/scalalib/src/mill/scalalib/MvnDepsTreeArgs.scala

@@ -1,15 +1,20 @@
 package mill
 
 package object scalalib extends mill.scalalib.JsonFormatters {
-  implicit class DepSyntax(ctx: StringContext) {
+
+  implicit class DepSyntax(ctx: StringContext) extends AnyVal {
     def mvn(args: Any*): Dep = Dep.parse {
       (
         ctx.parts.take(args.length).zip(args).flatMap { case (p, a) => Seq(p, a) } ++
           ctx.parts.drop(args.length)
       ).mkString
     }
+
+    @deprecated("Use `mvn` string interpolator instead.", "Mill 0.12.11")
+    def ivy(args: Any*): Dep = mvn(args*)
   }
 
   val CrossVersion = mill.define.CrossVersion
   type CrossVersion = mill.define.CrossVersion
+
 }