From fe9eb0310fd606057855e53737d24abba2dd7a79 Mon Sep 17 00:00:00 2001
From: Sergey Rostov <Sergey.Rostov@jetbrains.com>
Date: Thu, 7 Jun 2018 12:14:36 +0300
Subject: [PATCH] jps: add common source roots to the final platform modules as
 a workaround for absence of incremental k2metadata compiler

Original commit: 5455b6c0c0437c99de785fd86c378d759f71be0e
---
 .../jps/build/KotlinDirtySourceFilesHolder.kt    |  2 +-
 .../kotlin/jps/build/KotlinSourceRootProvider.kt | 16 ++++++++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinDirtySourceFilesHolder.kt b/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinDirtySourceFilesHolder.kt
index 06223112cc7..3604d5d5d99 100644
--- a/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinDirtySourceFilesHolder.kt
+++ b/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinDirtySourceFilesHolder.kt
@@ -92,7 +92,7 @@ class KotlinDirtySourceFilesHolder(
     val allDirtyFiles: Set<File>
         get() = byTarget.flatMapTo(mutableSetOf()) { it.value.dirty }
 
-    val allRemovedFilesFiles
+    val allRemovedFilesFiles: Set<File>
         get() = byTarget.flatMapTo(mutableSetOf()) { it.value.removed }
 }
 
diff --git a/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinSourceRootProvider.kt b/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinSourceRootProvider.kt
index c6e8bb87887..545d1996354 100644
--- a/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinSourceRootProvider.kt
+++ b/jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinSourceRootProvider.kt
@@ -14,6 +14,7 @@ import org.jetbrains.jps.incremental.ModuleBuildTarget
 import org.jetbrains.jps.model.java.JavaResourceRootType
 import org.jetbrains.jps.model.java.JavaSourceRootProperties
 import org.jetbrains.jps.model.java.JavaSourceRootType
+import org.jetbrains.jps.model.java.JpsJavaExtensionService
 import org.jetbrains.jps.model.module.JpsModule
 import org.jetbrains.jps.model.module.JpsModuleSourceRootType
 import org.jetbrains.kotlin.config.KotlinResourceRootType
@@ -85,6 +86,21 @@ class KotlinSourceRootProvider : AdditionalRootsProviderService<JavaSourceRootDe
                 )
             }
         }
+
+        // At the moment, incremental compilation is not supported by K2Metadata compiler.
+        // To avoid long running compilation of common modules, we do not run K2Metadata at all:
+        // instead all the common source roots are transitively added to the final platform modules.
+        JpsJavaExtensionService.dependencies(commonModule)
+            .also {
+                if (!target.isTests) it.productionOnly()
+            }
+            .processModules {
+                addCommonModuleSourceRoots(
+                    result,
+                    it,
+                    ModuleBuildTarget(it, target.targetType as JavaModuleBuildTargetType)
+                )
+            }
     }
 }