ERROR: build:before: None-Zero Exit(7); about npm-run-all HOT 10 CLOSED

Confirming that I'm seeing this on any script that uses npm-run-all, but running the scripts individually has no issues.

from npm-run-all.

I'm seeing the same problem here. It looks like this happened after the code refractor patch...

from npm-run-all.

Seeing this too. Confirmed that it is past 8004ce4 (“Refactoring”).

I’ve traced it down to stdout/err problems. When I added a fs.createWriteableStream() to the options passed to spawn() in spawn-posix.js (e.g. log stdout/err into files). Things work as expected.

Do this in spawn-posix.js if you want to reproduce:

function spawn(command, args, options) {
    var fs = require('fs')
    var ofd = fs.openSync('npm-run-all.stdout', 'w')
    var efd = fs.openSync('npm-run-all.stderr', 'w')
    var out = fs.createWriteStream('', {fd: ofd, flags:'w'});
    var err = fs.createWriteStream('', {fd: efd, flags:'w'});

    options.detached = true; // eslint-disable-line no-param-reassign
    options = {
      stdio: [options.stdio[0], out, err]
    }
    console.log(command, args, options);
    var child = _child_process2["default"].spawn(command, args, options);
    child.on("exit", removeFromPool);
    child.on("error", function(error) {
      console.log(error);
      removeFromPool()
    });
    child.kill = kill;

    // Add to the pool to kill on exit.
    children.push(child);

    return child;
}

Good luck with this one <3

from npm-run-all.

And again as a diff, if that’s easier, full commit here: janl@2889479 branch here: https://github.com/janl/npm-run-all/tree/issue/14

diff --git a/src/lib/spawn-posix.js b/src/lib/spawn-posix.js
index c04abc1..8f10254 100644
--- a/src/lib/spawn-posix.js
+++ b/src/lib/spawn-posix.js
@@ -48,8 +48,18 @@ function kill() {
  * @returns {ChildProcess} A ChildProcess instance of new process.
  * @private
  */
+
+var fs = require('fs')
 export default function spawn(command, args, options) {
+    var ofd = fs.openSync('npm-run-all.stdout', 'w')
+    var efd = fs.openSync('npm-run-all.stderr', 'w')
+    var out = fs.createWriteStream('', {fd: ofd});
+    var err = fs.createWriteStream('', {fd: efd});
+
     options.detached = true; // eslint-disable-line no-param-reassign
+    options = {
+      stdio: [options.stdio[0], out, err]
+    }

     const child = cp.spawn(command, args, options);
     child.on("exit", removeFromPool);

from npm-run-all.

Oh my God, I apologize for it.
For a first aid, I reverted to v1.2.13 and published as v1.3.1.

My bad, I had not been having tests for stdin/stdout, so these were not tested in Travis CI.

from npm-run-all.

@janl Thank you for investigating.

Hmm, I seem to need changing my strategy that I use process group id to kill child processes.

from npm-run-all.

Hmm, I'm trying to add tests for this bug, but it works in Travis CI...
This might be related with nodejs/node#3596

from npm-run-all.

@mysticatea Thanks for great support!

from npm-run-all.

👍

from npm-run-all.

nice! <3

from npm-run-all.