sandbox: We have to move all generated outputs, not just regular files.

Fix for a part of bug #397. -- MOS_MIGRATED_REVID=102343972
bazelbuild · Sep 4, 2015 · 3696355 · 3696355
1 parent e8f8dc5
commit 3696355
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/NamespaceSandboxRunner.java b/src/main/java/com/google/devtools/build/lib/sandbox/NamespaceSandboxRunner.java
@@ -163,9 +163,7 @@ private void copyOutputs(Collection<? extends ActionInput> outputs) throws IOExc
       Path source = sandboxExecRoot.getRelative(output.getExecPathString());
       Path target = execRoot.getRelative(output.getExecPathString());
       FileSystemUtils.createDirectoryAndParents(target.getParentDirectory());
-      if (source.isFile()) {
-        Files.move(new File(source.getPathString()), new File(target.getPathString()));
-      }
+      Files.move(source.getPathFile(), target.getPathFile());
     }
   }