Minor: rename KotlinCompilation.sourceSets -> allSourceSets
This commit is contained in:
committed by
Yaroslav Chernyshev
parent
1143865e88
commit
4927777ffb
+11
-11
@@ -289,7 +289,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
val path = toCanonicalPath(target.jar!!.archiveFile!!.absolutePath)
|
val path = toCanonicalPath(target.jar!!.archiveFile!!.absolutePath)
|
||||||
val currentModules = userData[path] ?: ArrayList<String>().apply { userData[path] = this }
|
val currentModules = userData[path] ?: ArrayList<String>().apply { userData[path] = this }
|
||||||
// Test modules should not be added. Otherwise we could get dependnecy of java.mail on jvmTest
|
// Test modules should not be added. Otherwise we could get dependnecy of java.mail on jvmTest
|
||||||
val allSourceSets = target.compilations.filter { !it.isTestModule }.flatMap { it.sourceSets }.toSet()
|
val allSourceSets = target.compilations.filter { !it.isTestModule }.flatMap { it.allSourceSets }.toSet()
|
||||||
val availableViaDependsOn = allSourceSets.flatMap { it.dependsOnSourceSets }.mapNotNull { mppModel.sourceSets[it] }
|
val availableViaDependsOn = allSourceSets.flatMap { it.dependsOnSourceSets }.mapNotNull { mppModel.sourceSets[it] }
|
||||||
allSourceSets.union(availableViaDependsOn).forEach { sourceSet ->
|
allSourceSets.union(availableViaDependsOn).forEach { sourceSet ->
|
||||||
currentModules.add(getKotlinModuleId(gradleModule, sourceSet, resolverCtx))
|
currentModules.add(getKotlinModuleId(gradleModule, sourceSet, resolverCtx))
|
||||||
@@ -362,7 +362,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
resolverCtx
|
resolverCtx
|
||||||
) ?: continue
|
) ?: continue
|
||||||
kotlinSourceSet.externalSystemRunTasks =
|
kotlinSourceSet.externalSystemRunTasks =
|
||||||
compilation.sourceSets.firstNotNullResult { sourceSetToRunTasks[it] } ?: emptyList()
|
compilation.allSourceSets.firstNotNullResult { sourceSetToRunTasks[it] } ?: emptyList()
|
||||||
|
|
||||||
if (compilation.platform == KotlinPlatform.JVM || compilation.platform == KotlinPlatform.ANDROID) {
|
if (compilation.platform == KotlinPlatform.JVM || compilation.platform == KotlinPlatform.ANDROID) {
|
||||||
compilationData.targetCompatibility = (kotlinSourceSet.compilerArguments as? K2JVMCompilerArguments)?.jvmTarget
|
compilationData.targetCompatibility = (kotlinSourceSet.compilerArguments as? K2JVMCompilerArguments)?.jvmTarget
|
||||||
@@ -374,7 +374,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
for (sourceSet in compilation.sourceSets) {
|
for (sourceSet in compilation.allSourceSets) {
|
||||||
sourceSetToCompilationData.getOrPut(sourceSet.name) { LinkedHashSet() } += compilationData
|
sourceSetToCompilationData.getOrPut(sourceSet.name) { LinkedHashSet() } += compilationData
|
||||||
for (dependentSourceSetName in sourceSet.dependsOnSourceSets) {
|
for (dependentSourceSetName in sourceSet.dependsOnSourceSets) {
|
||||||
sourceSetToCompilationData.getOrPut(dependentSourceSetName) { LinkedHashSet() } += compilationData
|
sourceSetToCompilationData.getOrPut(dependentSourceSetName) { LinkedHashSet() } += compilationData
|
||||||
@@ -495,7 +495,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
)
|
)
|
||||||
}
|
}
|
||||||
val allRunTasks = testRunTasks + nativeMainRunTasks
|
val allRunTasks = testRunTasks + nativeMainRunTasks
|
||||||
compilation.sourceSets.forEach { sourceSet ->
|
compilation.allSourceSets.forEach { sourceSet ->
|
||||||
sourceSetToRunTasks.getOrPut(sourceSet) { LinkedHashSet() } += allRunTasks
|
sourceSetToRunTasks.getOrPut(sourceSet) { LinkedHashSet() } += allRunTasks
|
||||||
sourceSet.dependsOnSourceSets.forEach { dependentModule ->
|
sourceSet.dependsOnSourceSets.forEach { dependentModule ->
|
||||||
dependsOnReverseGraph.getOrPut(dependentModule) { LinkedHashSet() } += sourceSet
|
dependsOnReverseGraph.getOrPut(dependentModule) { LinkedHashSet() } += sourceSet
|
||||||
@@ -519,7 +519,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
val mppModel = resolverCtx.getMppModel(gradleModule) ?: return
|
val mppModel = resolverCtx.getMppModel(gradleModule) ?: return
|
||||||
val sourceSetToPackagePrefix = mppModel.targets.flatMap { it.compilations }
|
val sourceSetToPackagePrefix = mppModel.targets.flatMap { it.compilations }
|
||||||
.flatMap { compilation ->
|
.flatMap { compilation ->
|
||||||
compilation.sourceSets.map { sourceSet -> sourceSet.name to compilation.kotlinTaskProperties.packagePrefix }
|
compilation.allSourceSets.map { sourceSet -> sourceSet.name to compilation.kotlinTaskProperties.packagePrefix }
|
||||||
}
|
}
|
||||||
.toMap()
|
.toMap()
|
||||||
if (resolverCtx.getExtraProject(gradleModule, ExternalProject::class.java) == null) return
|
if (resolverCtx.getExtraProject(gradleModule, ExternalProject::class.java) == null) return
|
||||||
@@ -601,7 +601,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
ideProject
|
ideProject
|
||||||
)
|
)
|
||||||
KotlinNativeLibrariesFixer.applyTo(dataNode, ideProject)
|
KotlinNativeLibrariesFixer.applyTo(dataNode, ideProject)
|
||||||
for (sourceSet in compilation.sourceSets) {
|
for (sourceSet in compilation.allSourceSets) {
|
||||||
(sourceSet.dependsOnSourceSets + sourceSet.name).forEach {
|
(sourceSet.dependsOnSourceSets + sourceSet.name).forEach {
|
||||||
sourceSetToCompilations
|
sourceSetToCompilations
|
||||||
.getOrPut(it) { mutableListOf() }
|
.getOrPut(it) { mutableListOf() }
|
||||||
@@ -655,7 +655,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
val dependeeSourceSets = closedSourceSetGraph.successors(sourceSet)
|
val dependeeSourceSets = closedSourceSetGraph.successors(sourceSet)
|
||||||
val sourceSetInfos = if (isAndroid) {
|
val sourceSetInfos = if (isAndroid) {
|
||||||
ideModule.kotlinAndroidSourceSets?.filter {
|
ideModule.kotlinAndroidSourceSets?.filter {
|
||||||
(it.kotlinModule as? KotlinCompilation)?.sourceSets?.contains(sourceSet) ?: false
|
(it.kotlinModule as? KotlinCompilation)?.allSourceSets?.contains(sourceSet) ?: false
|
||||||
} ?: emptyList()
|
} ?: emptyList()
|
||||||
} else {
|
} else {
|
||||||
listOfNotNull(fromDataNode.kotlinSourceSet)
|
listOfNotNull(fromDataNode.kotlinSourceSet)
|
||||||
@@ -940,7 +940,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
if (effectiveClassesDir != null) {
|
if (effectiveClassesDir != null) {
|
||||||
sourcesWithTypes += compilation.sourceType to DefaultExternalSourceDirectorySet().also { dirSet ->
|
sourcesWithTypes += compilation.sourceType to DefaultExternalSourceDirectorySet().also { dirSet ->
|
||||||
dirSet.outputDir = effectiveClassesDir
|
dirSet.outputDir = effectiveClassesDir
|
||||||
dirSet.srcDirs = compilation.sourceSets.flatMapTo(LinkedHashSet()) { it.sourceDirs }
|
dirSet.srcDirs = compilation.allSourceSets.flatMapTo(LinkedHashSet()) { it.sourceDirs }
|
||||||
dirSet.gradleOutputDirs += compilation.output.classesDirs
|
dirSet.gradleOutputDirs += compilation.output.classesDirs
|
||||||
dirSet.setInheritedCompilerOutput(false)
|
dirSet.setInheritedCompilerOutput(false)
|
||||||
}
|
}
|
||||||
@@ -948,7 +948,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
if (resourcesDir != null) {
|
if (resourcesDir != null) {
|
||||||
sourcesWithTypes += compilation.resourceType to DefaultExternalSourceDirectorySet().also { dirSet ->
|
sourcesWithTypes += compilation.resourceType to DefaultExternalSourceDirectorySet().also { dirSet ->
|
||||||
dirSet.outputDir = resourcesDir
|
dirSet.outputDir = resourcesDir
|
||||||
dirSet.srcDirs = compilation.sourceSets.flatMapTo(LinkedHashSet()) { it.resourceDirs }
|
dirSet.srcDirs = compilation.allSourceSets.flatMapTo(LinkedHashSet()) { it.resourceDirs }
|
||||||
dirSet.gradleOutputDirs += resourcesDir
|
dirSet.gradleOutputDirs += resourcesDir
|
||||||
dirSet.setInheritedCompilerOutput(false)
|
dirSet.setInheritedCompilerOutput(false)
|
||||||
}
|
}
|
||||||
@@ -1040,7 +1040,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
sourceSetInfo.gradleModuleId = getModuleId(resolverCtx, gradleModule)
|
sourceSetInfo.gradleModuleId = getModuleId(resolverCtx, gradleModule)
|
||||||
sourceSetInfo.actualPlatforms.addSimplePlatforms(listOf(compilation.platform))
|
sourceSetInfo.actualPlatforms.addSimplePlatforms(listOf(compilation.platform))
|
||||||
sourceSetInfo.isTestModule = compilation.isTestModule
|
sourceSetInfo.isTestModule = compilation.isTestModule
|
||||||
sourceSetInfo.dependsOn = compilation.sourceSets.flatMap { it.dependsOnSourceSets }.map {
|
sourceSetInfo.dependsOn = compilation.allSourceSets.flatMap { it.dependsOnSourceSets }.map {
|
||||||
getGradleModuleQualifiedName(resolverCtx, gradleModule, it)
|
getGradleModuleQualifiedName(resolverCtx, gradleModule, it)
|
||||||
}.distinct().toList()
|
}.distinct().toList()
|
||||||
sourceSetInfo.compilerArguments =
|
sourceSetInfo.compilerArguments =
|
||||||
@@ -1050,7 +1050,7 @@ open class KotlinMPPGradleProjectResolver : AbstractProjectResolverExtensionComp
|
|||||||
sourceSetInfo.dependencyClasspath = compilation.dependencyClasspath.toList()
|
sourceSetInfo.dependencyClasspath = compilation.dependencyClasspath.toList()
|
||||||
sourceSetInfo.defaultCompilerArguments =
|
sourceSetInfo.defaultCompilerArguments =
|
||||||
createCompilerArguments(compilation.arguments.defaultArguments.toList(), compilation.platform)
|
createCompilerArguments(compilation.arguments.defaultArguments.toList(), compilation.platform)
|
||||||
sourceSetInfo.addSourceSets(compilation.sourceSets, compilation.fullName(), gradleModule, resolverCtx)
|
sourceSetInfo.addSourceSets(compilation.allSourceSets, compilation.fullName(), gradleModule, resolverCtx)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -102,7 +102,7 @@ interface KotlinCompilation : KotlinModule {
|
|||||||
* All source sets participated in this compilation, including those available
|
* All source sets participated in this compilation, including those available
|
||||||
* via dependsOn.
|
* via dependsOn.
|
||||||
*/
|
*/
|
||||||
val sourceSets: Collection<KotlinSourceSet>
|
val allSourceSets: Collection<KotlinSourceSet>
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Only default source sets of this compilation, i.e. those which are included
|
* Only default source sets of this compilation, i.e. those which are included
|
||||||
|
|||||||
@@ -365,12 +365,12 @@ class KotlinMPPGradleModelBuilder : ModelBuilderService {
|
|||||||
private fun KotlinCompilationImpl.addDependsOnSourceSetsToCompilation(
|
private fun KotlinCompilationImpl.addDependsOnSourceSetsToCompilation(
|
||||||
importingContext: MultiplatformModelImportingContext
|
importingContext: MultiplatformModelImportingContext
|
||||||
): KotlinCompilationImpl {
|
): KotlinCompilationImpl {
|
||||||
val dependsOnSourceSets = this.sourceSets.flatMap { it.dependsOnSourceSets }.mapNotNull { importingContext.sourceSetByName(it) }
|
val dependsOnSourceSets = this.allSourceSets.flatMap { it.dependsOnSourceSets }.mapNotNull { importingContext.sourceSetByName(it) }
|
||||||
|
|
||||||
return KotlinCompilationImpl(
|
return KotlinCompilationImpl(
|
||||||
this.name,
|
this.name,
|
||||||
this.sourceSets.union(dependsOnSourceSets),
|
this.allSourceSets.union(dependsOnSourceSets),
|
||||||
this.sourceSets,
|
this.allSourceSets,
|
||||||
this.dependencies,
|
this.dependencies,
|
||||||
this.output,
|
this.output,
|
||||||
this.arguments,
|
this.arguments,
|
||||||
|
|||||||
@@ -127,7 +127,7 @@ data class KotlinNativeCompilationExtensionsImpl(
|
|||||||
|
|
||||||
data class KotlinCompilationImpl(
|
data class KotlinCompilationImpl(
|
||||||
override val name: String,
|
override val name: String,
|
||||||
override val sourceSets: Collection<KotlinSourceSet>,
|
override val allSourceSets: Collection<KotlinSourceSet>,
|
||||||
override val defaultSourceSets: Collection<KotlinSourceSet>,
|
override val defaultSourceSets: Collection<KotlinSourceSet>,
|
||||||
override val dependencies: Array<KotlinDependencyId>,
|
override val dependencies: Array<KotlinDependencyId>,
|
||||||
override val output: KotlinCompilationOutput,
|
override val output: KotlinCompilationOutput,
|
||||||
@@ -140,7 +140,7 @@ data class KotlinCompilationImpl(
|
|||||||
// create deep copy
|
// create deep copy
|
||||||
constructor(kotlinCompilation: KotlinCompilation, cloningCache: MutableMap<Any, Any>) : this(
|
constructor(kotlinCompilation: KotlinCompilation, cloningCache: MutableMap<Any, Any>) : this(
|
||||||
kotlinCompilation.name,
|
kotlinCompilation.name,
|
||||||
cloneSourceSetsWithCaching(kotlinCompilation.sourceSets, cloningCache),
|
cloneSourceSetsWithCaching(kotlinCompilation.allSourceSets, cloningCache),
|
||||||
cloneSourceSetsWithCaching(kotlinCompilation.defaultSourceSets, cloningCache),
|
cloneSourceSetsWithCaching(kotlinCompilation.defaultSourceSets, cloningCache),
|
||||||
kotlinCompilation.dependencies,
|
kotlinCompilation.dependencies,
|
||||||
KotlinCompilationOutputImpl(kotlinCompilation.output),
|
KotlinCompilationOutputImpl(kotlinCompilation.output),
|
||||||
|
|||||||
@@ -109,7 +109,7 @@ internal class MultiplatformModelImportingContextImpl(override val project: Proj
|
|||||||
|
|
||||||
for (target in targets) {
|
for (target in targets) {
|
||||||
for (compilation in target.compilations) {
|
for (compilation in target.compilations) {
|
||||||
for (sourceSet in compilation.sourceSets) {
|
for (sourceSet in compilation.allSourceSets) {
|
||||||
sourceSetToCompilations.getOrPut(sourceSet) { LinkedHashSet() } += compilation
|
sourceSetToCompilations.getOrPut(sourceSet) { LinkedHashSet() } += compilation
|
||||||
sourceSet.dependsOnSourceSets.mapNotNull { sourceSetByName(it) }.forEach {
|
sourceSet.dependsOnSourceSets.mapNotNull { sourceSetByName(it) }.forEach {
|
||||||
sourceSetToCompilations.getOrPut(it) { LinkedHashSet() } += compilation
|
sourceSetToCompilations.getOrPut(it) { LinkedHashSet() } += compilation
|
||||||
|
|||||||
Reference in New Issue
Block a user