Skip to content
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

io.js 2.2.x 2.3.x nodemon on Windows #1849

Closed
kyrylkov opened this issue May 31, 2015 · 22 comments
Closed

io.js 2.2.x 2.3.x nodemon on Windows #1849

kyrylkov opened this issue May 31, 2015 · 22 comments
Labels
windows Issues and PRs related to the Windows platform.

Comments

@kyrylkov
Copy link

C:\Users\Dev>nodemon
exception in nodemon killing node
Error: Cannot find module 'wrappy'
    at Function.Module._resolveFilename (module.js:332:15)
    at Function.Module._load (module.js:282:25)
    at Module.require (module.js:361:17)
    at require (module.js:380:17)
    at Object.<anonymous> (C:\Users\Sergiy Kyrylkov\AppData\Roaming\npm\node_modules\nodemon\node_modules\update-notifier\node_modules\latest-version\node_modules\package-json\node_modules\got\node_modules\duplexify\node_modules\end-of-stream\node_modules\once\once.js:1:76)
    at Module._compile (module.js:426:26)
    at Object.Module._extensions..js (module.js:444:10)
    at Module.load (module.js:351:32)
    at Function.Module._load (module.js:306:12)
    at Module.require (module.js:361:17)
    at require (module.js:380:17)
@kyrylkov kyrylkov changed the title io.js 2.2.0 breaks nodemon on Windows 10 io.js 2.2.0 x64 breaks nodemon on Windows 10 May 31, 2015
@mscdex mscdex added the windows Issues and PRs related to the Windows platform. label May 31, 2015
@targos
Copy link
Member

targos commented May 31, 2015

Is it possible that the wrappy module is not there because of too long filenames ?

@targos
Copy link
Member

targos commented May 31, 2015

I am trying to reproduce but I cannot use npm with v2.2.0 because of a strange SSL error

@kyrylkov
Copy link
Author

@targos it worked fine before the upgrade. the project was on io.js since the beginning.

@silverwind
Copy link
Contributor

Probably caused by #1850

@silverwind
Copy link
Contributor

@kyrylkov please retry with 2.2.1

@targos
Copy link
Member

targos commented Jun 1, 2015

It is not fixed in 2.2.1, I am getting the same error

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@silverwind 2.2.1 doesn't solve the problem

@rvagg
Copy link
Member

rvagg commented Jun 1, 2015

@kyrylkov can you npm install nodemon -g again just to be sure that something that needs a rebuild gets rebuilt?

@rvagg
Copy link
Member

rvagg commented Jun 1, 2015

@kyrylkov was this actually working on a previous version? You're running really close to the path limit on that one and I wouldn't be surprised if that's involved, already the error is being caused at once.js which has a 230 char path and if wrappy is supposed to be a dependency then you'd likely be pushing the limit

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg npm uninstall nodemon -g and npm install nodemon -g didn't help

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg it worked all the way from io.js 1.0 or so. do you suggest to move the project folder closer to the root to try?

@rvagg
Copy link
Member

rvagg commented Jun 1, 2015

yeah, I'm wondering if something in npm has changed or perhaps a new version of nodemon or one of its dependencies has changed to alter the dependency tree to make it too deep; perhaps go to c:\ then npm install nodemon and then call it from where you need it as c:\node_modules\.bin\nodemon and see if you get different results

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg If you referring to C:\Users\Dev\AppData\Roaming\npm\node_modules\nodemon\node_modules\update-notifier\node_modules\latest-version\node_modules\package-json\node_modules\got\node_modules\duplexify\node_modules\end-of-stream\node_modules\once\once.js, this is not something that a user can change, since on Windows this is a default installation path for global node modules.

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg local installation from a very shallow path (C:\a\node_modules\.bin\nodemon) works

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg however this doesn't solve a problem of a default io.js Windows installation, since the path for global node modules remains very long with no easy way to change it.

@rvagg
Copy link
Member

rvagg commented Jun 1, 2015

@kyrylkov this is going to have to be something to take up with the npm team over at https://github.com/npm/npm, we just bundle the tool here and it controls the installation of packages.

you should be able to npm config set prefix=c:\a\ to change the default though, I'm not sure how Windows will handle the executables though, you may need to add c:\a\bin or something to your PATH.

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

@rvagg got it. i'll open an issue at npm.

@kyrylkov
Copy link
Author

kyrylkov commented Jun 1, 2015

npm/npm#8419

@Fishrock123 Fishrock123 added the npm Issues and PRs related to the npm client dependency or the npm registry. label Jun 1, 2015
@neVERberleRfellerER
Copy link

Are you sure that this is problem with NPM? iojs 2.2.0 can't find modules in long paths that are already installed. When I replace exe with iojs 2.1.0 (keeping latest npm), even "npm install" works correctly. iojs 2.2.0 and later breaks this (modules in long paths are not found). Returning to 2.1.0 fixes everything.

@targos
Copy link
Member

targos commented Jun 16, 2015

@neVERberleRfellerER you are right, it is an issue introduced in 2.2.0.
See #1990 and #1991.

@kyrylkov
Copy link
Author

Yes, it looks like in the end io.js is to blame

@kyrylkov kyrylkov changed the title io.js 2.2.0 x64 breaks nodemon on Windows 10 io.js 2.2.x 2.3.x x64 breaks nodemon on Windows 10 Jun 16, 2015
@kyrylkov kyrylkov changed the title io.js 2.2.x 2.3.x x64 breaks nodemon on Windows 10 io.js 2.2.x 2.3.x nodemon on Windows Jun 16, 2015
@vkurchatkin vkurchatkin removed the npm Issues and PRs related to the npm client dependency or the npm registry. label Jun 16, 2015
targos added a commit to targos/node that referenced this issue Jun 16, 2015
nodejs#1801 introduced internal fs methods to speed up require.
The methods do not call path._makeLong like their counterpart
from the fs module. This brings back the old behaviour.

Fixes: nodejs#1990
Fixes: nodejs#1980
Fixes: nodejs#1849
piscisaureus pushed a commit that referenced this issue Jun 16, 2015
#1801 introduced internal fs methods
to speed up require. The methods do not call path._makeLong like their
counterpart from the fs module. This brings back the old behaviour.

Fixes: #1990
Fixes: #1980
Fixes: #1849

PR-URL: https://github.com/nodejs/io.js/pull/1991/files
Reviewed-By: Bert Belder <bertbelder@gmail.com>
@silverwind
Copy link
Contributor

Fixed by 671e64a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
windows Issues and PRs related to the Windows platform.
Projects
None yet
Development

No branches or pull requests

8 participants