Subject: Re: bug#75626: 31.0.50; Dired misses or double-processes files when auto-revert-mode is enabled

From: Eli Zaretskii <[email-masked]>
Date: Mon, 20 Jan 2025 15:07:17 +0200

> From: Tassilo Horn <[email-masked]>
> Cc: Eli Zaretskii <[email-masked]>,   [email-masked]
> Date: Mon, 20 Jan 2025 07:06:20 +0100
> 
> diff --git a/lisp/dired.el b/lisp/dired.el
> index 2087a6f6f21..c78a13800e6 100644
> --- a/lisp/dired.el
> +++ b/lisp/dired.el
> @@ -1289,6 +1289,12 @@ dired-buffer-stale-p
>          ;; Do not auto-revert when the dired buffer can be currently
>          ;; written by the user as in `wdired-mode'.
>          buffer-read-only
> +         ;; When a dired operation using dired-map-over-marks is in
> +         ;; progress, inhibit-read-only is bound to some non-nil value
> +         ;; and we must not auto-revert because that could change the
> +         ;; order of files leading to skipping or double-processing (see
> +         ;; bug#75626).
> +         (not inhibit-read-only)
>          (dired-directory-changed-p dirname))))
>  
>  (defcustom dired-auto-revert-buffer nil
> --8<---------------cut here---------------end--------------->8---
> 
> Eli, good to go?

I answered that in my other message a few minutes ago.  I think we
should find a cleaner solution.

> And if so, master or emacs-30?

master, of course.