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

Add support for FSEvent notifications #7

Open
jamesarosen opened this issue Feb 6, 2010 · 4 comments
Open

Add support for FSEvent notifications #7

jamesarosen opened this issue Feb 6, 2010 · 4 comments
Labels

Comments

@jamesarosen
Copy link

This will save on battery life and hard drive wear on OSX >= 10.5 systems. It should degrade cleanly on systems where FSEvents are unavailable.

@mynyml
Copy link
Owner

mynyml commented Feb 7, 2010

I would love for this feature to be added to watchr, but I don't own a mac so can't add it myself. I'd really appreciate if someone were to provide a patch.

Meanwhile, the Rev event library uses kqueue on bsd so OSX users are not left out. I don't know how kqueue compares to FSEvents performance-wise though.

@sandro
Copy link
Contributor

sandro commented Feb 7, 2010

My fork has a really dirty patch which adds ruby-fsevent to the library. Now that I see the author has some interest in supporting this feature I may try giving it another go. Overall, it'd be great if we could enable fsevents without the rubycocoa dependency.

@mynyml
Copy link
Owner

mynyml commented Feb 19, 2010

sandro, I pushed your changes to the fsevents branch to confirm the interest, but I can't do much more since I'm not able to run the handler. I'll keep merging changes in the branch, and will merge into master when it's ready (functional and cleaned up).

@mynyml
Copy link
Owner

mynyml commented Jul 5, 2010

I finally have a mac to play with. I've updated/cleaned up the fsevents branch. It still has a few quirks (reported event aren't always detected), but is already fairly usable. If someone has a bit of time, I would greatly appreciate feedback before I merge into master. Thanks.

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

No branches or pull requests

3 participants