Remove deprecated ConfigureUtil usages in JS configuration DSL

^KT-46019 In Progress
This commit is contained in:
Yahor Berdnikau
2022-06-27 23:06:07 +02:00
committed by Space
parent ae3451e2c8
commit ddeb8b0a1f
9 changed files with 32 additions and 35 deletions
@@ -30,7 +30,7 @@ import org.jetbrains.kotlin.gradle.utils.lowerCamelCaseName
import org.jetbrains.kotlin.gradle.utils.setProperty import org.jetbrains.kotlin.gradle.utils.setProperty
import javax.inject.Inject import javax.inject.Inject
open class KotlinJsTarget abstract class KotlinJsTarget
@Inject @Inject
constructor( constructor(
project: Project, project: Project,
@@ -70,9 +70,9 @@ constructor(
if (irTarget == null) if (irTarget == null)
super.kotlinComponents super.kotlinComponents
else { else {
val mainCompilation = compilations.getByName(KotlinCompilation.MAIN_COMPILATION_NAME) val mainCompilation = compilations.getByName(MAIN_COMPILATION_NAME)
val usageContexts = createUsageContexts(mainCompilation) + val usageContexts = createUsageContexts(mainCompilation) +
irTarget!!.createUsageContexts(irTarget!!.compilations.getByName(KotlinCompilation.MAIN_COMPILATION_NAME)) irTarget!!.createUsageContexts(irTarget!!.compilations.getByName(MAIN_COMPILATION_NAME))
val componentName = val componentName =
if (project.kotlinExtension is KotlinMultiplatformExtension) if (project.kotlinExtension is KotlinMultiplatformExtension)
@@ -96,7 +96,7 @@ constructor(
return usageContexts + return usageContexts +
DefaultKotlinUsageContext( DefaultKotlinUsageContext(
compilation = compilations.getByName(KotlinCompilation.MAIN_COMPILATION_NAME), compilation = compilations.getByName(MAIN_COMPILATION_NAME),
usage = project.usageByName("java-api-jars"), usage = project.usageByName("java-api-jars"),
dependencyConfigurationName = commonFakeApiElementsConfigurationName, dependencyConfigurationName = commonFakeApiElementsConfigurationName,
overrideConfigurationArtifacts = project.setProperty { emptyList() } overrideConfigurationArtifacts = project.setProperty { emptyList() }
@@ -5,10 +5,9 @@
package org.jetbrains.kotlin.gradle.targets.js.dsl package org.jetbrains.kotlin.gradle.targets.js.dsl
import groovy.lang.Closure import org.gradle.api.Action
import org.gradle.api.GradleException import org.gradle.api.GradleException
import org.gradle.api.NamedDomainObjectContainer import org.gradle.api.NamedDomainObjectContainer
import org.gradle.util.ConfigureUtil
import org.jetbrains.kotlin.gradle.dsl.KotlinJsDce import org.jetbrains.kotlin.gradle.dsl.KotlinJsDce
import org.jetbrains.kotlin.gradle.plugin.KotlinTarget import org.jetbrains.kotlin.gradle.plugin.KotlinTarget
import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinJsCompilation import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinJsCompilation
@@ -39,17 +38,17 @@ interface KotlinJsTargetDsl : KotlinTarget {
fun browser() = browser { } fun browser() = browser { }
fun browser(body: KotlinJsBrowserDsl.() -> Unit) fun browser(body: KotlinJsBrowserDsl.() -> Unit)
fun browser(fn: Closure<*>) { fun browser(fn: Action<KotlinJsBrowserDsl>) {
browser { browser {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
fun nodejs() = nodejs { } fun nodejs() = nodejs { }
fun nodejs(body: KotlinJsNodeDsl.() -> Unit) fun nodejs(body: KotlinJsNodeDsl.() -> Unit)
fun nodejs(fn: Closure<*>) { fun nodejs(fn: Action<KotlinJsNodeDsl>) {
nodejs { nodejs {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
@@ -77,16 +76,16 @@ interface KotlinJsSubTargetDsl {
fun distribution(body: Distribution.() -> Unit) fun distribution(body: Distribution.() -> Unit)
@ExperimentalDistributionDsl @ExperimentalDistributionDsl
fun distribution(fn: Closure<*>) { fun distribution(fn: Action<Distribution>) {
distribution { distribution {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
fun testTask(body: KotlinJsTest.() -> Unit) fun testTask(body: KotlinJsTest.() -> Unit)
fun testTask(fn: Closure<*>) { fun testTask(fn: Action<KotlinJsTest>) {
testTask { testTask {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
@@ -95,23 +94,23 @@ interface KotlinJsSubTargetDsl {
interface KotlinJsBrowserDsl : KotlinJsSubTargetDsl { interface KotlinJsBrowserDsl : KotlinJsSubTargetDsl {
fun commonWebpackConfig(body: KotlinWebpackConfig.() -> Unit) fun commonWebpackConfig(body: KotlinWebpackConfig.() -> Unit)
fun commonWebpackConfig(fn: Closure<*>) { fun commonWebpackConfig(fn: Action<KotlinWebpackConfig>) {
commonWebpackConfig { commonWebpackConfig {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
fun runTask(body: KotlinWebpack.() -> Unit) fun runTask(body: KotlinWebpack.() -> Unit)
fun runTask(fn: Closure<*>) { fun runTask(fn: Action<KotlinWebpack>) {
runTask { runTask {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
fun webpackTask(body: KotlinWebpack.() -> Unit) fun webpackTask(body: KotlinWebpack.() -> Unit)
fun webpackTask(fn: Closure<*>) { fun webpackTask(fn: Action<KotlinWebpack>) {
webpackTask { webpackTask {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
@@ -119,9 +118,9 @@ interface KotlinJsBrowserDsl : KotlinJsSubTargetDsl {
fun dceTask(body: KotlinJsDce.() -> Unit) fun dceTask(body: KotlinJsDce.() -> Unit)
@ExperimentalDceDsl @ExperimentalDceDsl
fun dceTask(fn: Closure<*>) { fun dceTask(fn: Action<KotlinJsDce>) {
dceTask { dceTask {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
} }
@@ -5,8 +5,7 @@
package org.jetbrains.kotlin.gradle.targets.js.dsl package org.jetbrains.kotlin.gradle.targets.js.dsl
import groovy.lang.Closure import org.gradle.api.Action
import org.gradle.util.ConfigureUtil
import org.jetbrains.kotlin.gradle.plugin.KotlinTarget import org.jetbrains.kotlin.gradle.plugin.KotlinTarget
import org.jetbrains.kotlin.gradle.targets.js.binaryen.BinaryenExec import org.jetbrains.kotlin.gradle.targets.js.binaryen.BinaryenExec
import org.jetbrains.kotlin.gradle.targets.js.d8.D8Exec import org.jetbrains.kotlin.gradle.targets.js.d8.D8Exec
@@ -24,17 +23,17 @@ interface KotlinWasmSubTargetContainerDsl : KotlinTarget {
interface KotlinWasmTargetDsl : KotlinJsTargetDsl { interface KotlinWasmTargetDsl : KotlinJsTargetDsl {
fun d8() = d8 { } fun d8() = d8 { }
fun d8(body: KotlinWasmD8Dsl.() -> Unit) fun d8(body: KotlinWasmD8Dsl.() -> Unit)
fun d8(fn: Closure<*>) { fun d8(fn: Action<KotlinWasmD8Dsl>) {
d8 { d8 {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
fun applyBinaryen() = applyBinaryen { } fun applyBinaryen() = applyBinaryen { }
fun applyBinaryen(body: BinaryenExec.() -> Unit) fun applyBinaryen(body: BinaryenExec.() -> Unit)
fun applyBinaryen(fn: Closure<*>) { fun applyBinaryen(fn: Action<BinaryenExec>) {
applyBinaryen { applyBinaryen {
ConfigureUtil.configure(fn, this) fn.execute(this)
} }
} }
} }
@@ -10,7 +10,6 @@ import org.gradle.api.provider.Provider
import org.gradle.api.tasks.Copy import org.gradle.api.tasks.Copy
import org.gradle.language.base.plugins.LifecycleBasePlugin import org.gradle.language.base.plugins.LifecycleBasePlugin
import org.jetbrains.kotlin.gradle.dsl.KotlinJsDce import org.jetbrains.kotlin.gradle.dsl.KotlinJsDce
import org.jetbrains.kotlin.gradle.plugin.KotlinPlatformType
import org.jetbrains.kotlin.gradle.plugin.PropertiesProvider import org.jetbrains.kotlin.gradle.plugin.PropertiesProvider
import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinJsCompilation import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinJsCompilation
import org.jetbrains.kotlin.gradle.report.BuildMetricsReporterService import org.jetbrains.kotlin.gradle.report.BuildMetricsReporterService
@@ -31,7 +30,7 @@ import org.jetbrains.kotlin.gradle.utils.newFileProperty
import java.io.File import java.io.File
import javax.inject.Inject import javax.inject.Inject
open class KotlinBrowserJsIr @Inject constructor(target: KotlinJsIrTarget) : abstract class KotlinBrowserJsIr @Inject constructor(target: KotlinJsIrTarget) :
KotlinJsIrSubTarget(target, "browser"), KotlinJsIrSubTarget(target, "browser"),
KotlinJsBrowserDsl { KotlinJsBrowserDsl {
@@ -15,7 +15,7 @@ import org.jetbrains.kotlin.gradle.tasks.locateTask
import org.jetbrains.kotlin.gradle.tasks.withType import org.jetbrains.kotlin.gradle.tasks.withType
import javax.inject.Inject import javax.inject.Inject
open class KotlinD8Ir @Inject constructor(target: KotlinJsIrTarget) : abstract class KotlinD8Ir @Inject constructor(target: KotlinJsIrTarget) :
KotlinJsIrSubTargetBase(target, "d8"), KotlinJsIrSubTargetBase(target, "d8"),
KotlinWasmD8Dsl { KotlinWasmD8Dsl {
@@ -28,7 +28,7 @@ import org.jetbrains.kotlin.gradle.utils.lowerCamelCaseName
import org.jetbrains.kotlin.gradle.utils.setProperty import org.jetbrains.kotlin.gradle.utils.setProperty
import javax.inject.Inject import javax.inject.Inject
open class KotlinJsIrTarget abstract class KotlinJsIrTarget
@Inject @Inject
constructor( constructor(
project: Project, project: Project,
@@ -16,7 +16,7 @@ import org.jetbrains.kotlin.gradle.tasks.locateTask
import org.jetbrains.kotlin.gradle.tasks.withType import org.jetbrains.kotlin.gradle.tasks.withType
import javax.inject.Inject import javax.inject.Inject
open class KotlinNodeJsIr @Inject constructor(target: KotlinJsIrTarget) : abstract class KotlinNodeJsIr @Inject constructor(target: KotlinJsIrTarget) :
KotlinJsIrSubTargetBase(target, "node"), KotlinJsIrSubTargetBase(target, "node"),
KotlinJsNodeDsl { KotlinJsNodeDsl {
@@ -37,7 +37,7 @@ import java.io.File
import javax.inject.Inject import javax.inject.Inject
import org.jetbrains.kotlin.gradle.tasks.KotlinJsDce as KotlinJsDceTask import org.jetbrains.kotlin.gradle.tasks.KotlinJsDce as KotlinJsDceTask
open class KotlinBrowserJs @Inject constructor(target: KotlinJsTarget) : abstract class KotlinBrowserJs @Inject constructor(target: KotlinJsTarget) :
KotlinJsSubTarget(target, "browser"), KotlinJsSubTarget(target, "browser"),
KotlinJsBrowserDsl { KotlinJsBrowserDsl {
@@ -16,7 +16,7 @@ import org.jetbrains.kotlin.gradle.tasks.dependsOn
import org.jetbrains.kotlin.gradle.tasks.withType import org.jetbrains.kotlin.gradle.tasks.withType
import javax.inject.Inject import javax.inject.Inject
open class KotlinNodeJs @Inject constructor(target: KotlinJsTarget) : abstract class KotlinNodeJs @Inject constructor(target: KotlinJsTarget) :
KotlinJsSubTarget(target, "node"), KotlinJsSubTarget(target, "node"),
KotlinJsNodeDsl { KotlinJsNodeDsl {
override val testTaskDescription: String override val testTaskDescription: String