Replace "Maintainer: FSF" with the emacs-devel mailing address
[bpt/emacs.git] / lisp / mail / reporter.el
CommitLineData
bf3b07d3
RS
1;;; reporter.el --- customizable bug reporting of lisp programs
2
ba318903 3;; Copyright (C) 1993-1998, 2001-2014 Free Software Foundation, Inc.
b578f267 4
72fe4615 5;; Author: 1993-1998 Barry A. Warsaw
34dc21db 6;; Maintainer: emacs-devel@gnu.org
bf3b07d3 7;; Created: 19-Apr-1993
c71437cf 8;; Keywords: maint mail tools
bf3b07d3 9
54d2ecd3 10;; This file is part of GNU Emacs.
9ee7c69d 11
b1fc2b50 12;; GNU Emacs is free software: you can redistribute it and/or modify
bf3b07d3 13;; it under the terms of the GNU General Public License as published by
b1fc2b50
GM
14;; the Free Software Foundation, either version 3 of the License, or
15;; (at your option) any later version.
9ee7c69d
RS
16
17;; GNU Emacs is distributed in the hope that it will be useful,
bf3b07d3
RS
18;; but WITHOUT ANY WARRANTY; without even the implied warranty of
19;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
20;; GNU General Public License for more details.
9ee7c69d 21
bf3b07d3 22;; You should have received a copy of the GNU General Public License
b1fc2b50 23;; along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>.
bf3b07d3 24
4f40f169 25;;; Commentary:
b578f267 26
c71437cf
MB
27;; End User Interface
28;; ==================
29;; The variable `mail-user-agent' contains a symbol indicating which
30;; Emacs mail package end users would like to use to compose outgoing
72fe4615
RS
31;; mail. See that variable for details (it is no longer defined in
32;; this file).
c71437cf 33
c71437cf
MB
34;; Lisp Package Authors
35;; ====================
72fe4615
RS
36;; reporter.el was written primarily for Emacs Lisp package authors so
37;; that their users can more easily report bugs. When invoked,
38;; `reporter-submit-bug-report' will set up an outgoing mail buffer
39;; with the appropriate bug report address, including a lisp
40;; expression the maintainer of the package can evaluate to completely
41;; reproduce the environment in which the bug was observed (e.g. by
42;; using `eval-last-sexp'). This package proved especially useful
43;; during my development of CC Mode, which is highly dependent on its
c71437cf 44;; configuration variables.
bf3b07d3
RS
45;;
46;; Do a "C-h f reporter-submit-bug-report" for more information.
47;; Here's an example usage:
48;;
4f40f169
RS
49;;(defconst mypkg-version "9.801")
50;;(defconst mypkg-maintainer-address "mypkg-help@foo.com")
51;;(defun mypkg-submit-bug-report ()
52;; "Submit via mail a bug report on mypkg"
53;; (interactive)
72fe4615 54;; (require 'reporter)
4f40f169
RS
55;; (reporter-submit-bug-report
56;; mypkg-maintainer-address
57;; (concat "mypkg.el " mypkg-version)
58;; (list 'mypkg-variable-1
59;; 'mypkg-variable-2
60;; ;; ...
61;; 'mypkg-variable-last)))
62
31257c3f 63;;; Code:
c71437cf 64
72fe4615 65\f
4b4052b2
RS
66;; vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
67;; Package author interface variables
68
4f40f169
RS
69(defvar reporter-prompt-for-summary-p nil
70 "Interface variable controlling prompting for problem summary.
71When non-nil, `reporter-submit-bug-report' prompts the user for a
72brief summary of the problem, and puts this summary on the Subject:
c71437cf
MB
73line. If this variable is a string, that string is used as the prompt
74string.
4f40f169 75
87617309 76Default behavior is to not prompt (i.e. nil). If you want reporter to
c71437cf 77prompt, you should `let' bind this variable before calling
4f40f169
RS
78`reporter-submit-bug-report'. Note that this variable is not
79buffer-local so you should never just `setq' it.")
80
313f3cb4 81(defvar reporter-dont-compact-list nil
22cc6690 82 "Interface variable controlling compacting of list values.
313f3cb4
RS
83When non-nil, this must be a list of variable symbols. When a
84variable containing a list value is formatted in the bug report mail
85buffer, it normally is compacted so that its value fits one the fewest
86number of lines. If the variable's symbol appears in this list, its
87value is printed in a more verbose style, specifically, one elemental
88sexp per line.
89
90Note that this variable is not buffer-local so you should never just
91`setq' it. If you want to changes its default value, you should `let'
92bind it.")
bf3b07d3 93
c71437cf 94;; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
4b4052b2
RS
95;; End of editable variables
96
c71437cf 97\f
9ee9b53e
RS
98(defvar reporter-eval-buffer nil
99 "Buffer to retrieve variable's value from.
100This is necessary to properly support the printing of buffer-local
101variables. Current buffer will always be the mail buffer being
102composed.")
9ee7c69d 103
4f40f169
RS
104(defvar reporter-initial-text nil
105 "The automatically created initial text of a bug report.")
106(make-variable-buffer-local 'reporter-initial-text)
107
c71437cf 108
313f3cb4 109\f
c71437cf 110;; status feedback to the user
313f3cb4
RS
111(defvar reporter-status-message nil)
112(defvar reporter-status-count nil)
113
114(defun reporter-update-status ()
87617309 115 "Periodically output a status message."
313f3cb4
RS
116 (if (zerop (% reporter-status-count 10))
117 (progn
274f1353 118 (message "%s" reporter-status-message)
313f3cb4
RS
119 (setq reporter-status-message (concat reporter-status-message "."))))
120 (setq reporter-status-count (1+ reporter-status-count)))
4f40f169 121
bf3b07d3 122\f
c71437cf 123;; dumping/pretty printing of values
313f3cb4 124(defun reporter-beautify-list (maxwidth compact-p)
87617309 125 "Pretty print a list."
313f3cb4 126 (reporter-update-status)
72fe4615
RS
127 (let ((move t)
128 linebreak indent-enclosing-p indent-p here)
313f3cb4
RS
129 (condition-case nil ;loop exit
130 (progn
131 (down-list 1)
132 (setq indent-enclosing-p t)
72fe4615 133 (while move
313f3cb4 134 (setq here (point))
72fe4615
RS
135 ;; The following line is how we break out of the while
136 ;; loop, in one of two ways. Either we've hit the end of
137 ;; the buffer, in which case scan-sexps returns nil, or
138 ;; we've crossed unbalanced parens and it will raise an
139 ;; error we're expecting to catch.
140 (setq move (scan-sexps (point) 1))
141 (goto-char move)
313f3cb4
RS
142 (if (<= maxwidth (current-column))
143 (if linebreak
144 (progn
145 (goto-char linebreak)
146 (newline-and-indent)
147 (setq linebreak nil))
148 (goto-char here)
149 (setq indent-p (reporter-beautify-list maxwidth compact-p))
150 (goto-char here)
151 (forward-sexp 1)
152 (if indent-p
153 (newline-and-indent))
154 t)
155 (if compact-p
156 (setq linebreak (point))
157 (newline-and-indent))
158 ))
159 t)
160 (error indent-enclosing-p))))
161
162(defun reporter-lisp-indent (indent-point state)
87617309 163 "A better lisp indentation style for bug reporting."
313f3cb4
RS
164 (save-excursion
165 (goto-char (1+ (nth 1 state)))
166 (current-column)))
167
2b54af74
DN
168(declare-function mail-position-on-field "sendmail" (field &optional soft))
169(declare-function mail-text "sendmail" ())
170
4f40f169 171(defun reporter-dump-variable (varsym mailbuf)
87617309
DL
172 "Pretty-print the value of the variable in symbol VARSYM.
173MAILBUF is the mail buffer being composed."
313f3cb4 174 (reporter-update-status)
4f40f169 175 (condition-case nil
937e6a56 176 (let ((val (with-current-buffer reporter-eval-buffer
4f40f169
RS
177 (symbol-value varsym)))
178 (sym (symbol-name varsym))
179 (print-escape-newlines t)
313f3cb4 180 (maxwidth (1- (window-width)))
4f40f169
RS
181 (here (point)))
182 (insert " " sym " "
183 (cond
184 ((memq val '(t nil)) "")
185 ((listp val) "'")
186 ((symbolp val) "'")
187 (t ""))
188 (prin1-to-string val))
313f3cb4 189 (lisp-indent-line)
4f40f169
RS
190 ;; clean up lists, but only if the line as printed was long
191 ;; enough to wrap
313f3cb4
RS
192 (if (and val ;nil is a list, but short
193 (listp val)
194 (<= maxwidth (current-column)))
4f40f169 195 (save-excursion
313f3cb4
RS
196 (let ((compact-p (not (memq varsym reporter-dont-compact-list)))
197 (lisp-indent-function 'reporter-lisp-indent))
198 (goto-char here)
199 (reporter-beautify-list maxwidth compact-p))))
4f40f169
RS
200 (insert "\n"))
201 (void-variable
937e6a56 202 (with-current-buffer mailbuf
b7e615b9
GM
203 (save-excursion
204 (mail-position-on-field "X-Reporter-Void-Vars-Found")
205 (end-of-line)
206 (insert (symbol-name varsym) " "))))
c71437cf
MB
207 (error
208 (error ""))))
bf3b07d3
RS
209
210(defun reporter-dump-state (pkgname varlist pre-hooks post-hooks)
87617309
DL
211 "Dump the state of the mode specific variables.
212PKGNAME contains the name of the mode as it will appear in the bug
213report (you must explicitly concat any version numbers).
214
215VARLIST is the list of variables to dump. Each element in
216VARLIST can be a variable symbol, or a cons cell. If a symbol,
217this will be passed to `reporter-dump-variable' for insertion
218into the mail buffer. If a cons cell, the car must be a variable
219symbol and the cdr must be a function which will be `funcall'd
220with arguments the symbol and the mail buffer being composed. Use
221this to write your own custom variable value printers for
222specific variables.
223
224Note that the global variable `reporter-eval-buffer' will be bound to
225the buffer in which `reporter-submit-bug-report' was invoked. If you
226want to print the value of a buffer local variable, you should wrap
227the `eval' call in your custom printer inside a `set-buffer' (and
228probably a `save-excursion'). `reporter-dump-variable' handles this
229properly.
230
231PRE-HOOKS is run after the Emacs version and PKGNAME are inserted, but
232before the VARLIST is dumped. POST-HOOKS is run after the VARLIST is
233dumped."
bf3b07d3
RS
234 (let ((buffer (current-buffer)))
235 (set-buffer buffer)
4f40f169
RS
236 (insert "Emacs : " (emacs-version) "\n")
237 (and pkgname
238 (insert "Package: " pkgname "\n"))
bf3b07d3 239 (run-hooks 'pre-hooks)
4f40f169
RS
240 (if (not varlist)
241 nil
242 (insert "\ncurrent state:\n==============\n")
243 ;; create an emacs-lisp-mode buffer to contain the output, which
244 ;; we'll later insert into the mail buffer
245 (condition-case fault
246 (let ((mailbuf (current-buffer))
247 (elbuf (get-buffer-create " *tmp-reporter-buffer*")))
937e6a56 248 (with-current-buffer elbuf
4f40f169
RS
249 (emacs-lisp-mode)
250 (erase-buffer)
251 (insert "(setq\n")
252 (lisp-indent-line)
cfedf6cc 253 (mapc
4f40f169
RS
254 (function
255 (lambda (varsym-or-cons-cell)
256 (let ((varsym (or (car-safe varsym-or-cons-cell)
257 varsym-or-cons-cell))
258 (printer (or (cdr-safe varsym-or-cons-cell)
259 'reporter-dump-variable)))
260 (funcall printer varsym mailbuf)
261 )))
262 varlist)
313f3cb4
RS
263 (lisp-indent-line)
264 (insert ")\n"))
134c6d58 265 (insert-buffer-substring elbuf))
4f40f169
RS
266 (error
267 (insert "State could not be dumped due to the following error:\n\n"
268 (format "%s" fault)
269 "\n\nYou should still send this bug report."))))
bf3b07d3
RS
270 (run-hooks 'post-hooks)
271 ))
272
4f40f169 273\f
c71437cf 274(defun reporter-compose-outgoing ()
87617309
DL
275 "Compose the outgoing mail buffer.
276
277Return the selected paradigm, with the current buffer tacked onto the
278beginning of the list."
c71437cf
MB
279 (let* ((agent mail-user-agent)
280 (compose (get mail-user-agent 'composefunc)))
281 ;; Sanity check. If this fails then we'll try to use the SENDMAIL
282 ;; protocol, otherwise we must signal an error.
c6d354e7 283 (if (not (and compose (functionp compose)))
c71437cf
MB
284 (progn
285 (setq agent 'sendmail-user-agent
286 compose (get agent 'composefunc))
c6d354e7 287 (if (not (and compose (functionp compose)))
31257c3f 288 (error "Could not find a valid `mail-user-agent'")
c71437cf 289 (ding)
31257c3f 290 (message "`%s' is an invalid `mail-user-agent'; using `sendmail-user-agent'"
c71437cf
MB
291 mail-user-agent)
292 )))
293 (funcall compose)
294 agent))
295
296\f
4f40f169 297;;;###autoload
bf3b07d3
RS
298(defun reporter-submit-bug-report
299 (address pkgname varlist &optional pre-hooks post-hooks salutation)
93baa0ea 300"Begin submitting a bug report via email.
4f40f169 301
93baa0ea
GM
302ADDRESS is the email address for the package's maintainer. PKGNAME is
303the name of the package (if you want to include version numbers,
304you must put them into PKGNAME before calling this function).
87617309
DL
305Optional PRE-HOOKS and POST-HOOKS are passed to `reporter-dump-state'.
306Optional SALUTATION is inserted at the top of the mail buffer,
307and point is left after the salutation.
bf3b07d3 308
93baa0ea
GM
309VARLIST is the list of variables to dump (see `reporter-dump-state'
310for details). The optional argument PRE-HOOKS and POST-HOOKS are
311passed to `reporter-dump-state'. Optional argument SALUTATION is text
312to be inserted at the top of the mail buffer; in that case, point is
313left after that text.
bf3b07d3 314
93baa0ea
GM
315This function prompts for a summary if `reporter-prompt-for-summary-p'
316is non-nil.
317
318This function does not send a message; it uses the given information
c6c8cba4 319to initialize a message, which the user can then edit and finally send
1407571d 320\(or decline to send). The variable `mail-user-agent' controls which
93baa0ea 321mail-sending package is used for editing and sending the message."
9ee9b53e 322 (let ((reporter-eval-buffer (current-buffer))
4f40f169
RS
323 final-resting-place
324 after-sep-pos
313f3cb4
RS
325 (reporter-status-message "Formatting bug report buffer...")
326 (reporter-status-count 0)
4f40f169 327 (problem (and reporter-prompt-for-summary-p
c71437cf
MB
328 (read-string (if (stringp reporter-prompt-for-summary-p)
329 reporter-prompt-for-summary-p
330 "(Very) brief summary of problem: "))))
331 (agent (reporter-compose-outgoing))
332 (mailbuf (current-buffer))
333 hookvar)
334 ;; do the work
bf3b07d3 335 (require 'sendmail)
3199b96f
CY
336 ;; Just in case the original buffer is not visible now, bring it
337 ;; back somewhere
338 (display-buffer reporter-eval-buffer)
c71437cf 339 ;; If mailbuf did not get made visible before, make it visible now.
3199b96f 340 (pop-to-buffer mailbuf)
bf3b07d3 341 (goto-char (point-min))
fc48f381
RS
342 (mail-position-on-field "to")
343 (insert address)
344 ;; insert problem summary if available
868490bb
GJ
345 (when (and reporter-prompt-for-summary-p problem)
346 (mail-position-on-field "subject")
347 (if pkgname (insert pkgname "; "))
348 (insert problem))
fc48f381
RS
349 ;; move point to the body of the message
350 (mail-text)
351 (forward-line 1)
352 (setq after-sep-pos (point))
353 (and salutation (insert "\n" salutation "\n\n"))
354 (unwind-protect
355 (progn
356 (setq final-resting-place (point-marker))
357 (insert "\n\n")
358 (reporter-dump-state pkgname varlist pre-hooks post-hooks)
359 (goto-char final-resting-place))
360 (set-marker final-resting-place nil))
4f40f169
RS
361
362 ;; save initial text and set up the `no-empty-submission' hook.
c71437cf
MB
363 ;; This only works for mailers that support a pre-send hook, and
364 ;; for which the paradigm has a non-nil value for the `hookvar'
365 ;; key in its agent (i.e. sendmail.el's mail-send-hook).
366 (save-excursion
367 (goto-char (point-max))
368 (skip-chars-backward " \t\n")
369 (setq reporter-initial-text (buffer-substring after-sep-pos (point))))
370 (if (setq hookvar (get agent 'hookvar))
1407571d 371 (add-hook hookvar 'reporter-bug-hook nil t))
c71437cf
MB
372
373 ;; compose the minibuf message and display this.
374 (let* ((sendkey-whereis (where-is-internal
375 (get agent 'sendfunc) nil t))
376 (abortkey-whereis (where-is-internal
377 (get agent 'abortfunc) nil t))
378 (sendkey (if sendkey-whereis
379 (key-description sendkey-whereis)
380 "C-c C-c")) ; TBD: BOGUS hardcode
381 (abortkey (if abortkey-whereis
382 (key-description abortkey-whereis)
383 "M-x kill-buffer")) ; TBD: BOGUS hardcode
384 )
385 (message "Please enter your report. Type %s to send, %s to abort."
386 sendkey abortkey))
bf3b07d3
RS
387 ))
388
4f40f169 389(defun reporter-bug-hook ()
87617309 390 "Prohibit sending mail if empty bug report."
4f40f169
RS
391 (let ((after-sep-pos
392 (save-excursion
0b28758d 393 (rfc822-goto-eoh)
4f40f169
RS
394 (forward-line 1)
395 (point))))
396 (save-excursion
397 (goto-char (point-max))
398 (skip-chars-backward " \t\n")
399 (if (and (= (- (point) after-sep-pos)
400 (length reporter-initial-text))
401 (string= (buffer-substring after-sep-pos (point))
402 reporter-initial-text))
72fe4615 403 (error "Empty bug report cannot be sent"))
4f40f169
RS
404 )))
405
406\f
c71437cf 407(provide 'reporter)
ab5796a9 408
bf3b07d3 409;;; reporter.el ends here