Doesn't work well with Yarn

[mattzuba]

I see some previous commits and changes to make it work with Yarn, but it doesn't work for me on Win10CU. I modified some code in run-task.js and now it runs like a champ. I'm sure this entire if/else can be condensed a little bit.

        if (path.extname(options.npmPath || "a.js") === ".js") {
            const npmPath = options.npmPath || process.env.npm_execpath  //eslint-disable-line no-process-env
            const execPath = npmPath ? npmPath : "npm"
            const spawnArgs = [].concat(
                ["run"],
                options.prefixOptions,
                parseArgs(task)
            )

            // Execute.
            cp = spawn(execPath, spawnArgs, spawnOptions)
        }

[ohtake]

I have the same issue with nodejs 6.10.3, yarn 0.24.5, and npm-run-all 4.0.2 on Windows 10 1607 64bit.
Using npm 3.10.10 instead of yarn works for me.

[mysticatea]

Thank you for this issue.

However, I'm using Windows 7 Pro and I could not reproduce it.
Could you tell me the repro steps?

---

package.json:

{
  "name": "sandbox",
  "version": "1.0.0",
  "description": "",
  "scripts": {
    "test": "run-s a",
    "a": "node -p \"process.env.npm_execpath\""
  },
  "author": "Toru Nagashima (https://github.com/mysticatea)",
  "license": "MIT"
}

Command:

~\Documents\GitHub\sandbox> yarn run test
yarn run v0.24.4
$ run-s a
yarn run v0.24.4
$ node -p "process.env.npm_execpath"
C:\Users\t-nagashima.AD\nodist\bin\node_modules\yarn\bin\yarn.js
Done in 0.38s.
Done in 1.23s.
~\Documents\GitHub\sandbox>

[ohtake]

Using package.json you provided.

$ yarn add --dev npm-run-all
yarn add v0.24.5
info No lockfile found.
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
success Saved 64 new dependencies.
├─ ansi-regex@2.1.1
├─ ansi-styles@2.2.1
├─ array-filter@0.0.1
├─ array-map@0.0.0
├─ array-reduce@0.0.0
├─ balanced-match@0.4.2
├─ brace-expansion@1.1.7
├─ builtin-modules@1.1.1
├─ chalk@1.1.3
├─ concat-map@0.0.1
├─ cross-spawn@5.1.0
├─ define-properties@1.1.2
├─ duplexer@0.1.1
├─ error-ex@1.3.1
├─ es-abstract@1.7.0
├─ es-to-primitive@1.1.1
├─ escape-string-regexp@1.0.5
├─ event-stream@3.3.4
├─ foreach@2.0.5
├─ from@0.1.7
├─ function-bind@1.1.0
├─ graceful-fs@4.1.11
├─ has-ansi@2.0.0
├─ has@1.0.1
├─ hosted-git-info@2.4.2
├─ is-arrayish@0.2.1
├─ is-builtin-module@1.0.0
├─ is-callable@1.1.3
├─ is-date-object@1.0.1
├─ is-regex@1.0.4
├─ is-symbol@1.0.1
├─ isexe@2.0.0
├─ jsonify@0.0.0
├─ load-json-file@2.0.0
├─ lru-cache@4.0.2
├─ map-stream@0.1.0
├─ minimatch@3.0.4
├─ normalize-package-data@2.3.8
├─ npm-run-all@4.0.2
├─ object-keys@1.0.11
├─ parse-json@2.2.0
├─ path-type@2.0.0
├─ pause-stream@0.0.11
├─ pify@2.3.0
├─ ps-tree@1.1.0
├─ pseudomap@1.0.2
├─ read-pkg@2.0.0
├─ semver@5.3.0
├─ shebang-command@1.2.0
├─ shebang-regex@1.0.0
├─ shell-quote@1.6.1
├─ spdx-correct@1.0.2
├─ spdx-expression-parse@1.0.4
├─ spdx-license-ids@1.2.2
├─ split@0.3.3
├─ stream-combiner@0.0.4
├─ string.prototype.padend@3.0.0
├─ strip-ansi@3.0.1
├─ strip-bom@3.0.0
├─ supports-color@2.0.0
├─ through@2.3.8
├─ validate-npm-package-license@3.0.1
├─ which@1.2.14
└─ yallist@2.1.2
Done in 8.86s.

$ yarn run test
yarn run v0.24.5
$ run-s a
yarn run v0.24.5
$ node -p "process.env.npm_execpath"
C:\Program Files (x86)\Yarn\bin\yarn.js
Done in 0.23s.
Done in 1.00s.

$ yarn run test
yarn run v0.24.5
$ run-s a
^C

run-s a won't run in the second time. I hit Ctrl+C.
I noticed that yarn run test runs successfully in odd time,
and it fails in even time.

• node 6.10.3 x64
• yarn 0.24.5
• Windows 10 x64 1607 14393.1198

I also tested it on another machine, which installed older yarn.
The issue did not happen.

• node 6.10.3 x64
• yarn 0.23.4
• Windows 10 x64 1607 14393.1198

After upgrading yarn to 0.24.5, the issue was reproduced.

[mysticatea]

Thank you. I'll check it.

.... yarn@0.24.5 does not seem to be published to npm.

> npm show yarn dist-tags
{ latest: '0.24.4', rc: '0.25.2' }

[ohtake]

You can get yarn v0.24.5 at GitHub release.
It is not recommended to install yarn via npm.
I also reproduced the issue with v0.24.3 and v0.24.4.
I have not tested with pre-released v0.24.[0-2].

[mysticatea]

Hmm, I could not reproduce it with yarn@0.24.5 on PowerShell and cmd.exe on Windows 7.
I'll check on Windows 10 later.

[ohtake]

I also cannot reproduced the issue with PowerShell and Command Prompt on Windows 10.
But I can reproduced it with bash bundled with Git for Windows.
I tested 2 versions of bash and both of them have the issue.

git version 2.11.1.windows.1
GNU bash, version 4.3.46(2)-release (x86_64-pc-msys)

git version 2.13.0.windows.1
GNU bash, version 4.4.12(1)-release (x86_64-pc-msys)

The terminal for bash is mintty, the default terminal of Git for Windows.
If bash is executed from cmd.exe, the issue does not happen.

[mysticatea]

Thank you for the information.

Yeah, Git Bash of Windows is super problematic about stdin/stdout.

I'm guessing related to nodejs/node#5620.
Please try defining the following alias. (This is similar to nodejs/node#5620 (comment))

$ alias -p yarn="winpty yarn.cmd"

[ohtake]

Thank you for workarounds.
Defining the alias or hitting enter key solves the issue.

[mysticatea]

Thank you.

Actually this is a bug of Node.js or MinGW.
The process.stdin getter of Node.js hungs up on specific cases if you are using MinGW.
winpty is the workaround of MinGW side.

I'm not sure whether I can do something in npm-run-all.