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