all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#30407: dired-diff ignores backup-directory-alist
@ 2018-02-09 22:53 積丹尼 Dan Jacobson
  2018-02-10  9:52 ` Eli Zaretskii
  0 siblings, 1 reply; 2+ messages in thread
From: 積丹尼 Dan Jacobson @ 2018-02-09 22:53 UTC (permalink / raw)
  To: 30407

dired-diff prompts with
Diff p with (default            !home!jidanni!org!debconf!p.~8~): ~/org/debconf/ #WRONG
Diff p with (default ~/.backups/!home!jidanni!org!debconf!p.~8~): ~/org/debconf/ #Correct
when in .emacs the user does
(setq
 auto-save-list-file-prefix "~/.backups/.saves-";less clutter in $HOME
 backup-directory-alist '((".*" . "~/.backups/"))
 backup-by-copying-when-linked t ;don't destroy other hard links
 backup-by-copying-when-mismatch t)

It correctly detects that we are already at version ~8~ in that faraway
directory, but then it forgets to add the prefix to the default.

It then prompts with a non-existent wrong file doomed to fail if the
user hits RET.

Also using M-n, M-p do not make the default that it shows available to
the user to edit.

In fact the total correct prompt should probably look like one of
Diff p with (default ~/.backups/!home!jidanni!org!debconf!p.~8~): ~/.backups/
Diff p with (default ~/.backups/!home!jidanni!org!debconf!p.~8~): ~/.backups/!home!jidanni!org!debconf!p.~8~
Diff p with: ~/.backups/!home!jidanni!org!debconf!p.~8~
The idea being that the user can easily change
Diff p with: ~/.backups/!home!jidanni!org!debconf!p.~8~ to
Diff p with: ~/.backups/!home!jidanni!org!debconf!p.~7~ etc.

emacs-version "25.2.2" and I do recall I reported something similar
years ago...





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#30407: dired-diff ignores backup-directory-alist
  2018-02-09 22:53 bug#30407: dired-diff ignores backup-directory-alist 積丹尼 Dan Jacobson
@ 2018-02-10  9:52 ` Eli Zaretskii
  0 siblings, 0 replies; 2+ messages in thread
From: Eli Zaretskii @ 2018-02-10  9:52 UTC (permalink / raw)
  To: 積丹尼 Dan Jacobson; +Cc: 30407

> From: 積丹尼 Dan Jacobson
> 	<jidanni@jidanni.org>
> Date: Sat, 10 Feb 2018 06:53:47 +0800
> 
> dired-diff prompts with
> Diff p with (default            !home!jidanni!org!debconf!p.~8~): ~/org/debconf/ #WRONG
> Diff p with (default ~/.backups/!home!jidanni!org!debconf!p.~8~): ~/org/debconf/ #Correct
> when in .emacs the user does
> (setq
>  auto-save-list-file-prefix "~/.backups/.saves-";less clutter in $HOME
>  backup-directory-alist '((".*" . "~/.backups/"))
>  backup-by-copying-when-linked t ;don't destroy other hard links
>  backup-by-copying-when-mismatch t)
> 
> It correctly detects that we are already at version ~8~ in that faraway
> directory, but then it forgets to add the prefix to the default.
> 
> It then prompts with a non-existent wrong file doomed to fail if the
> user hits RET.

I believe this bug is already fixed in Emacs 26, please try the latest
pretest.

> Also using M-n, M-p do not make the default that it shows available to
> the user to edit.

Because M-n and M-p show possible directories where that other file
could live, not only possible backup file names.  Dired-diff is not
necessarily for comparison with the backup file, it's more general.

> emacs-version "25.2.2" and I do recall I reported something similar
> years ago...

If you are willing to make such remarks, please always make a point of
trying the latest codebase first.





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-02-10  9:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-09 22:53 bug#30407: dired-diff ignores backup-directory-alist 積丹尼 Dan Jacobson
2018-02-10  9:52 ` Eli Zaretskii

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.