unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: iota@whxvd.name
Cc: 64493-done@debbugs.gnu.org
Subject: bug#64493: 29.0.92; Setting ‘dired-no-confirm’ to t does not have the expected effect
Date: Sat, 15 Jul 2023 10:50:16 +0300	[thread overview]
Message-ID: <83y1jhd35z.fsf@gnu.org> (raw)
In-Reply-To: <835y6xibxl.fsf@gnu.org> (message from Eli Zaretskii on Thu, 06 Jul 2023 13:14:46 +0300)

> Cc: 64493@debbugs.gnu.org
> Date: Thu, 06 Jul 2023 13:14:46 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> > From: Sebastian Miele <iota@whxvd.name>
> > Date: Thu, 06 Jul 2023 11:25:23 +0200
> > 
> > Run src/emacs -Q.  Evaluate (setopt dired-no-confirm t).  Try to delete
> > one or more files using Dired (using d and x, or (m and) D).  Then Emacs
> > still asks for confirmation.  Expected: After pressing x or D, the files
> > get deleted without any further questions.
> > 
> > After also trying to rename (R) files, my impression is that setting
> > ‘dired-no-confirm’ to t currently just suppresses the pop-up buffer that
> > shows the list of marked/flagged files while the action is about to be
> > executed (i.e., while waiting for the confirmation when deleting, or
> > while finding the destination when renaming multiple files).
> > 
> > The fact that ‘dired-no-confirm’ does not actually suppress confirmation
> > questions (at least when deleting files, I did not try all other
> > possible actions) probably is a bug.
> > 
> > However, it also would be nice if ‘dired-no-confirm’ would not have the
> > double meaning of also suppressing the pop-up buffer listing the marked
> > files (e.g., when finding the destination for renaming).  Ideally there
> > would be a second variable controlling just that.
> 
> The name of the variable is unfortunate, because its only effect is
> indeed to prevent the pop-up window with the list of affected files to
> be shown.  This doesn't look like a bug, because this behavior existed
> since the time the variable was added (30 years ago).  It looks like
> intentional (albeit strange) behavior.
> 
> I have now updated the doc string on the emacs-29 branch to explain
> the actual effect of the variable.
> 
> If you want to avoid the confirmation prompts, you can write a custom
> dired-deletion-confirmer function that always returns t.
> 
> Patches implementing a new option that actually prevents the
> confirmation prompts will also be welcome.

No further comments, so I'm closing the bug now.  Feel free to reopen
if something else should be done, or if someone has patches for a new
option as described above.





      reply	other threads:[~2023-07-15  7:50 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-06  9:25 bug#64493: 29.0.92; Setting ‘dired-no-confirm’ to t does not have the expected effect Sebastian Miele
2023-07-06  9:51 ` Andreas Schwab
2023-07-06 10:14 ` Eli Zaretskii
2023-07-15  7:50   ` Eli Zaretskii [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83y1jhd35z.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=64493-done@debbugs.gnu.org \
    --cc=iota@whxvd.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).