Make MiMa sbt plugin tell you which artifact/project it is processing

[bantonsson]

Output if I in akka do mima-report-binary-issues from the top level (only akka-actor and akka-testkit have been configured)

mima-report-binary-issues
[info] Resolving com.typesafe.akka#akka-actor;2.0 ...
[info] Resolving org.scala-lang#scala-library;2.9.1-1 ...
[info] Resolving com.typesafe.akka#akka-testkit;2.0 ...
[info] Resolving com.typesafe.akka#akka-actor;2.0 ...
[info] Resolving org.scala-lang#scala-library;2.9.1-1 ...
[info] Found 17 potential binary incompatibilities
[error] * class akka.actor.ReceiveTimeout was concrete; is declared abstract in new version
< SNIP SNAP >
[error] * class akka.routing.CurrentRoutees was concrete; is declared abstract in new version
[info] Found 0 potential binary incompatibilities

Kind of hard to see what's ok and what's not.

[viktorklang]

How can it be an error and then reported as 0 problems?

[bantonsson]

The first [info] is akka-actor, and the second [info] is akka-testkit. Isn't it obvious?

[viktorklang]

I don't think it's obvious that the following is outputted:

[error] * class akka.routing.CurrentRoutees was concrete; is declared abstract in new version

and then it says:

[info] Found 0 potential binary incompatibilities

How can something be an error, but not something that needs fixing?

[bantonsson]

That's the meaning of this ticket.

If the output had looked something like this:

mima-report-binary-issues
[info] Resolving com.typesafe.akka#akka-actor;2.0 ...
[info] Resolving org.scala-lang#scala-library;2.9.1-1 ...
[info] Resolving com.typesafe.akka#akka-testkit;2.0 ...
[info] Resolving com.typesafe.akka#akka-actor;2.0 ...
[info] Resolving org.scala-lang#scala-library;2.9.1-1 ...
[info] akka-actor: Found 17 potential binary incompatibilities
[error] * class akka.actor.ReceiveTimeout was concrete; is declared abstract in new version

[error] * class akka.routing.CurrentRoutees was concrete; is declared abstract in new version
[info] akka-testkit: Found 0 potential binary incompatibilities

It would make more sense

[viktorklang]

Or simply:

[error] [akka-actor] [major] * class akka.actor.ReceiveTimeout was concrete; is declared abstract in new version

[bantonsson]

Yes, that would be fine as well, but there needs to be an output when it is ok that contains the name as well.