[api refactor] Remove fork hack since we are now using node-fork

lib/forever/monitor.js

@@ -223,9 +223,12 @@ Monitor.prototype.trySpawn = function () {
   this.spawnWith.cwd = this.cwd || this.spawnWith.cwd;
   this.spawnWith.env = this._getEnv();
 
-  return this.fork
-    ? this._forkSpawn()
-    : spawn(this.command, this.args, this.spawnWith);
+  if (this.fork) {
+    this.spawnWith.command = this.command;
+    return fork(this.args[0], this.args.slice(1), this.spawnWith);
+  }
+
+  return spawn(this.command, this.args, this.spawnWith);
 };
 
 //
@@ -393,48 +396,4 @@ Monitor.prototype._getEnv = function () {
   });
 
   return merged;
-};
-
-//
-// ### @private function _forkSpawn()
-//
-// This is an unfortunate hack which works around two inconsistencies
-// in the `child_process` APIs in node.js core as of `v0.6.6`. Pull-requests
-// are open to resolve both issues.
-//
-// https://github.com/joyent/node/pull/2455
-// https://github.com/joyent/node/pull/2454
-//
-// Until then, setup `options.customFds` to always return null.
-//
-//
-Monitor.prototype._forkSpawn = function () {
-  var execPath = process.execPath,
-      hackSpawn = {},
-      self = this,
-      forked;
-
-  if (this.spawnWith && typeof this.spawnWith === 'object') {
-    Object.keys(this.spawnWith).forEach(function (key) {
-      hackSpawn[key] = self.spawnWith[key];
-    });
-  }
-
-  delete hackSpawn.customFds;
-  Object.defineProperty(hackSpawn, 'customFds', {
-    get: function () {
-      return null;
-    },
-    set: function () {
-      //
-      // Do nothing, ignore the attempt to overwrite here:
-      // https://github.com/joyent/node/blob/master/lib/child_process.js#L172
-      //
-    } 
-  });
-
-  process.execPath = this.command;
-  forked = fork(this.args[0], this.args.slice(1), hackSpawn);
-  process.execPath = execPath;
-  return forked;
-};
+};