SAMZA-1595: Fix scalacCompileOptions format to build with zinc scala compiler. master
authorShanthoosh Venkataraman <svenkataraman@linkedin.com>
Fri, 23 Feb 2018 00:44:00 +0000 (16:44 -0800)
committerxiliu <xiliu@linkedin.com>
Fri, 23 Feb 2018 00:44:00 +0000 (16:44 -0800)
Zinc scala compiler(part of gradle version >= 3.0) expects the scala compilation arguments as a list(where each compilation argument is an element of the list).

In samza, the compilation arguments are concatenated into a single string and passed to the compiler.

This causes build failures when samza is built with Zinc scala compiler.

Existing ant scala compiler used to build samza in open source accepts the compilation arguments both as list and string.

Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com>

Reviewers: Xinyu Liu <xinyuliu.us@gmail.com>

Closes #430 from shanthoosh/master

build.gradle
gradle/dependency-versions-scala-2.10.gradle
gradle/dependency-versions-scala-2.11.gradle
gradle/dependency-versions-scala-2.12.gradle

index 4aab830..a15a456 100644 (file)
@@ -100,7 +100,7 @@ allprojects {
   // For all scala compilation, add extra compiler options, taken from version-specific
   // dependency-versions-scala file applied above.
   tasks.withType(ScalaCompile) {
-    scalaCompileOptions.additionalParameters = [ scalaOptions ]
+    scalaCompileOptions.additionalParameters = scalaOptions
   }
 }
 
index 698cfbc..f42775f 100644 (file)
@@ -23,7 +23,7 @@ ext {
   // -feature: Give detailed warnings about language feature use (rather than just 'there were 4 warnings')
   // -language:implicitConversions: Allow the use of implicit conversions without warning or library import
   // -language:reflectiveCalls: Allow the automatic use of reflection to access fields without warning or library import
-  scalaOptions = "-feature -language:implicitConversions -language:reflectiveCalls"
+  scalaOptions = ["-feature", "-language:implicitConversions", "-language:reflectiveCalls"]
   scalatraVersion = "2.2.1"
   jettyVersion = "9.2.7.v20150116"
 }
index e033a86..2fbb22b 100644 (file)
@@ -23,7 +23,7 @@ ext {
   // -feature: Give detailed warnings about language feature use (rather than just 'there were 4 warnings')
   // -language:implicitConversions: Allow the use of implicit conversions without warning or library import
   // -language:reflectiveCalls: Allow the automatic use of reflection to access fields without warning or library import
-  scalaOptions = "-feature -language:implicitConversions -language:reflectiveCalls"
+  scalaOptions = ["-feature", "-language:implicitConversions", "-language:reflectiveCalls"]
   scalatraVersion = "2.5.0"
   jettyVersion = "9.2.7.v20150116"
 }
index f3eec81..3c8e93d 100644 (file)
@@ -23,7 +23,7 @@ ext {
   // -feature: Give detailed warnings about language feature use (rather than just 'there were 4 warnings')
   // -language:implicitConversions: Allow the use of implicit conversions without warning or library import
   // -language:reflectiveCalls: Allow the automatic use of reflection to access fields without warning or library import
-  scalaOptions = "-feature -language:implicitConversions -language:reflectiveCalls"
+  scalaOptions = ["-feature", "-language:implicitConversions", "-language:reflectiveCalls"]
   scalatraVersion = "2.5.0"
   jettyVersion = "9.2.7.v20150116"
 }