Specify module name via moduleName option instead of freeCompilerArgs

This allows to get rid of warnings about duplicate module name in some of the modules.
JetBrains · Nov 6, 2020 · d326d6a · d326d6a
1 parent bdb8a58
commit d326d6a
Show file tree

Hide file tree

Showing 13 changed files with 20 additions and 20 deletions.
diff --git a/libraries/kotlin.test/junit/build.gradle b/libraries/kotlin.test/junit/build.gradle
@@ -31,7 +31,8 @@ configureSourcesJar()
 configureJavadocJar()
 
 compileKotlin {
-    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package", "-module-name", project.name]
+    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package"]
+    kotlinOptions.moduleName = project.name
 }
 
 compileTestKotlin {

diff --git a/libraries/kotlin.test/junit5/build.gradle b/libraries/kotlin.test/junit5/build.gradle
@@ -41,7 +41,8 @@ tasks.withType(org.jetbrains.kotlin.gradle.tasks.KotlinCompile) {
 }
 
 compileKotlin {
-    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package", "-module-name", project.name]
+    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package"]
+    kotlinOptions.moduleName = project.name
 }
 
 compileTestKotlin {

diff --git a/libraries/kotlin.test/jvm/build.gradle b/libraries/kotlin.test/jvm/build.gradle
@@ -45,8 +45,8 @@ configureJavadocJar()
 
 compileKotlin {
     kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package", "-Xnormalize-constructor-calls=enable",
-                                      "-Xopt-in=kotlin.contracts.ExperimentalContracts",
-                                      "-module-name", project.archivesBaseName, ]
+                                      "-Xopt-in=kotlin.contracts.ExperimentalContracts"]
+    kotlinOptions.moduleName = project.archivesBaseName
 }
 
 compileTestKotlin {

diff --git a/libraries/kotlin.test/testng/build.gradle b/libraries/kotlin.test/testng/build.gradle
@@ -34,7 +34,8 @@ configureSourcesJar()
 configureJavadocJar()
 
 compileKotlin {
-    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package", "-module-name", project.name]
+    kotlinOptions.freeCompilerArgs = ["-Xallow-kotlin-package"]
+    kotlinOptions.moduleName = project.name
 }
 
 compileTestKotlin {

diff --git a/libraries/reflect/api/build.gradle b/libraries/reflect/api/build.gradle
@@ -47,8 +47,8 @@ compileKotlin {
                             "-Xnormalize-constructor-calls=enable",
                             "-Xno-optimized-callable-references",
                             "-Xno-kotlin-nothing-value-exception",
-                            "-Xopt-in=kotlin.RequiresOptIn",
-                            "-module-name", "kotlin-reflection"]
+                            "-Xopt-in=kotlin.RequiresOptIn"]
+        moduleName = "kotlin-reflection"
     }
 }
 

diff --git a/libraries/stdlib/coroutines-experimental/build.gradle.kts b/libraries/stdlib/coroutines-experimental/build.gradle.kts
@@ -51,9 +51,9 @@ tasks {
                 "-Xopt-in=kotlin.contracts.ExperimentalContracts",
                 "-Xcoroutines=enable",
                 "-XXLanguage:-ReleaseCoroutines",
-                "-Xno-use-ir",
-                "-module-name", "kotlin-coroutines-experimental-compat"
+                "-Xno-use-ir"
             )
+            moduleName = "kotlin-coroutines-experimental-compat"
         }
     }
     val compileTestKotlin by existing(KotlinCompile::class) {

diff --git a/libraries/stdlib/jdk7/build.gradle b/libraries/stdlib/jdk7/build.gradle
@@ -76,10 +76,10 @@ compileKotlin {
             "-Xallow-kotlin-package",
             "-Xmultifile-parts-inherit",
             "-Xnormalize-constructor-calls=enable",
-            "-module-name", project.name,
             "-Xopt-in=kotlin.RequiresOptIn",
             "-Xopt-in=kotlin.contracts.ExperimentalContracts",
     ]
+    kotlinOptions.moduleName = project.name
 }
 
 compileTestKotlin {

diff --git a/libraries/stdlib/jdk8/build.gradle b/libraries/stdlib/jdk8/build.gradle
@@ -70,8 +70,8 @@ compileKotlin {
             "-Xallow-kotlin-package",
             "-Xmultifile-parts-inherit",
             "-Xnormalize-constructor-calls=enable",
-            "-module-name", project.name
     ]
+    kotlinOptions.moduleName = project.name
 }
 
 compileTestKotlin {

diff --git a/libraries/stdlib/jvm-minimal-for-test/build.gradle.kts b/libraries/stdlib/jvm-minimal-for-test/build.gradle.kts
@@ -63,13 +63,12 @@ tasks.withType<KotlinCompile> {
     dependsOn(copySources)
     kotlinOptions {
         freeCompilerArgs += listOf(
-            "-module-name",
-            "kotlin-stdlib",
             "-Xallow-kotlin-package",
             "-Xmulti-platform",
             "-Xopt-in=kotlin.RequiresOptIn",
             "-Xopt-in=kotlin.contracts.ExperimentalContracts"
         )
+        moduleName = "kotlin-stdlib"
     }
 }
 

diff --git a/libraries/stdlib/jvm/build.gradle b/libraries/stdlib/jvm/build.gradle
@@ -117,12 +117,12 @@ compileKotlin {
                 "-Xallow-result-return-type",
                 "-Xmultifile-parts-inherit",
                 "-Xnormalize-constructor-calls=enable",
-                "-module-name", "kotlin-stdlib",
                 "-Xopt-in=kotlin.RequiresOptIn",
                 "-Xopt-in=kotlin.ExperimentalMultiplatform",
                 "-Xopt-in=kotlin.contracts.ExperimentalContracts",
                 "-Xinline-classes"
         ]
+        moduleName = "kotlin-stdlib"
     }
 }
 

diff --git a/libraries/tools/kotlin-annotations-android/build.gradle.kts b/libraries/tools/kotlin-annotations-android/build.gradle.kts
@@ -16,10 +16,8 @@ jvmTarget = "1.6"
 javaHome = rootProject.extra["JDK_16"] as String
 
 tasks.withType<KotlinCompile> {
-    kotlinOptions.freeCompilerArgs += listOf(
-            "-Xallow-kotlin-package",
-            "-module-name", project.name
-    )
+    kotlinOptions.freeCompilerArgs += listOf("-Xallow-kotlin-package")
+    kotlinOptions.moduleName = project.name
 }
 
 sourceSets {

diff --git a/libraries/tools/kotlin-annotations-jvm/build.gradle b/libraries/tools/kotlin-annotations-jvm/build.gradle
@@ -29,8 +29,8 @@ compileKotlin {
     kotlinOptions.freeCompilerArgs = [
             "-Xallow-kotlin-package",
             "-Xnormalize-constructor-calls=enable",
-            "-module-name", project.name
     ]
+    kotlinOptions.moduleName = project.name
 }
 
 configureJvmIrBackend(project)
diff --git a/libraries/tools/script-runtime/build.gradle b/libraries/tools/script-runtime/build.gradle
@@ -36,8 +36,8 @@ compileKotlin {
     kotlinOptions.freeCompilerArgs = [
             "-Xallow-kotlin-package",
             "-Xnormalize-constructor-calls=enable",
-            "-module-name", project.name
     ]
+    kotlinOptions.moduleName = project.name
 }
 
 configureJvmIrBackend(project)