diff options
author | John Millikin <jmillikin@stripe.com> | 2018-01-12 08:18:00 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-01-12 08:24:16 -0800 |
commit | 38f815c2c8a4bc4d6cf79d366d200e37fa0935f3 (patch) | |
tree | 3762e8f7bfeba80b4021ebc3f70dc872da52a6b0 | |
parent | e5436745e1732f5e43fc55f0deb5b19e23ce8524 (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/BUILD | 1 | ||||
-rw-r--r-- | src/tools/remote/src/main/java/com/google/devtools/build/remote/worker/RemoteWorker.java | 5 |
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( |