Rebuild only if proto changed.

Original commit: 018b58b51f

jps/jps-plugin/src/org/jetbrains/jet/jps/build/KotlinBuilder.java

@@ -198,6 +198,8 @@ public class KotlinBuilder extends ModuleLevelBuilder {
         IncrementalCache cache = new IncrementalCache(KotlinBuilderModuleScriptGenerator.getIncrementalCacheDir(context));
 
         try {
+            boolean significantChanges = false;
+
             for (SimpleOutputItem outputItem : outputItemCollector.getOutputs()) {
                 BuildTarget<?> target = null;
                 Collection<File> sourceFiles = outputItem.getSourceFiles();
@@ -215,7 +217,9 @@ public class KotlinBuilder extends ModuleLevelBuilder {
                 File outputFile = outputItem.getOutputFile();
 
                 if (IncrementalCompilation.ENABLED) {
-                    cache.saveFileToCache(target.getId(), outputFile);
+                    if (cache.saveFileToCache(target.getId(), outputFile)) {
+                        significantChanges = true;
+                    }
                 }
 
                 outputConsumer.registerOutputFile(target, outputFile, paths(sourceFiles));
@@ -223,18 +227,21 @@ public class KotlinBuilder extends ModuleLevelBuilder {
 
             if (IncrementalCompilation.ENABLED) {
                 // TODO should mark dependencies as dirty, as well
-                FSOperations.markDirty(context, chunk, new FileFilter() {
-                    @Override
-                    public boolean accept(@NotNull File file) {
-                        return !allCompiledFiles.contains(file);
-                    }
-                });
+                if (significantChanges) {
+                    FSOperations.markDirty(context, chunk, new FileFilter() {
+                        @Override
+                        public boolean accept(@NotNull File file) {
+                            return !allCompiledFiles.contains(file);
+                        }
+                    });
+                }
                 return ExitCode.ADDITIONAL_PASS_REQUIRED;
             }
             else {
                 return ExitCode.OK;
             }
-        } finally {
+        }
+        finally {
             cache.close();
         }
     }

jps/jps-plugin/test/org/jetbrains/jet/jps/build/IncrementalJpsTest.kt

@@ -108,6 +108,14 @@ public class IncrementalJpsTest : JpsBuildTestCase() {
         doTest()
     }
 
+    fun testReturnTypeChanged() {
+        doTest()
+    }
+
+    fun testTopLevelFunctionSameSignature() {
+        doTest()
+    }
+
     private class MyLogger(val rootPath: String) : ProjectBuilderLoggerBase() {
         private val logBuf = StringBuilder()
         public val log: String

jps/jps-plugin/testData/incremental/independentClasses/build.log

@@ -3,10 +3,4 @@ out/production/module/test/Foo.class
 End of files
 Compiling files:
 src/Foo.kt
-End of files
-Cleaning output files:
-out/production/module/test/Bar.class
-End of files
-Compiling files:
-src/Bar.kt
-End of files
+End of files

jps/jps-plugin/testData/incremental/returnTypeChanged/build.log

@@ -0,0 +1,14 @@
+Cleaning output files:
+out/production/module/test/TestPackage-fun-*.class
+out/production/module/test/TestPackage.class
+End of files
+Compiling files:
+src/fun.kt
+End of files
+Cleaning output files:
+out/production/module/test/TestPackage-usage-*.class
+out/production/module/test/TestPackage.class
+End of files
+Compiling files:
+src/usage.kt
+End of files

jps/jps-plugin/testData/incremental/returnTypeChanged/fun.kt

@@ -0,0 +1,4 @@
+package test
+
+fun foo() {
+}

jps/jps-plugin/testData/incremental/returnTypeChanged/fun.kt.new

@@ -0,0 +1,3 @@
+package test
+
+fun foo(): String = ":)"

jps/jps-plugin/testData/incremental/returnTypeChanged/usage.kt

@@ -0,0 +1,5 @@
+package test
+
+fun usage() {
+    foo()
+}

jps/jps-plugin/testData/incremental/simpleClassDependency/build.log

@@ -4,10 +4,4 @@ out/production/module/test/Foo.class
 End of files
 Compiling files:
 src/Foo.kt
-End of files
-Cleaning output files:
-out/production/module/test/Bar.class
-End of files
-Compiling files:
-src/Bar.kt
 End of files

jps/jps-plugin/testData/incremental/topLevelFunctionSameSignature/build.log

@@ -0,0 +1,7 @@
+Cleaning output files:
+out/production/module/test/TestPackage-fun-*.class
+out/production/module/test/TestPackage.class
+End of files
+Compiling files:
+src/fun.kt
+End of files

jps/jps-plugin/testData/incremental/topLevelFunctionSameSignature/fun.kt

@@ -0,0 +1,4 @@
+package test
+
+fun foo() {
+}

jps/jps-plugin/testData/incremental/topLevelFunctionSameSignature/fun.kt.new

@@ -0,0 +1,5 @@
+package test
+
+fun foo() {
+    println("body is changed, but it doesn't matter, we won't rebuild dependencies")
+}

jps/jps-plugin/testData/incremental/topLevelFunctionSameSignature/usage.kt

@@ -0,0 +1,5 @@
+package test
+
+fun usage() {
+    foo()
+}

jps/jps-plugin/testData/incremental/topLevelMembersInTwoPackages/build.log

@@ -4,11 +4,4 @@ out/production/module/test/TestPackage.class
 End of files
 Compiling files:
 src/a.kt
-End of files
-Cleaning output files:
-out/production/module/test/TestPackage-b-*.class
-out/production/module/test/TestPackage.class
-End of files
-Compiling files:
-src/b.kt
-End of files
+End of files