bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it

bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it

[Drew Adams]

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'

bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it

[Lars Ingebrigtsen]

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

bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it

[Stefan Kangas]

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

bug#23799: 25.0.90; Replying "no" to send bug report means you cannot later send it

[Lars Ingebrigtsen]

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