Skip to content

Commit 5fd45b2

Browse files
committed
avoid deleteRecursively(), using FileSystemOperations instead
1 parent 537360e commit 5fd45b2

File tree

3 files changed

+2
-7
lines changed

3 files changed

+2
-7
lines changed

firebase-dataconnect/gradleplugin/plugin/src/main/kotlin/com/google/firebase/dataconnect/gradle/plugin/DataConnectExecutableDownloadTask.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ abstract class DataConnectExecutableDownloadTask : DefaultTask() {
6969
logger.info("outputFile: {}", outputFile)
7070

7171
logger.info("Deleting build directory: {}", buildDirectory)
72-
buildDirectory.deleteRecursively()
72+
fileSystemOperations.delete { it.delete(buildDirectory) }
7373

7474
if (inputFile !== null && version !== null) {
7575
throw DataConnectGradleException(

firebase-dataconnect/gradleplugin/plugin/src/main/kotlin/com/google/firebase/dataconnect/gradle/plugin/DataConnectGenerateCodeTask.kt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,6 @@ abstract class DataConnectGenerateCodeTask : DefaultTask() {
8181
logger.info("outputDirectory={}", outputDirectory.absolutePath)
8282
logger.info("ktfmtJarFile={}", ktfmtJarFile?.absolutePath)
8383

84-
if (outputDirectory.exists()) {
85-
logger.info("Deleting directory: $outputDirectory")
86-
outputDirectory.deleteRecursively()
87-
}
88-
8984
if (configDirectory === null) {
9085
logger.info("No Data Connect config directories found; nothing to do")
9186
return

firebase-dataconnect/gradleplugin/plugin/src/main/kotlin/com/google/firebase/dataconnect/gradle/plugin/DataConnectMergeConfigDirectoriesTask.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ abstract class DataConnectMergeConfigDirectoriesTask : DefaultTask() {
6363
logger.info("mergedDirectory: {}", mergedDirectory?.absolutePath)
6464

6565
logger.info("Deleting build directory: {}", buildDirectory)
66-
buildDirectory.deleteRecursively()
66+
fileSystemOperations.delete { it.delete(buildDirectory) }
6767

6868
val configDirectories =
6969
buildList {

0 commit comments

Comments
 (0)