[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: filenotify.el
From: |
Michael Albinus |
Subject: |
Re: filenotify.el |
Date: |
Wed, 17 Jul 2013 16:54:34 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
Rüdiger Sonderfeld <address@hidden> writes:
> Hello,
Hi Rüdiger,
> I've written a new "Auto Update" module for magit using the new API:
> https://github.com/magit/magit/pull/721
Nice :-)
One remark: the following code is not sufficient:
(defun magit-filenotify-start ()
"Start watching for changes to the source tree using filenotify.
This can only be called from a magit status buffer."
(unless file-notify-support
(error "Support for `file-notify' required."))
...
There could be support even if `file-notify-support' is nil. This
variable reflects only, whether a respective library has been linked to
Emacs, working for local files. `file-notify' could run for remote
files, even if it is not supported locally.
Maybe we shall spend a new function
(defun file-notify-supported-p (file)
"Returns non-nil if filesystem pertaining to FILE could be watched."
...
Comments?
> Best regards,
> Rüdiger
Best regards, Michael.
- Re: filenotify.el, Rüdiger Sonderfeld, 2013/07/17
- Re: filenotify.el,
Michael Albinus <=
- Re: filenotify.el, Rüdiger Sonderfeld, 2013/07/17
- Re: filenotify.el, Davis Herring, 2013/07/22
- Re: filenotify.el, Michael Albinus, 2013/07/22
- Re: filenotify.el, Stefan Monnier, 2013/07/22
- Re: filenotify.el, Michael Albinus, 2013/07/23
- Re: filenotify.el, Stefan Monnier, 2013/07/23
- Re: filenotify.el, Michael Albinus, 2013/07/23
- Re: filenotify.el, Stefan Monnier, 2013/07/23
- Re: filenotify.el, Michael Albinus, 2013/07/23