all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it
@ 2016-06-19  0:12 Drew Adams
  2019-07-28 11:48 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2016-06-19  0:12 UTC (permalink / raw)
  To: 23799

emacs -Q

Hit C-c C-c to send a bug report, then reply "no", perhaps because you
want to change something before submitting the report.

Now try C-c C-c again.

You get an error/message saying:

  message-send: No methods specified to send by

That's presumably because you never got to the part of the dialog where
it asks you how to send the message.

This is a bug.  Just because you used C-c C-c and chose not to send the
message yet is not a reason to no longer give you a means to specify the
"method to send by".

This happens regardless of whether you have changed the "To:" field -
 e.g., in response to this message in *Bug Help*:

  You invoked the command M-x report-emacs-bug,
  but you decided not to mail the bug report to the Emacs maintainers.

  If you want to mail it to someone else instead,
  please insert the proper e-mail address after "To: ",
  and send the mail again using C-c C-c.

And perhaps the message about not having specified a mail method should
be incorporated into *Bug Help*?

It seems that this part of dialoguing with the user (what happens if
s?he says "no") has not been thought through well enough and tested well
enough.


In GNU Emacs 25.0.90.4 (i686-w64-mingw32)
 of 2016-03-20
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --host=i686-w64-mingw32 --without-dbus
 --without-compress-install CFLAGS=-static'





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

* bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it
  2016-06-19  0:12 bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it Drew Adams
@ 2019-07-28 11:48 ` Lars Ingebrigtsen
  2020-08-12  2:20   ` Stefan Kangas
  0 siblings, 1 reply; 4+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-28 11:48 UTC (permalink / raw)
  To: Drew Adams; +Cc: 23799

Drew Adams <drew.adams@oracle.com> writes:

> Hit C-c C-c to send a bug report, then reply "no", perhaps because you
> want to change something before submitting the report.
>
> Now try C-c C-c again.
>
> You get an error/message saying:
>
>   message-send: No methods specified to send by

It looks like this puzzling behaviour is because of this puzzling code:

      (goto-char (point-min))
      (if (search-forward "To: ")
          (delete-region (point) (line-end-position)))
      (if report-emacs-bug-send-hook
          (kill-local-variable report-emacs-bug-send-hook))

Which was introduced by the patch below, apparently, which doesn't say
anything about why it's removing the To: header or why it's removing the
hook (you don't get any queries the second time around, either).

Hm...  Oh!

I didn't even notice that when you type "no", it switches out the help
buffer text:

      (with-output-to-temp-buffer "*Bug Help*"
	(princ (substitute-command-keys
                (format "\
You invoked the command M-x report-emacs-bug,
but you decided not to mail the bug report to the Emacs maintainers.

If you want to mail it to someone else instead,
please insert the proper e-mail address after \"To: \",
and send the mail again%s."

So the intention here is that if you say "no", then you're never ever
going to send the message to the Emacs maintainers.  I think that's a
pretty unexpected thing to infer from just the "no".

I've now removed that code that removes the To and the hook, which then
allows the user to change their minds.

commit fe1d8b33cbeadad8cd9c186553ecf87b8bbf314d
Author: Kenichi Handa <handa@m17n.org>
Date:   Tue Jul 15 08:23:58 1997 +0000

    (report-emacs-bug-run-tersely): New variable
    (report-emacs-bug): Insert warnings for novice usres in *mail* buffer.
    Set enable-multibyte-characters to nil.
    (report-emacs-bug-hook): Check non-English letters.  Confirm about
    sending a report to FSF.

diff --git a/lisp/mail/emacsbug.el b/lisp/mail/emacsbug.el
--- a/lisp/mail/emacsbug.el
+++ b/lisp/mail/emacsbug.el
@@ -150,1 +187,11 @@
-	(error "No text entered in bug report"))))
+    (if (or report-emacs-bug-run-tersely
+	    (yes-or-no-p
+	     "Do you surely send this mail to Free Software Foundation? "))
+	;; Just send the current mail.
+	nil
+      (goto-char (point-min))
+      (if (search-forward "To: ")
+	  (let ((pos (point)))
+	    (end-of-line)
+	    (delete-region pos (point))))
+      (kill-local-variable 'mail-send-hook)



-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

* bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it
  2019-07-28 11:48 ` Lars Ingebrigtsen
@ 2020-08-12  2:20   ` Stefan Kangas
  2020-08-12 10:54     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Kangas @ 2020-08-12  2:20 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 23799

Lars Ingebrigtsen <larsi@gnus.org> writes:

> I've now removed that code that removes the To and the hook, which then
> allows the user to change their minds.

Does commit 3806ddcb76 mean that this should also be closed, or is there
anything more to do here?

Best regards,
Stefan Kangas





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

* bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it
  2020-08-12  2:20   ` Stefan Kangas
@ 2020-08-12 10:54     ` Lars Ingebrigtsen
  0 siblings, 0 replies; 4+ messages in thread
From: Lars Ingebrigtsen @ 2020-08-12 10:54 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 23799

Stefan Kangas <stefan@marxist.se> writes:

> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
>> I've now removed that code that removes the To and the hook, which then
>> allows the user to change their minds.
>
> Does commit 3806ddcb76 mean that this should also be closed, or is there
> anything more to do here?

I think that was all, so I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





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

end of thread, other threads:[~2020-08-12 10:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-19  0:12 bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it Drew Adams
2019-07-28 11:48 ` Lars Ingebrigtsen
2020-08-12  2:20   ` Stefan Kangas
2020-08-12 10:54     ` Lars Ingebrigtsen

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.