Use sync instead of delete-then-copy

google · Jan 22, 2025 · 0cd9e08 · 0cd9e08
1 parent 2810499
commit 0cd9e08
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/src/main/groovy/com/google/protobuf/gradle/CopyActionFacade.java b/src/main/groovy/com/google/protobuf/gradle/CopyActionFacade.java
@@ -50,6 +50,7 @@
 interface CopyActionFacade {
     WorkResult copy(Action<? super CopySpec> var1);
     WorkResult delete(Action<? super DeleteSpec> action);
+    WorkResult sync(Action<? super CopySpec> var1);
 
     @CompileStatic
     final class Loader {
@@ -79,6 +80,11 @@ public WorkResult copy(Action<? super CopySpec> action) {
         public WorkResult delete(Action<? super DeleteSpec> action) {
             return project.delete(action);
         }
+
+        @Override
+        public WorkResult sync(Action<? super CopySpec> action) {
+            return project.sync(action);
+        }
     }
 
     @CompileStatic
@@ -95,5 +101,10 @@ public WorkResult copy(Action<? super CopySpec> action) {
         public WorkResult delete(Action<? super DeleteSpec> action) {
             return getFileSystemOperations().delete(action);
         }
+
+        @Override
+        public WorkResult sync(Action<? super CopySpec> action) {
+            return getFileSystemOperations().sync(action);
+        }
     }
 }
diff --git a/src/main/groovy/com/google/protobuf/gradle/ProtobufExtract.groovy b/src/main/groovy/com/google/protobuf/gradle/ProtobufExtract.groovy
@@ -84,10 +84,7 @@ abstract class ProtobufExtract extends DefaultTask {
 
   @TaskAction
   public void extract() {
-    copyActionFacade.delete { spec ->
-      spec.delete(destDir)
-    }
-    copyActionFacade.copy { spec ->
+    copyActionFacade.sync { spec ->
       spec.includeEmptyDirs = false
       spec.from(inputProtoFiles)
       spec.into(destDir)