Don't allow launching from a directory with a space in the name

Further improvements on #167.

--
MOS_MIGRATED_REVID=92459390

1 files changed, 8 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
index 6d0c14b035..d1830159ea 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
@@ -22,6 +22,7 @@ import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.ListMultimap;
 import com.google.common.collect.Lists;
 import com.google.common.io.Flushables;
+import com.google.devtools.build.lib.Constants;
 import com.google.devtools.build.lib.events.Event;
 import com.google.devtools.build.lib.events.EventHandler;
 import com.google.devtools.build.lib.events.Reporter;
@@ -155,6 +156,13 @@ public class BlazeCommandDispatcher {
     }
 
     Path workspace = runtime.getWorkspace();
+    // TODO(kchodorow): Remove this once spaces are supported.
+    if (workspace.getPathString().contains(" ")) {
+      outErr.printErrLn(Constants.PRODUCT_NAME + " does not currently work properly from paths "
+          + "containing spaces (" + workspace + ").");
+      return ExitCode.LOCAL_ENVIRONMENTAL_ERROR;
+    }
+
     Path doNotBuild = workspace.getParentDirectory().getRelative(
         BlazeRuntime.DO_NOT_BUILD_FILE_NAME);
     if (doNotBuild.exists()) {