-
Notifications
You must be signed in to change notification settings - Fork 29.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
EMFILE error appears in iojs v1.2.0 on Windows #3984
Comments
would you describe in detail? |
Sorry, I misclicked enter. I'm editing the OP right now. |
v1.1.0...v1.2.0, maybe e653080. @miklen can you try to |
@silverwind I've never tried it before, but I'll see if I can figure it out. I'll get back when I know which commit it is... or to declare failure :) |
@miklen are you seeing this error on a more recent version of Node (4.2.x and up)? |
@cjihrig yes. I've tried it with both 4.2.2 LTS and 5.1.0 stable, and both versions produce the error. The most recent version I've found to not produce the error is iojs 1.1.0. I tried all minor versions down from the last iojs version and down and all above 1.1.0 produces the error. |
It looks like I'm not able to build node from the source code. When I try I get this:
When it was done it did not work however. When I try to verify the build I get this:
Any ideas how to get it to build so I can test the commits? |
According to http://stackoverflow.com/questions/17300638/building-node-js-v0-10-12-on-windows, reinstalling Python in a path without spaces might help, e.g. |
Thanks - I got it working. And to my surprise, it looks like the issue has been resolved in the current HEAD commit on the master branch. I installed 5.1.0 again to recheck that I get the error in that version, and I do. I will try and investigate which commit since 5.1.0 and the HEAD on master fixed the issue. |
It might be interesting to inspect open file handles when you hit |
I tried bisecting the commits between HEAD and 5.1.0 and found that they all works. I then tried checking out the v5.1.0 commit (which fails for me when I install it from nodejs.org), and that commit works as well. So apparently I am unable to reproduce the error when I compile the code myself. The error is not isolated to my machine though. Everyone in my team has the issue when using the installer versions from the website. I'll see if I can inspect the file handles when I experience the error. |
@miklen Any luck? Could this be due to MSVS build differences? DO binaries you build work on the other machines at your work? |
@Fishrock123 unfortunately not yet. I haven't had time to do more testing yet. We're all running Visual Studio 2015 on Windows 10 and Window 7. I can try my binaries on one of the other machines. Unfortunately I don't have a machine available without VS2015. |
Closing due to inactivity. |
In our Windows build environment we are opening a lot of files through the use of browserify and debowerify. Our build stopped working when we upgraded to the latest NodeJS, with an EMFILE error.
I reverted back to NodeJS 0.12.7 and the build works again. I started testing the different versions and found that the error started happning on iojs v.1.2.0 (iojs 1.1.0 runs our build with no errors).
The error I'm getting is this:
In the versions of iojs and nodejs which are throwing the errors, I found that limiting the number of
require
calls in our app made it work again. Leaving out parts of the app is not really an option, so I hope someone can shed some light on this for me here.The text was updated successfully, but these errors were encountered: