Fix yarn run on window

[Volune]

Summary

Fix for #3773

Yarn run no longer works on windows with executables not mentioned in package.json/scripts

Test plan

I think we don't have automated tests for this case, it's OS specific and the fix relies on the shell option of child_process.spawn, so we cannot use mock to reliably test this issue.
(Any idea welcomed)

I ran manual tests on Windows 10 and a linux VM.
We should get more manual tests from the community before merging.

[BYK]

shell option is added in Node 5 and we need to support Node 4 for some more. For this reason, we can't go with the current solution. That said the code you removed was trying to mimic the shell: true behavior as documented in Node docs.

I think the main issue is with getting a full path or something on Windows so we can look there.

[BYK]

Rejecting due to shell not being available on Node 4.

[Volune]

shell is in Node 4 documentation

I just tested with node 4.8.4 to confirm, it worked.

[BYK]

Oh, wow. Apologies. I looked at the latest docs and it says "v5.7.0 | The shell option is supported now." in the history part. I was also curious why our Node 4 builds didn't fail :D

This means we may use this in one more place.

[Vanuan]

Looks like it was introduced in 4.8, since it fails in 4.7
Couldn't find any information about node requirement. Looks like it's the time to upgrade to 6.x