-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
pm2 install on Windows (fix suggestion) #2223
Comments
OK, looks like you already have this fixed in master as of yesterday but not yet in a release. Any idea when the next release will be? |
@serganus pm2-monit module does not exists Else you can try PM2 v2:
|
@Unitech Tried but doesn't work Check the output for
|
NPM version? Node.js version? PM2 version? Have you tried with the PM2 on the development branch? |
@Unitech Yes i tried the development branch. |
2.15.8 |
Could you upgrade your npm?
|
I got this error resolved by launching the npm install into a shell in Modularizer.js#L111 : install_instance = spawn('npm', [
'install', module_name, '--prefix', cst.PM2_ROOT_PATH, '--loglevel=error'
],{
stdio : 'inherit',
async : true,
shell: true
}); Maybe a windows specific behavior ? |
@vmarchaud What do you mean by launching the npm install into a shell in Modularizer.js#L111? |
Windows fix from @vmarchaud has been merge to development branch,
Waiting for feedbacks |
@serganus Can you give us your npm/node/pm2/windows exact version ? |
I did the following but still same error.
|
@serganus Working on node v6.X.X. If you want you can update to use it correctly while i'm debuging with node v4.X.X |
@serganus Merged on development branch, tested and working on my workstation, its good for you ? |
As you know, module installation has been broken on Windows. Here's a simple fix:
In file Modularizer.js,
The text was updated successfully, but these errors were encountered: