diff --git a/libraries/tools/kotlin-gradle-plugin-api/api/kotlin-gradle-plugin-api.api b/libraries/tools/kotlin-gradle-plugin-api/api/kotlin-gradle-plugin-api.api index ea5b252e4b3..2d6167b4caf 100644 --- a/libraries/tools/kotlin-gradle-plugin-api/api/kotlin-gradle-plugin-api.api +++ b/libraries/tools/kotlin-gradle-plugin-api/api/kotlin-gradle-plugin-api.api @@ -1109,7 +1109,6 @@ public abstract interface class org/jetbrains/kotlin/gradle/plugin/KotlinSourceS public abstract fun getLanguageSettings ()Lorg/jetbrains/kotlin/gradle/plugin/LanguageSettingsBuilder; public abstract fun getResources ()Lorg/gradle/api/file/SourceDirectorySet; public abstract fun getRuntimeOnlyMetadataConfigurationName ()Ljava/lang/String; - public abstract fun getSourceSets ()Lorg/gradle/api/NamedDomainObjectContainer; public abstract fun kotlin (Lkotlin/jvm/functions/Function1;)Lorg/gradle/api/file/SourceDirectorySet; public abstract fun kotlin (Lorg/gradle/api/Action;)Lorg/gradle/api/file/SourceDirectorySet; public abstract fun languageSettings (Lkotlin/jvm/functions/Function1;)Lorg/jetbrains/kotlin/gradle/plugin/LanguageSettingsBuilder; diff --git a/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinSourceSet.kt b/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinSourceSet.kt index 80a11c30c02..092da201452 100644 --- a/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinSourceSet.kt +++ b/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinSourceSet.kt @@ -7,7 +7,6 @@ package org.jetbrains.kotlin.gradle.plugin import org.gradle.api.Action import org.gradle.api.Named -import org.gradle.api.NamedDomainObjectContainer import org.gradle.api.file.SourceDirectorySet import org.jetbrains.kotlin.gradle.dsl.KotlinGradlePluginDsl import org.jetbrains.kotlin.tooling.core.HasMutableExtras @@ -48,12 +47,4 @@ interface KotlinSourceSet : Named, HasProject, HasMutableExtras, HasKotlinDepend val customSourceFilesExtensions: Iterable // lazy iterable expected fun addCustomSourceFilesExtensions(extensions: List) {} - - @Deprecated( - "Accessing 'sourceSets' container inside another KotlinSourceSet is deprecated. " + - "Consider accessing 'sourceSets' only on the Kotlin extension level.", - level = DeprecationLevel.WARNING, - replaceWith = ReplaceWith("project.kotlin.sourceSets") - ) - val sourceSets: NamedDomainObjectContainer } diff --git a/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinTarget.kt b/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinTarget.kt index 14fd51077fd..e1b3f6b58d5 100644 --- a/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinTarget.kt +++ b/libraries/tools/kotlin-gradle-plugin-api/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinTarget.kt @@ -83,9 +83,8 @@ interface KotlinTarget : Named, HasAttributes, HasProject, HasMutableExtras { @Deprecated( "Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " + - "Consider accessing 'sourceSets' on the Kotlin extension level.", - level = DeprecationLevel.WARNING, - replaceWith = ReplaceWith("project.kotlin.sourceSets") + "Consider configuring 'sourceSets' on the Kotlin extension level.", + level = DeprecationLevel.WARNING ) val sourceSets: NamedDomainObjectContainer } diff --git a/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/mpp/InternalKotlinTarget.kt b/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/mpp/InternalKotlinTarget.kt index b4474fd9fb3..4fc2f15cee4 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/mpp/InternalKotlinTarget.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/mpp/InternalKotlinTarget.kt @@ -31,9 +31,8 @@ internal interface InternalKotlinTarget : KotlinTarget, HasMutableExtras { @Deprecated( "Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " + - "Consider accessing 'sourceSets' on the Kotlin extension level.", - level = DeprecationLevel.WARNING, - replaceWith = ReplaceWith("project.kotlin.sourceSets") + "Consider configuring 'sourceSets' on the Kotlin extension level.", + level = DeprecationLevel.WARNING ) override val sourceSets: NamedDomainObjectContainer get() = project.extensions.getByType().sourceSets diff --git a/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/sources/InternalKotlinSourceSet.kt b/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/sources/InternalKotlinSourceSet.kt index f52eb11f51a..f031a50e962 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/sources/InternalKotlinSourceSet.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/sources/InternalKotlinSourceSet.kt @@ -5,8 +5,6 @@ package org.jetbrains.kotlin.gradle.plugin.sources -import org.gradle.api.NamedDomainObjectContainer -import org.jetbrains.kotlin.gradle.dsl.KotlinProjectExtension import org.jetbrains.kotlin.gradle.plugin.KotlinCompilation import org.jetbrains.kotlin.gradle.plugin.KotlinPluginLifecycle import org.jetbrains.kotlin.gradle.plugin.KotlinSourceSet @@ -14,7 +12,6 @@ import org.jetbrains.kotlin.gradle.plugin.await import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinMetadataCompilation import org.jetbrains.kotlin.gradle.utils.MutableObservableSet import org.jetbrains.kotlin.gradle.utils.ObservableSet -import org.jetbrains.kotlin.gradle.utils.getByType internal val KotlinSourceSet.internal: InternalKotlinSourceSet get() = (this as? InternalKotlinSourceSet) ?: throw IllegalArgumentException( @@ -29,15 +26,6 @@ internal interface InternalKotlinSourceSet : KotlinSourceSet { /** Configuration that resolves into sources variants of all Source Set dependencies */ val dependencySourcesConfigurationName: String - - @Deprecated( - "Accessing 'sourceSets' container inside another KotlinSourceSet is deprecated. " + - "Consider accessing 'sourceSets' only on the Kotlin extension level.", - level = DeprecationLevel.WARNING, - replaceWith = ReplaceWith("project.kotlin.sourceSets") - ) - override val sourceSets: NamedDomainObjectContainer - get() = project.extensions.getByType().sourceSets } internal suspend fun InternalKotlinSourceSet.awaitPlatformCompilations(): Set> { diff --git a/libraries/tools/kotlin-gradle-plugin/src/functionalTest/kotlin/org/jetbrains/kotlin/gradle/unitTests/InternalKotlinCompilationTest.kt b/libraries/tools/kotlin-gradle-plugin/src/functionalTest/kotlin/org/jetbrains/kotlin/gradle/unitTests/InternalKotlinCompilationTest.kt index 6129e3b008a..1b5685d8029 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/functionalTest/kotlin/org/jetbrains/kotlin/gradle/unitTests/InternalKotlinCompilationTest.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/functionalTest/kotlin/org/jetbrains/kotlin/gradle/unitTests/InternalKotlinCompilationTest.kt @@ -7,30 +7,14 @@ package org.jetbrains.kotlin.gradle.unitTests -import org.jetbrains.kotlin.gradle.dsl.multiplatformExtension import org.jetbrains.kotlin.gradle.plugin.KotlinCompilation import org.jetbrains.kotlin.gradle.plugin.mpp.InternalKotlinCompilation import org.jetbrains.kotlin.gradle.util.assertAllImplementationsAlsoImplement -import org.jetbrains.kotlin.gradle.util.buildProjectWithMPP import org.junit.Test -import kotlin.test.assertEquals class InternalKotlinSourceSetTest { @Test fun `test - all implementations of KotlinCompilation - implement InternalKotlinCompilation`() { assertAllImplementationsAlsoImplement(KotlinCompilation::class, InternalKotlinCompilation::class) } - - @Test - fun `sourceSets inside should be equal to extension one`() { - val project = buildProjectWithMPP {} - - project.evaluate() - - val projectSourceSets = project.multiplatformExtension.sourceSets - project.multiplatformExtension.sourceSets.forEach { - @Suppress("DEPRECATION") - assertEquals(projectSourceSets, it.sourceSets) - } - } }