Use the subprocess factory that uses the Win32 API on Windows.

We should really do something about the mess that is loading our JNI libraries -- io.bazel.EnableJNI is mentioned eight times in the code in various diverse contexts. This change is not the right place to do it, though.

--
MOS_MIGRATED_REVID=126570481

1 files changed, 7 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/shell/SubprocessBuilder.java b/src/main/java/com/google/devtools/build/lib/shell/SubprocessBuilder.java
index 0b6ecfb2f5..b89cf9644b 100644
--- a/src/main/java/com/google/devtools/build/lib/shell/SubprocessBuilder.java
+++ b/src/main/java/com/google/devtools/build/lib/shell/SubprocessBuilder.java
@@ -46,6 +46,12 @@ public class SubprocessBuilder {
   private File stderrFile;
   private File workingDirectory;
 
+  private static Subprocess.Factory factory = JavaSubprocessFactory.INSTANCE;
+
+  public static void setSubprocessFactory(Subprocess.Factory factory) {
+    SubprocessBuilder.factory = factory;
+  }
+
   public SubprocessBuilder() {
     stdoutAction = StreamAction.STREAM;
     stderrAction = StreamAction.STREAM;
@@ -154,6 +160,6 @@ public class SubprocessBuilder {
   }
 
   public Subprocess start() throws IOException {
-    return JavaSubprocessFactory.INSTANCE.create(this);
+    return factory.create(this);
   }
 }