Make it work with npx

[zloirock]

Original issue: mysticatea/npm-run-all#209

[bcomnes]

Probably should fix this. Anyone attempt patches upstream I can pull in?

[zloirock]

mysticatea/npm-run-all#245

[bcomnes]

Cool ty. I'll look at getting that merged and tested.

[bcomnes]

@MarmadileManteater That would be great and I appreciate it!

[bcomnes]

https://github.com/bcomnes/npm-run-all2/releases/tag/v6.0.3

Reopen if there are issue

[zloirock]

It seems it's still not fixed

@MarmadileManteater

[bcomnes]

@zloirock you are right this didn't quite work out.

@MarmadileManteater Can you provide more information on the following env vars you referenced in your patch?

• process.env.NPM_CLI_JS
• rocess.env.npm_config_user_agent

Are these standard or set by a tool somewhere or did you expect people to have to set these?

[bcomnes]

@MarmadileManteater #96

[bcomnes]

This could be addressed by looking for npm_cli.js in the same directory that npx_cli.js is in since npm_execpath seems to exist on all systems.

Great minds think alike!

[zloirock]

Yep, at debugging I see:

options.npmPath: null
process.env.NPM_CLI_JS: undefined
process.env.npm_execpath: /usr/local/lib/node_modules/npm/bin/npx-cli.js
npmPath: /usr/local/lib/node_modules/npm/bin/npx-cli.js
npmPathIsJs: true
execPath: /usr/local/bin/node

so #96 looks like a proper fix.

[bcomnes]

Ok everyone give https://github.com/bcomnes/npm-run-all2/releases/tag/v6.0.4 a whirl.

[zloirock]

It works for me, thanks 👍