strip snapshot suffix safely (#1004)

Author: colin-passiv

src/main/scala/com/typesafe/sbt/packager/rpm/RpmPlugin.scala

@@ -105,7 +105,7 @@ object RpmPlugin extends AutoPlugin {
     packageArchitecture in Rpm := "noarch",
     rpmMetadata := RpmMetadata(
       (packageName in Rpm).value,
-      if (isSnapshot.value) stripSnapshot((version in Rpm).value) else (version in Rpm).value,
+      (version in Rpm).value.stripSuffix("-SNAPSHOT"),
       rpmRelease.value,
       rpmPrefix.value,
       (packageArchitecture in Rpm).value,
@@ -163,11 +163,6 @@ object RpmPlugin extends AutoPlugin {
       }
     }
   )
-
-  def stripSnapshot(version: String): String = {
-    val suffixPosition = version.indexOf("-SNAPSHOT")
-    version.slice(0, suffixPosition)
-  }
 }
 
 object RpmDeployPlugin extends AutoPlugin {

src/sbt-test/rpm/snapshot-override-rpm/build.sbt

@@ -0,0 +1,26 @@
+enablePlugins(RpmPlugin)
+
+name := "rpm-test"
+
+version := "0.1.0-SNAPSHOT"
+
+maintainer := "Keir Lawson <[email protected]>"
+
+packageSummary := "Snapshot test rpm package"
+
+packageDescription := """A fun package description of our software,
+  with multiple lines."""
+
+rpmVendor := "typesafe"
+
+rpmUrl := Some("http://github.com/sbt/sbt-native-packager")
+
+rpmLicense := Some("BSD")
+
+version in Rpm := "1"
+
+TaskKey[Unit]("check-snapshot") := {
+  assert(rpmRelease.value == "SNAPSHOT", s"RPM has incorrect value ${rpmRelease.value}")
+  assert(rpmMetadata.value.version == "1", s"RPM has incorrect value ${rpmMetadata.value.version}")
+  assert((version in Rpm).value == "1", s"RPM has incorrect value ${(version in Rpm).value}")
+}

src/sbt-test/rpm/snapshot-override-rpm/project/plugins.sbt

@@ -0,0 +1 @@
+addSbtPlugin("com.typesafe.sbt" % "sbt-native-packager" % sys.props("project.version"))

src/sbt-test/rpm/snapshot-override-rpm/src/universal/conf/test

@@ -0,0 +1 @@
+# Test configuration file!

src/sbt-test/rpm/snapshot-override-rpm/test

@@ -0,0 +1,6 @@
+# Run the debian packaging.
+> rpm:package-bin
+$ exists target/rpm/RPMS/noarch/rpm-test-1-SNAPSHOT.noarch.rpm
+
+#Check release and version configured correctly
+> check-snapshot