aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar John Millikin <jmillikin@stripe.com>2018-01-12 08:18:00 -0800
committerGravatar Copybara-Service <copybara-piper@google.com>2018-01-12 08:24:16 -0800
commit38f815c2c8a4bc4d6cf79d366d200e37fa0935f3 (patch)
tree3762e8f7bfeba80b4021ebc3f70dc872da52a6b0
parente5436745e1732f5e43fc55f0deb5b19e23ce8524 (diff)
Always use the JavaIO VFS implementation in the remote worker.
The JNI implementation doesn't work from a deployable jar. Fixes https://github.com/bazelbuild/bazel/issues/3249 cc @ulfjack Closes #4438. PiperOrigin-RevId: 181746081
-rw-r--r--src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/BUILD1
-rw-r--r--src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java5
2 files changed, 1 insertions, 5 deletions
diff --git a/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/BUILD b/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/BUILD
index ee2abe9e45..fc890d8e43 100644
--- a/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/BUILD
+++ b/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/BUILD
@@ -17,7 +17,6 @@ java_library(
"//src/main/java/com/google/devtools/build/lib:process_util",
"//src/main/java/com/google/devtools/build/lib:runtime",
"//src/main/java/com/google/devtools/build/lib:single-line-formatter",
- "//src/main/java/com/google/devtools/build/lib:unix",
"//src/main/java/com/google/devtools/build/lib:util",
"//src/main/java/com/google/devtools/build/lib/actions",
"//src/main/java/com/google/devtools/build/lib/remote",
diff --git a/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java b/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java
index 12a98d348b..8035e78162 100644
--- a/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java
+++ b/src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java
@@ -36,7 +36,6 @@ import com.google.devtools.build.lib.runtime.LinuxSandboxUtil;
import com.google.devtools.build.lib.shell.Command;
import com.google.devtools.build.lib.shell.CommandException;
import com.google.devtools.build.lib.shell.CommandResult;
-import com.google.devtools.build.lib.unix.UnixFileSystem;
import com.google.devtools.build.lib.util.OS;
import com.google.devtools.build.lib.util.ProcessUtils;
import com.google.devtools.build.lib.util.SingleLineFormatter;
@@ -97,9 +96,7 @@ public final class RemoteWorker {
} catch (OptionsParsingException e) {
throw new Error("The specified hash function '" + value + "' is not supported.");
}
- return OS.getCurrent() == OS.WINDOWS
- ? new JavaIoFileSystem(hashFunction)
- : new UnixFileSystem(hashFunction);
+ return new JavaIoFileSystem(hashFunction);
}
public RemoteWorker(