-
Notifications
You must be signed in to change notification settings - Fork 822
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
NPM Webpack not watching #396
Comments
@BeneCollyridam inotify is currently not implemented (this is also brought up in #216). Does the code you're compiling require inotify support? If we were reporting max_user_watches = 0 would compilation continue? |
No, it does not require I notify. It was meant to automatically recompile when changed but I guess I'll just have to wait. |
@BeneCollyridam that seems very useful! Inotify is definitely something that is on our backlog. |
Related: it seems that brunch does not work at all without inotify. (Brunch is an alternative to webpack) edit: I was wrong, Brunch live reloading is working when |
Basically anything that uses chokidar doesn't work. A few things I've tried: |
Just FYI, I landed a patch on Chokidar that will fix things for the time being. Before you run your subdependency that uses chokidar, set the environment variable Requires [email protected], but better than nothing. |
@yaycmyk Although your method works, it's really not ideal. Using it will cause nodejs to use 100% cpu, sadly - at least in my case. Is there a workaround for that? |
@ivoviz Not until the proper subsystems are implemented, unfortunately. |
Take a hard look at what's being watched and try to make the patterns as narrow as possible - your watch is likely looking at more things than it needs to... |
Unfortunately I can't really control what's being watched, it's been done by angular-cli. This really is a huge blocker. |
I also see this as a huge blocker for BashOnWindows. I can't get my webpack config to work at all, even if I try to remove watch and inline options. I'm using webpack-dev-server. I get many errors like this one:
|
yes, BashOnWindows is exciting, but without this being fixed, we just can not start our dev work with it -- just need the watch feature ~ |
This issue has been solved as of build 14942 See #216 |
@mikeesouth This issue still is not solved as far as I can tell. On the newest fast-ring. uv_interface address issues are seperate from inotify. This is network enunaration. Looking for a fix to a very similar error. |
@ALL as of build #14946 WSL supports inotify. @BeneCollyridam please verify and then close this issue. @mikeesouth the uv_interface_address error has also been solved as of 14986 |
This still doesn't work for me. Surface Pro 4 with FCU. I keep getting the same error message as in the original question on the thread. Did something change in the FCU? |
did you install inotify-tools in ubuntu? |
I just did. Doesn't seem to make a difference. |
A few things that might help some of you:
|
Did anyone find a solution for this in windows? |
Using node with webpack and babel
New code doesn't compile automatically. Possibly due to not enough watchers. When trying to fix this with:
I get the error:
I am running Windows 10 Insider Preview Build 14342
The text was updated successfully, but these errors were encountered: