From 5e70a15eb2c5e8f1dc5e987904a596215d799e62 Mon Sep 17 00:00:00 2001
From: Gaston Tonietti <gaston.tonietti@gmail.com>
Date: Thu, 28 Apr 2016 16:41:50 +1000
Subject: [PATCH] Encode Scalaz compatibility in the build version

- Use `settingKey` macro to use val name as setting id
- Override `version` instead of `artifactName` since it's only used to
  define local artifact path, it's not used for publishing. For more
  info: http://www.scala-sbt.org/0.13/docs/Artifacts.html#Modifying+default+artifacts
- Define `releaseVersion` just removing '-SNAPSHOT' since
  `withoutQualifier` also removes the Scalaz compatibility qualifier ('a')
---
 project/common.scala | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/project/common.scala b/project/common.scala
index 5de420f..127fb78 100644
--- a/project/common.scala
+++ b/project/common.scala
@@ -15,9 +15,9 @@ object common {
     testSettings ++
     customSettings
 
-  val scalaTestVersion    = SettingKey[String]("scalatest version")
-  val scalaCheckVersion   = SettingKey[String]("scalacheck version")
-  val scalazStreamVersion = SettingKey[String]("scalaz stream version")
+  val scalaTestVersion    = settingKey[String]("scalatest version")
+  val scalaCheckVersion   = settingKey[String]("scalacheck version")
+  val scalazStreamVersion = settingKey[String]("scalaz stream version")
 
   def testSettings = Seq(
     scalaTestVersion     := "2.2.5",
@@ -42,13 +42,17 @@ object common {
     bintrayPackage := "knobs"
   )
 
+  def withoutSnapshot(ver: Version) =
+    if(ver.qualifier.exists(_ == "-SNAPSHOT")) ver.withoutQualifier
+    else ver.copy(qualifier = ver.qualifier.map(_.replaceAll("-SNAPSHOT", "")))
+
   def releaseSettings = Seq(
     releaseCrossBuild := true,
     releaseVersion := { ver =>
       sys.env.get("TRAVIS_BUILD_NUMBER").orElse(sys.env.get("BUILD_NUMBER"))
         .map(s => try Option(s.toInt) catch { case _: NumberFormatException => Option.empty[Int] })
-        .flatMap(ci => Version(ver).map(_.withoutQualifier.copy(bugfix = ci).string))
-        .orElse(Version(ver).map(_.withoutQualifier.string))
+        .flatMap(ci => Version(ver).map(v => withoutSnapshot(v).copy(bugfix = ci).string))
+        .orElse(Version(ver).map(v => withoutSnapshot(v).string))
         .getOrElse(versionFormatError)
     },
     releaseProcess := Seq(
@@ -103,14 +107,12 @@ object common {
       sys.env.get("SCALAZ_STREAM_VERSION").getOrElse("0.7.3a")
     },
     unmanagedSourceDirectories in Compile += (sourceDirectory in Compile).value / s"scalaz-stream-${scalazStreamVersion.value.take(3)}",
-    artifactName := { (scalaVersion: ScalaVersion, module: ModuleID, artifact: Artifact) =>
-      val classifierStr = artifact.classifier.fold("")("-"+_)
-      val cross = CrossVersion(module.crossVersion, scalaVersion.full, scalaVersion.binary)
-      val base = CrossVersion.applyCross(artifact.name, cross)
-      val qualifier =
-        if(scalazStreamVersion.value.startsWith("0.7")) "scalaz71"
-        else "scalaz72"
-      base + "_" + qualifier + "-" + releaseVersion.value(version.value) + classifierStr + "." + artifact.extension
+    version := {
+      val suffix = if(scalazStreamVersion.value.startsWith("0.7")) "" else "a"
+      val versionValue = version.value
+      if(versionValue.endsWith("-SNAPSHOT"))
+        versionValue.replaceAll("-SNAPSHOT", s"$suffix-SNAPSHOT")
+      else s"$versionValue$suffix"
     }
   )
 }