-
Notifications
You must be signed in to change notification settings - Fork 150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Execution of local bin scripts fails on Windows #652
Comments
@bzoz any idea? I'm pretty sure |
Is it only |
I think it's actually more than that. If there is an npm lifecycle script (install, postinstall, prepare) that calls a local dep, it fails. |
I'm going afk now but I will try to create a small module with such a case later to use in our tests |
This really seems to be a general issue, not only during install. |
See https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1688/nodes=win-vs2017/#showFailuresLink for all modules failing because of this |
I made a simple bin script module here: https://github.com/targos/citgm-bin-script |
I'm lost... This passes: Both use |
|
I didn't think this could get even more weird 😆 |
And now (without changing anything in citgm) both |
I don't know what to think of it but you're right, everything works now... |
I can reproduce with these:
|
extensive windows failures in this run --> https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1718/nodes=win-vs2017/testReport/ |
/cc @refack maybe it rings a bell? |
This happens pretty frequently and the citgm output becomes pretty hard to read that way. @refack PTAL |
Will do, first thing tomorrow. |
This is still unresolved. I do not have a Windows machine, so I can't look into this. @refack do you still plan on pursuing this? |
looks like it's not reproducible anymore 🤷♂ |
To reproduce:
It cannot find
node-gyp-build
. Probably another PATH issue...The text was updated successfully, but these errors were encountered: