Merge pull request #207 from sbt/wip/fix-205

Handle spaces in java arguments properly

build.sbt

@@ -9,7 +9,7 @@ sbtVersion in Global := {
   }
 }
 
-scalaVersion in Global := "2.9.2"
+scalaVersion in Global := "2.10.2"
 
 crossScalaVersions := Seq("2.9.2", "2.10.2")
 

src/main/resources/com/typesafe/sbt/packager/archetypes/bash-template

@@ -249,12 +249,12 @@ run() {
   if [[ "$JAVA_OPTS" != "" ]]; then
     java_opts="${JAVA_OPTS}"
   fi
-  
+
   # run sbt
   execRunner "$java_cmd" \
     $(get_mem_opts $app_mem) \
-    ${java_opts} \
-    ${java_args[@]} \
+    ${java_opts[@]} \
+    "${java_args[@]}" \
     -cp "$(fix_classpath "$app_classpath")" \
     $app_mainclass \
     "${app_commands[@]}" \

src/sbt-test/debian/java-app-archetype/build.sbt

@@ -37,4 +37,7 @@ TaskKey[Unit]("check-script") <<= (stagingDirectory in Universal, name, streams)
   val output = Process("bash " + script.getAbsolutePath).!!
   val expected = "SUCCESS!"
   assert(output contains expected, "Failed to correctly run the main script!.  Found ["+output+"] wanted ["+expected+"]")
+  val expected2 = "Something with spaces"
+  val output2 = Process(Seq("bash", script.getAbsolutePath, "-Dresult.string="+expected2)).!!
+  assert(output2 contains expected2, "Failed to correctly run the main script with spaced java args!.  Found ["+output2+"] wanted ["+expected2+"]")
 }

src/sbt-test/debian/java-app-archetype/src/main/scala/test/Test.scala

@@ -1,5 +1,9 @@
 package test
 
 object Test extends App {
-  println("SUCCESS!")
+  Option(sys.props("result.string")) match {
+  	case Some(value) => println(value)
+  	case _ => println("SUCCESS!")
+  }
+
 }

test-project/project/build.properties

@@ -1 +1 @@
-sbt.version=0.13.0
+sbt.version=0.13.1

test-project/project/plugins.sbt

@@ -2,4 +2,3 @@ resolvers += Resolver.typesafeRepo("releases")
 
 addSbtPlugin("com.typesafe.play" % "sbt-plugin" % "2.2.0")
 
-addSbtPlugin("com.typesafe.sbt" % "sbt-native-packager" % "0.6.3")