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

Changing root still watches outside new root #46

Closed
mdwhatcott opened this issue Oct 16, 2013 · 4 comments
Closed

Changing root still watches outside new root #46

mdwhatcott opened this issue Oct 16, 2013 · 4 comments

Comments

@mdwhatcott
Copy link
Collaborator

When root is adjusted, the server still holds on to watches completely outside the new root.

I've opened an issue with fsnotify.

We may end up rolling our own "fsnotify" that works like somewhat like the python idle.py script (not sure if it would perform across lots of packages though...).

@mdwhatcott
Copy link
Collaborator Author

Looks like fsnotify has been patched by the author:

howeyc/fsnotify#71 (reference)

So, now it's a matter of updating fsnotify to see if that fixed things.

@mholt
Copy link
Contributor

mholt commented Oct 19, 2013

Excellent.

Are you still planning on using your own file system watcher/checksummer thing? That mechanism worked really well and was much simpler for our needs.

@mdwhatcott
Copy link
Collaborator Author

How we detect file changes/additions/deletions is actually just an implementation detail and really doesn't matter in the grand scheme of things. But, for the rewrite I think I'll go with the home-grown solution. As for the current version, go get -u github.com/howeyc/fsnotify will fix this issue.

@mdwhatcott
Copy link
Collaborator Author

The fix from fsnotify is good. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants