Revert "[Gradle] Add deprecated accessor to sourceSets inside KotlinSourceSet"
This reverts commit 5fba2962b9.
This commit is contained in:
committed by
Space Team
parent
cf290a73fc
commit
e8704cc556
@@ -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 getLanguageSettings ()Lorg/jetbrains/kotlin/gradle/plugin/LanguageSettingsBuilder;
|
||||||
public abstract fun getResources ()Lorg/gradle/api/file/SourceDirectorySet;
|
public abstract fun getResources ()Lorg/gradle/api/file/SourceDirectorySet;
|
||||||
public abstract fun getRuntimeOnlyMetadataConfigurationName ()Ljava/lang/String;
|
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 (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 kotlin (Lorg/gradle/api/Action;)Lorg/gradle/api/file/SourceDirectorySet;
|
||||||
public abstract fun languageSettings (Lkotlin/jvm/functions/Function1;)Lorg/jetbrains/kotlin/gradle/plugin/LanguageSettingsBuilder;
|
public abstract fun languageSettings (Lkotlin/jvm/functions/Function1;)Lorg/jetbrains/kotlin/gradle/plugin/LanguageSettingsBuilder;
|
||||||
|
|||||||
-9
@@ -7,7 +7,6 @@ package org.jetbrains.kotlin.gradle.plugin
|
|||||||
|
|
||||||
import org.gradle.api.Action
|
import org.gradle.api.Action
|
||||||
import org.gradle.api.Named
|
import org.gradle.api.Named
|
||||||
import org.gradle.api.NamedDomainObjectContainer
|
|
||||||
import org.gradle.api.file.SourceDirectorySet
|
import org.gradle.api.file.SourceDirectorySet
|
||||||
import org.jetbrains.kotlin.gradle.dsl.KotlinGradlePluginDsl
|
import org.jetbrains.kotlin.gradle.dsl.KotlinGradlePluginDsl
|
||||||
import org.jetbrains.kotlin.tooling.core.HasMutableExtras
|
import org.jetbrains.kotlin.tooling.core.HasMutableExtras
|
||||||
@@ -48,12 +47,4 @@ interface KotlinSourceSet : Named, HasProject, HasMutableExtras, HasKotlinDepend
|
|||||||
|
|
||||||
val customSourceFilesExtensions: Iterable<String> // lazy iterable expected
|
val customSourceFilesExtensions: Iterable<String> // lazy iterable expected
|
||||||
fun addCustomSourceFilesExtensions(extensions: List<String>) {}
|
fun addCustomSourceFilesExtensions(extensions: List<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")
|
|
||||||
)
|
|
||||||
val sourceSets: NamedDomainObjectContainer<KotlinSourceSet>
|
|
||||||
}
|
}
|
||||||
|
|||||||
+2
-3
@@ -83,9 +83,8 @@ interface KotlinTarget : Named, HasAttributes, HasProject, HasMutableExtras {
|
|||||||
|
|
||||||
@Deprecated(
|
@Deprecated(
|
||||||
"Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " +
|
"Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " +
|
||||||
"Consider accessing 'sourceSets' on the Kotlin extension level.",
|
"Consider configuring 'sourceSets' on the Kotlin extension level.",
|
||||||
level = DeprecationLevel.WARNING,
|
level = DeprecationLevel.WARNING
|
||||||
replaceWith = ReplaceWith("project.kotlin.sourceSets")
|
|
||||||
)
|
)
|
||||||
val sourceSets: NamedDomainObjectContainer<KotlinSourceSet>
|
val sourceSets: NamedDomainObjectContainer<KotlinSourceSet>
|
||||||
}
|
}
|
||||||
|
|||||||
+2
-3
@@ -31,9 +31,8 @@ internal interface InternalKotlinTarget : KotlinTarget, HasMutableExtras {
|
|||||||
|
|
||||||
@Deprecated(
|
@Deprecated(
|
||||||
"Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " +
|
"Accessing 'sourceSets' container on the Kotlin target level DSL is deprecated. " +
|
||||||
"Consider accessing 'sourceSets' on the Kotlin extension level.",
|
"Consider configuring 'sourceSets' on the Kotlin extension level.",
|
||||||
level = DeprecationLevel.WARNING,
|
level = DeprecationLevel.WARNING
|
||||||
replaceWith = ReplaceWith("project.kotlin.sourceSets")
|
|
||||||
)
|
)
|
||||||
override val sourceSets: NamedDomainObjectContainer<KotlinSourceSet>
|
override val sourceSets: NamedDomainObjectContainer<KotlinSourceSet>
|
||||||
get() = project.extensions.getByType<KotlinProjectExtension>().sourceSets
|
get() = project.extensions.getByType<KotlinProjectExtension>().sourceSets
|
||||||
|
|||||||
-12
@@ -5,8 +5,6 @@
|
|||||||
|
|
||||||
package org.jetbrains.kotlin.gradle.plugin.sources
|
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.KotlinCompilation
|
||||||
import org.jetbrains.kotlin.gradle.plugin.KotlinPluginLifecycle
|
import org.jetbrains.kotlin.gradle.plugin.KotlinPluginLifecycle
|
||||||
import org.jetbrains.kotlin.gradle.plugin.KotlinSourceSet
|
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.plugin.mpp.KotlinMetadataCompilation
|
||||||
import org.jetbrains.kotlin.gradle.utils.MutableObservableSet
|
import org.jetbrains.kotlin.gradle.utils.MutableObservableSet
|
||||||
import org.jetbrains.kotlin.gradle.utils.ObservableSet
|
import org.jetbrains.kotlin.gradle.utils.ObservableSet
|
||||||
import org.jetbrains.kotlin.gradle.utils.getByType
|
|
||||||
|
|
||||||
internal val KotlinSourceSet.internal: InternalKotlinSourceSet
|
internal val KotlinSourceSet.internal: InternalKotlinSourceSet
|
||||||
get() = (this as? InternalKotlinSourceSet) ?: throw IllegalArgumentException(
|
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 */
|
/** Configuration that resolves into sources variants of all Source Set dependencies */
|
||||||
val dependencySourcesConfigurationName: String
|
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<KotlinSourceSet>
|
|
||||||
get() = project.extensions.getByType<KotlinProjectExtension>().sourceSets
|
|
||||||
}
|
}
|
||||||
|
|
||||||
internal suspend fun InternalKotlinSourceSet.awaitPlatformCompilations(): Set<KotlinCompilation<*>> {
|
internal suspend fun InternalKotlinSourceSet.awaitPlatformCompilations(): Set<KotlinCompilation<*>> {
|
||||||
|
|||||||
-16
@@ -7,30 +7,14 @@
|
|||||||
|
|
||||||
package org.jetbrains.kotlin.gradle.unitTests
|
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.KotlinCompilation
|
||||||
import org.jetbrains.kotlin.gradle.plugin.mpp.InternalKotlinCompilation
|
import org.jetbrains.kotlin.gradle.plugin.mpp.InternalKotlinCompilation
|
||||||
import org.jetbrains.kotlin.gradle.util.assertAllImplementationsAlsoImplement
|
import org.jetbrains.kotlin.gradle.util.assertAllImplementationsAlsoImplement
|
||||||
import org.jetbrains.kotlin.gradle.util.buildProjectWithMPP
|
|
||||||
import org.junit.Test
|
import org.junit.Test
|
||||||
import kotlin.test.assertEquals
|
|
||||||
|
|
||||||
class InternalKotlinSourceSetTest {
|
class InternalKotlinSourceSetTest {
|
||||||
@Test
|
@Test
|
||||||
fun `test - all implementations of KotlinCompilation - implement InternalKotlinCompilation`() {
|
fun `test - all implementations of KotlinCompilation - implement InternalKotlinCompilation`() {
|
||||||
assertAllImplementationsAlsoImplement(KotlinCompilation::class, InternalKotlinCompilation::class)
|
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)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user