Merge pull request #62 from taimir/stdio_refactoring

Refactored the stdio redirection when gulp spawns tasks

build/deploy.js

@@ -27,7 +27,7 @@ import conf from './conf';
  * @param {function(?Error=)} doneFn
  */
 function spawnDockerProcess(args, doneFn) {
-  let dockerTask = child.spawn('docker', args);
+  let dockerTask = child.spawn('docker', args, {stdio: 'inherit'});
 
   // Call Gulp callback on task exit. This has to be done to make Gulp dependency management
   // work.
@@ -38,14 +38,6 @@ function spawnDockerProcess(args, doneFn) {
       doneFn(new Error('Docker command error, code:' + code));
     }
   });
-
-  dockerTask.stdout.on('data', function (data) {
-    console.log('' + data);
-  });
-
-  dockerTask.stderr.on('data', function (data) {
-    console.error('' + data);
-  });
 }
 
 

build/gocommand.js

@@ -34,7 +34,7 @@ export default function spawnGoProcess(args, doneFn) {
 
   let goTask = child.spawn('godep', ['go'].concat(args), {
     env: env,
-  });
+  }, {stdio: 'inherit'});
 
   // Call Gulp callback on task exit. This has to be done to make Gulp dependency management
   // work.
@@ -45,12 +45,4 @@ export default function spawnGoProcess(args, doneFn) {
       doneFn(new Error('Go command error, code:' + code));
     }
   });
-
-  goTask.stdout.on('data', function (data) {
-    console.log('' + data);
-  });
-
-  goTask.stderr.on('data', function (data) {
-    console.error('' + data);
-  });
 }

build/serve.js

@@ -120,20 +120,12 @@ gulp.task('serve:prod', ['build-frontend'], function () {
 gulp.task('spawn-backend', ['backend', 'kill-backend'], function () {
   runningBackendProcess = child.spawn(
       path.join(conf.paths.serve, conf.backend.binaryName),
-      [`--port=${conf.backend.devServerPort}`]);
+      [`--port=${conf.backend.devServerPort}`], {stdio: 'inherit'});
 
   runningBackendProcess.on('exit', function() {
     // Mark that there is no backend process running anymore.
     runningBackendProcess = null;
   });
-
-  runningBackendProcess.stdout.on('data', function (data) {
-    console.log('' + data);
-  });
-
-  runningBackendProcess.stderr.on('data', function (data) {
-    console.error('' + data);
-  });
 });