Magit inotify #586

Merged
merged 5 commits into from Mar 23, 2013

Conversation

Projects
None yet
3 participants
Contributor

ruediger commented Mar 10, 2013

This module uses the new inotify feature in Emacs 24.4 to watch the source tree for changes and auto updates the status buffer.

ruediger added some commits Mar 10, 2013

@ruediger ruediger Initial commit of magit-inotify.
Auto refresh mode using inotify to watch for changes in the source
tree.

Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
9851a35
@ruediger ruediger magit-inotify: Added description.
Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
e84c942
Member

sigma commented Mar 21, 2013

I think this proposal is mostly good. Only a couple of things missing: some autoload cookies, and code to handle lack of inotify.el gracefully for older versions of Emacs (at compilation and loading time)

Contributor

ruediger commented Mar 21, 2013

code to handle lack of inotify.el gracefully for older versions of Emacs (at compilation and loading time)

Any suggestion on how to do this? Something like (unless (featurep 'inotify) (error "inotify not supported")) is probably not the solution. Wrapping everything in a (when (featurep 'inotify) ...) isn't either.

ruediger added some commits Mar 21, 2013

@ruediger ruediger inotify: Add autoload cookie.
Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
67b8abf
@ruediger ruediger inotify: Use `derived-mode-p' instead of `eq'-check.
Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
ed33a47
@ruediger ruediger Move magit-inotify to contrib/.
Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
63d2222

@sigma sigma merged commit 63d2222 into magit:master Mar 23, 2013

1 check passed

default The Travis build passed
Details

ruediger deleted the ruediger:magit-inotify branch Mar 23, 2013

tarsius added this to the 2.1.0 milestone Feb 17, 2014

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