Fixed issue sbt#789 where brpJavaRepack was negated (set to 'false' instead of 'true')

Author: yuval.itzchakov

src/main/resources/com/typesafe/sbt/packager/rpm/brpJavaRepackJar

@@ -1,6 +1 @@
-%define __os_install_post \
-%{_rpmconfigdir}/brp-compress \
-%{!?__debug_package:%{_rpmconfigdir}/brp-strip %{__strip}} \
-%{_rpmconfigdir}/brp-strip-static-archive %{__strip} \
-%{_rpmconfigdir}/brp-strip-comment-note %{__strip} %{__objdump} \
-%{nil}
+%global __os_install_post %(echo '%{__os_install_post}' | sed -e 's!/usr/lib[^[:space:]]*/brp-java-repack-jars[[:space:]].*$!!g')

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

@@ -81,7 +81,7 @@ object RpmPlugin extends AutoPlugin {
     rpmConflicts := Seq.empty,
     rpmSetarch := None,
     rpmChangelogFile := None,
-    rpmBrpJavaRepackJars := false,
+    rpmBrpJavaRepackJars := true,
     rpmPretrans := None,
     rpmPre := None,
     rpmPost := None,
@@ -121,7 +121,7 @@ object RpmPlugin extends AutoPlugin {
       (rpmProvides, rpmRequirements, rpmPrerequisites, rpmObsoletes, rpmConflicts) apply RpmDependencies,
     maintainerScripts in Rpm := {
       val scripts = (maintainerScripts in Rpm).value
-      if (rpmBrpJavaRepackJars.value) {
+      if (!rpmBrpJavaRepackJars.value) {
         val pre = scripts.getOrElse(Names.Pre, Nil)
         val scriptBits = IO.readStream(RpmPlugin.osPostInstallMacro.openStream, Charset forName "UTF-8")
         scripts + (Names.Pre -> (pre :+ scriptBits))

src/sbt-test/rpm/changelog-test/test

@@ -5,5 +5,5 @@ $ exists target/rpm/SPECS/rpm-test.spec
 
 # Check files for defaults
 > check-spec-file
-> set NativePackagerKeys.rpmBrpJavaRepackJars := true
+> set NativePackagerKeys.rpmBrpJavaRepackJars := false
 > check-spec-file

src/sbt-test/rpm/scriplets-no-javarepack-rpm/build.sbt

@@ -0,0 +1,35 @@
+
+enablePlugins(JavaServerAppPackaging)
+
+name := "rpm-test"
+
+version := "0.1.0"
+
+maintainer := "Josh Suereth <[email protected]>"
+
+packageSummary := "Test rpm package"
+
+packageDescription :=
+  """A fun package description of our software,
+  with multiple lines."""
+
+rpmRelease := "1"
+
+rpmVendor := "typesafe"
+
+rpmUrl := Some("http://github.com/sbt/sbt-native-packager")
+
+rpmLicense := Some("BSD")
+
+rpmBrpJavaRepackJars := false
+
+TaskKey[Unit]("check-spec-file") <<= (target, streams) map { (target, out) =>
+  val spec = IO.read(target / "rpm" / "SPECS" / "rpm-test.spec")
+  out.log.info(spec)
+  assert(
+    spec contains
+      """sed -e 's!/usr/lib[^[:space:]]*/brp-java-repack-jars[[:space:]].*$!!g'""",
+    "Missing java repack disabling in %pre")
+  out.log.success("Successfully tested rpm test file")
+  ()
+}

src/sbt-test/rpm/scriplets-no-javarepack-rpm/project/plugins.sbt

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

src/sbt-test/rpm/scriplets-no-javarepack-rpm/test

@@ -0,0 +1,6 @@
+# Run the debian packaging.
+> rpm:package-bin
+$ exists target/rpm/RPMS/noarch/rpm-test-0.1.0-1.noarch.rpm
+$ exists target/rpm/SPECS/rpm-test.spec
+
+> check-spec-file

src/sbt-test/rpm/scriptlets-override-build-rpm/test

@@ -5,5 +5,5 @@ $ exists target/rpm/SPECS/rpm-test.spec
 
 # Check files for defaults
 > check-spec-file
-> set NativePackagerKeys.rpmBrpJavaRepackJars := true
+> set NativePackagerKeys.rpmBrpJavaRepackJars := false
 > check-spec-file

src/sbt-test/rpm/scriptlets-override-rpm/test

@@ -20,7 +20,7 @@ $ exists var/run/rpm-test
 # TODO symlinks aren't checked
 
 > check-spec-file
-> set NativePackagerKeys.rpmBrpJavaRepackJars := true
+> set NativePackagerKeys.rpmBrpJavaRepackJars := false
 > check-spec-file
 
 > unique-scripts-in-spec-file

src/sbt-test/rpm/scriptlets-rpm/test

@@ -6,5 +6,5 @@ $ exists target/rpm/SPECS/rpm-test.spec
 # Check files for defaults
 > check-spec-file
 > check-rpm-version
-> set NativePackagerKeys.rpmBrpJavaRepackJars := true
+> set NativePackagerKeys.rpmBrpJavaRepackJars := false
 > check-spec-file

src/sbt-test/rpm/sysvinit-rpm/test

@@ -23,7 +23,7 @@ $ exists var/run/rpm-test
 
 # TODO symlinks aren't checked
 
-> set NativePackagerKeys.rpmBrpJavaRepackJars := true
+> set NativePackagerKeys.rpmBrpJavaRepackJars := false
 > check-spec-file
 > check-spec-autostart