[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Emacs-diffs] trunk r115439: * autorevert.el (auto-revert-notify-add

From: Michael Albinus
Subject: Re: [Emacs-diffs] trunk r115439: * autorevert.el (auto-revert-notify-add-watch): Do not handle symlinked files.
Date: Wed, 11 Dec 2013 16:17:46 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)

Eli Zaretskii <address@hidden> writes:

>> From: Michael Albinus <address@hidden>
>> Cc: Stefan Monnier <address@hidden>,  address@hidden
>> Date: Tue, 10 Dec 2013 05:58:18 +0100
>> Eli Zaretskii <address@hidden> writes:
>> >> I think this patch is acceptable, but if/when we try to write a generic
>> >> file-watcher Elisp API, it would be good to make this work (probably by
>> >> adding a watcher on the symlink's target).
>> >
>> > It's up to the user to say whether she wants to watch the symlink or
>> > its target, I think.
>> In general I agree. But Stefan was commenting on my change in
>> autorevert.el. Here it is obvious, that the target shall be watched.
>> I believe.
> What if the target is a directory?

This is not relevant for autorevert.el and file notifications - if the
target is a directory, file notifications are not used (yet) in
autorevert.el. I've created Bug#16112 for this.

Best regards, Michael.

reply via email to

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