[Top][All Lists]

[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."


> Best regards,
> Rüdiger

Best regards, Michael.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]