Create copies instead of links for config and keystore files

uber · Jan 26, 2017 · 525df7a · 525df7a
1 parent aa660e7
commit 525df7a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/buildSrc/src/main/groovy/com/uber/okbuck/composer/android/KeystoreRuleComposer.groovy b/buildSrc/src/main/groovy/com/uber/okbuck/composer/android/KeystoreRuleComposer.groovy
@@ -2,6 +2,7 @@ package com.uber.okbuck.composer.android
 
 import com.uber.okbuck.core.model.android.AndroidAppTarget
 import com.uber.okbuck.core.util.FileUtil
+import org.apache.commons.io.FileUtils
 
 import java.nio.file.Files
 
@@ -25,7 +26,7 @@ final class KeystoreRuleComposer {
             String storeFilePropsName = "${target.name}.keystore.properties"
 
             File keyStoreCopy = new File(storeDir, storeFileName)
-            FileUtil.createLink(keystore.storeFile, keyStoreCopy)
+            FileUtils.copyFile(keystore.storeFile, keyStoreCopy)
 
             PrintWriter writer = new PrintWriter(new FileOutputStream(new File(storeDir,
                     storeFilePropsName)))

diff --git a/.../main/groovy/com/uber/okbuck/composer/android/TrasformDependencyWriterRuleComposer.groovy b/.../main/groovy/com/uber/okbuck/composer/android/TrasformDependencyWriterRuleComposer.groovy
@@ -63,7 +63,7 @@ final class TrasformDependencyWriterRuleComposer extends AndroidBuckRuleComposer
     static String getTransformConfigRuleForFile(Project project, File config) {
         String path = getTransformFilePathForFile(project, config)
         File configFile = new File("${TransformUtil.TRANSFORM_CACHE}/${path}")
-        FileUtil.createLink(config, configFile)
+        FileUtils.copyFile(config, configFile)
         return "//${TransformUtil.TRANSFORM_CACHE}:${path}"
     }
 

diff --git a/buildSrc/src/main/java/com/uber/okbuck/core/util/LintUtil.java b/buildSrc/src/main/java/com/uber/okbuck/core/util/LintUtil.java
@@ -14,7 +14,6 @@
 import java.io.File;
 import java.io.IOException;
 import java.nio.charset.Charset;
-import java.nio.file.Files;
 import java.util.Collections;
 
 public final class LintUtil {
@@ -72,14 +71,18 @@ private static String getLintwConfigName(Project project, File config) {
     @SuppressWarnings("ResultOfMethodCallIgnored")
     public static synchronized String getLintwConfigRule(Project project, File config) {
         File configFile = new File(LINT_DEPS_CACHE + "/" + getLintwConfigName(project, config));
-        FileUtil.createLink(config, configFile);
+        try {
+            FileUtils.copyFile(config, configFile);
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
         return "//" + LINT_DEPS_CACHE + ":" + getLintwConfigName(project, config);
     }
 
     public static DependencyCache getLintDepsCache(Project project) {
         OkBuckGradlePlugin okBuckGradlePlugin = ProjectUtil.getPlugin(project);
         if (okBuckGradlePlugin.lintDepCache == null) {
-            OkBuckExtension okBuckExtension= project.getExtensions().getByType(OkBuckExtension.class);
+            OkBuckExtension okBuckExtension = project.getExtensions().getByType(OkBuckExtension.class);
             okBuckGradlePlugin.lintDepCache = new DependencyCache("lint",
                     project.getRootProject(),
                     LINT_DEPS_CACHE,