Escape exec *args* only if space

This matches npm's functionality. It's also been tested against the "some\"file" test, etc. It's hard too...and late.

lib/config/exec.js

@@ -69,10 +69,12 @@ function parseExecutable(str) {
     execArgs.push(token);
   }
 
-  return {
+  var result = {
     exec: exec,
     execArgs: execArgs,
   };
+
+  return result;
 }
 
 /**

lib/monitor/run.js

@@ -47,6 +47,13 @@ function run(options) {
     shFlag = '/c';
   }
 
+  cmd.args = cmd.args.map(function (arg) {
+    if (arg.indexOf(' ') !== -1) {
+      arg = '"' + arg.replace(/"/g, '\\"') + '"';
+    }
+    return arg;
+  });
+
   var args = [cmd.executable].concat(cmd.args).join(' ').trim();
   var spawnArgs = [sh, [shFlag, args]];
 
@@ -96,6 +103,12 @@ function run(options) {
   });
 
   child.on('exit', function (code, signal) {
+    if (code === 2) {
+      // something wrong with parsed command
+      utils.log.error('failed to start process, possible issue with exec arguments');
+      process.exit();
+    }
+
     // In case we killed the app ourselves, set the signal thusly
     if (killedAfterChange) {
       killedAfterChange = false;

test/fixtures/some file

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+console.log(process.env.USER || 'OK');

test/fixtures/some"file

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+console.log(process.env.USER || 'OK');