GNU bug report logs - #75626
31.0.50; Dired misses or double-processes files when auto-revert-mode is enabled

Previous Next

Package: emacs;

Reported by: Tassilo Horn <tsdh <at> gnu.org>

Date: Fri, 17 Jan 2025 07:43:01 UTC

Severity: normal

Found in version 31.0.50

Done: Tassilo Horn <tsdh <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Tassilo Horn <tsdh <at> gnu.org>
Cc: Michael Heerdegen <michael_heerdegen <at> web.de>, Eli Zaretskii <eliz <at> gnu.org>, 75626-done <at> debbugs.gnu.org
Subject: bug#75626: 31.0.50; Dired misses or double-processes files when auto-revert-mode is enabled
Date: Sun, 02 Feb 2025 11:10:44 +0100
Tassilo Horn <tsdh <at> gnu.org> writes:

Hi Tassilo,

> Yes, I have the basically the same questions.  IIRC, the problem you,
> Michael A., wanted to address in your proposal was that if a buffer has
> been skipped during auto-revert and the trigger for auto-revert was a
> filesystem notification, the auto-revert should be retried later.
>
> But isn't that the case anyway because auto-revert-buffers is run from a
> timer every auto-revert-interval seconds, so unless one sets
> auto-revert-interval to some very high value, no harm is done?
>
> Even if we would like to cater for this case, I don't understand how
> your approach addresses the retries for buffers that skipped/inhibited
> auto-revert.  I mean, I can envision that auto-revert maintains a list
> of buffers it skipped but how would that be worked through?  Another
> timer that runs more often that auto-revert-interval?

I haven't investigated this yet. Perhaps you are right, and a later
polling does the job. But this problem is recent anyway (if it is a
problem); my proposal doesn't make it worse.

> One fact in favour for some inhibit-auto-revert variable is that
> currently we abuse buffer-stale-function whose docs state "Function to
> check wether a buffer _needs_ reverting" with functions that actually
> check if we need and _allow_ reverting.  If we had inhibit-auto-revert,
> the allowance part could and should be removed from
> buffer-stale-function.

See my patch, it does exactly this.

> Bye,
> Tassilo

Best regards, Michael.




This bug report was last modified 196 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.