-
-
Notifications
You must be signed in to change notification settings - Fork 573
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
Chokidar exits when the watched folder is removed #324
Comments
If it's behaving this way it is a bug, not by design. |
Unfortunately, yes it does. I'm on Windows too, and I battled with some meteor/sanjo:jasmine/karma issues, and I just traced one thing down to chokidar. Karma uses chokidar to watch its files. When Meteor's builder renames/removes a directory with those files, and then creates/renames a new directory on the same place, Karma/Chokidar stops watching this directory for file changes. I just noticed that it's the Chokidar that simply stops issuing events in this case. |
We are having this issue as well |
This problem persists in release 1.5.2 on Windows 7 Service Pack 1.
EDIT 2016-06-09 16:53 (CEST) Updated npm ( |
I'm also wondering whether |
@lkraav from the readme:
|
@es128 tyvm, I also happened to find this info at that time, but forgot to update the comment. |
when i ran script in debug mode in webstrom, it could catch error event. |
let me try out if meterjs/karma/velocity still break on windows due to exactly this issue.. ..but then, I don't even have them on this new machine, so that may take a while.. |
Given our discussion in #319 , if Chokidar is happy about placing a watch on a non-existent directory at startup, symmetry should demand that it should continue watching even if the directory is removed after issuing an
unlink
event (of course, assumingpersistent: true
). Right now, the watcher stops when the path is deleted at least on Windows.I think the circumstances are identical in that a directory does not exist in both cases, why should they be treated differently?
The text was updated successfully, but these errors were encountered: