]> code.delx.au - gnu-emacs-elpa/blobdiff - packages/debbugs/debbugs-gnu.el
(debbugs-gnu-send-control-message): Add donenotabug and donewontfix.
[gnu-emacs-elpa] / packages / debbugs / debbugs-gnu.el
index d3023249ecb20073f8772b844c8cfe0208de09a4..ded88aaee9b3b65573d5743ed0033890c193e1ea 100644 (file)
 
 ;;; Commentary:
 
+;; This package provides an interface to bug reports which are located
+;; on the GNU bug tracker debbugs.gnu.org.  It's main purpose is to
+;; show and manipulate bug reports from Emacs, but it could be used
+;; also for other GNU projects which use the same bug tracker.
+
+;; If you have `debbugs-gnu.el' in your load-path, you could enable
+;; the bug tracker command by the following line in your ~/.emacs
+;;
+;;   (autoload 'debbugs-gnu "debbugs-gnu" "" 'interactive)
+
+;; The bug tracker is called interactively by
+;;
+;;   M-x debbugs-gnu
+
+;; It asks for the severities, for which bugs shall be shown. This can
+;; be either just one severity, or a list of severities, separated by
+;; comma.  Valid severities are "important", "normal", "minor" or
+;; "wishlist".  There is also the pseudo severity "tagged", which
+;; selects locally tagged bugs.
+
+;; If a prefix is given, more search parameters are asked for, like
+;; packages (also a comma separated list, "emacs" is the default),
+;; whether archived bugs shall be shown, and whether closed bugs shall
+;; be shown.
+
+;; The bug reports are downloaded from the bug tracker.  In order to
+;; not generate too much load of the server, up to 500 bugs will be
+;; downloaded at once.  If there are more hits, you will be asked to
+;; change this limit, but please don't increase this number too much.
+
+;; These default values could be changed also by customer options
+;; `debbugs-gnu-default-severities', `debbugs-gnu-default-packages'
+;; and `debbugs-gnu-default-hits-per-page'.
+
+;; The command creates one or more pages of bug lists.  Every bug is
+;; shown in one line, including the bug number, the status (combining
+;; merged bug numbers, keywords and severities), the name of the
+;; submitter, and the title of the bug.  On every bug line you could
+;; apply the following actions by the following keystrokes:
+
+;;   RET: Show corresponding messages in Gnus
+;;   "C": Send a control message
+;;   "t": Mark the bug locally as tagged
+;;   "d": Show bug attributes
+
+;; Furthermore, you could apply the global actions
+
+;;   "s": Toggle bug sorting
+;;   "g": Rescan bugs
+;;   "x": Suppress closed bugs
+;;   "q": Quit the buffer
+
+;; When you visit the related bug messages in Gnus, you could also
+;; send control messages by keystroke "C".
+
 ;;; Code:
 
 (require 'debbugs)
+(require 'widget)
 (eval-when-compile (require 'cl))
 
 (autoload 'gnus-read-ephemeral-emacs-bug-group "gnus-group")
 (autoload 'gnus-summary-article-header "gnus-sum")
 (autoload 'message-make-from "message")
 
-(defface debbugs-new '((t (:foreground "red")))
-  "Face for new reports that nobody has answered.")
+(defgroup debbugs-gnu ()
+  "UI for the debbugs.gnu.org bug tracker."
+  :group 'debbugs
+  :version "24.1")
 
-(defface debbugs-handled '((t (:foreground "ForestGreen")))
+(defcustom debbugs-gnu-default-severities '("normal")
+  "*The list severities bugs are searched for.
+\"tagged\" is not a severity but marks locally tagged bugs."
+  :group 'debbugs-gnu
+  :type '(set (const "important")
+             (const "normal")
+             (const "minor")
+             (const "wishlist")
+             (const "tagged"))
+  :version "24.1")
+
+(defcustom debbugs-gnu-default-packages '("emacs")
+  "*The list of packages to be searched for."
+  :group 'debbugs-gnu
+  :type '(set (const "automake")
+             (const "coreutils")
+             (const "emacs")
+             (const "gnus")
+             (const "libtool"))
+  :version "24.1")
+
+(defcustom debbugs-gnu-default-hits-per-page 500
+  "*The number of bugs shown per page."
+  :group 'debbugs-gnu
+  :type 'integer
+  :version "24.1")
+
+(defface debbugs-gnu-new '((t (:foreground "red")))
   "Face for new reports that nobody has answered.")
 
-(defface debbugs-stale '((t (:foreground "orange")))
+(defface debbugs-gnu-handled '((t (:foreground "ForestGreen")))
+  "Face for new reports that have been modified recently.")
+
+(defface debbugs-gnu-stale '((t (:foreground "orange")))
   "Face for new reports that nobody has answered.")
 
-(defface debbugs-done '((t (:foreground "DarkGrey")))
+(defface debbugs-gnu-done '((t (:foreground "DarkGrey")))
   "Face for closed bug reports.")
 
-(defun debbugs-emacs (severities &optional package list-done archivedp)
+(defface debbugs-gnu-tagged '((t (:foreground "red")))
+  "Face for reports that have been tagged locally.")
+
+(defvar debbugs-gnu-widgets nil)
+
+(defvar debbugs-gnu-widget-map
+  (let ((map (make-sparse-keymap)))
+    (define-key map "\r" 'widget-button-press)
+    (define-key map [mouse-1] 'widget-button-press)
+    (define-key map [mouse-2] 'widget-button-press)
+    map))
+
+(defvar debbugs-gnu-local-tags nil
+  "List of bug numbers tagged locally, and kept persistent.")
+
+(defvar debbugs-gnu-persistency-file
+  (expand-file-name (locate-user-emacs-file "debbugs"))
+  "File name of a persistency store for debbugs variables")
+
+(defun debbugs-gnu-dump-persistency-file ()
+  "Function to store debbugs variables persistently."
+  (with-temp-file debbugs-gnu-persistency-file
+    (insert
+     ";; -*- emacs-lisp -*-\n"
+     ";; Debbugs tags connection history.  Don't change this file.\n\n"
+     (format "(setq debbugs-gnu-local-tags '%S)"
+            (sort (copy-sequence debbugs-gnu-local-tags) '<)))))
+
+(defvar debbugs-gnu-current-severities nil
+  "The severities strings to be searched for.")
+
+(defvar debbugs-gnu-current-packages nil
+  "The package names to be searched for.")
+
+(defvar debbugs-gnu-current-archive nil
+  "Whether to search in the archive.")
+
+(defun debbugs-gnu (severities &optional packages archivedp suppress-done)
   "List all outstanding Emacs bugs."
   (interactive
-   (list
-    (completing-read "Severity: "
-                    '("important" "normal" "minor" "wishlist")
-                    nil t "normal")))
+   (let (archivedp)
+     (list
+      (completing-read-multiple
+       "Severity: "
+       (mapcar 'cadr (cdr (get 'debbugs-gnu-default-severities 'custom-type)))
+       nil t (mapconcat 'identity debbugs-gnu-default-severities ","))
+      ;; The optional parameters are asked only when there is a prefix.
+      (if current-prefix-arg
+         (completing-read-multiple
+          "Packages: "
+          (mapcar 'cadr (cdr (get 'debbugs-gnu-default-packages 'custom-type)))
+          nil t (mapconcat 'identity debbugs-gnu-default-packages ","))
+       debbugs-gnu-default-packages)
+      (when current-prefix-arg
+       (setq archivedp (y-or-n-p "Show archived bugs?")))
+      (when (and current-prefix-arg (not archivedp))
+       (y-or-n-p "Suppress closed bugs?")))))
+
+  ;; Initialize variables.
+  (when (and (file-exists-p debbugs-gnu-persistency-file)
+            (not debbugs-gnu-local-tags))
+    (with-temp-buffer
+      (insert-file-contents debbugs-gnu-persistency-file)
+      (eval (read (current-buffer)))))
+  ;; Set lists.
   (unless (consp severities)
     (setq severities (list severities)))
-  (pop-to-buffer (get-buffer-create "*Emacs Bugs*"))
-  (debbugs-mode)
+  (unless (consp packages)
+    (setq packages (list packages)))
+
+  (setq debbugs-gnu-current-severities severities
+       debbugs-gnu-current-packages packages
+       debbugs-gnu-current-archive (if archivedp "1" "0")
+       debbugs-gnu-widgets nil)
+
+  (let ((hits debbugs-gnu-default-hits-per-page)
+       (ids (debbugs-gnu-get-bugs)))
+
+    (if (> (length ids) hits)
+       (let ((cursor-in-echo-area nil))
+         (setq hits
+               (string-to-number
+                (read-string
+                 (format
+                  "How many reports (available %d, default %d): "
+                  (length ids) hits)
+                 nil
+                 nil
+                 (number-to-string hits))))))
+
+    (if (> (length ids) hits)
+       (let ((i 0)
+             curr-ids)
+         (while ids
+           (setq i (1+ i)
+                 curr-ids (butlast ids (- (length ids) hits)))
+           (add-to-list
+            'debbugs-gnu-widgets
+            (widget-convert
+             'push-button
+             :follow-link 'mouse-face
+             :notify (lambda (widget &rest ignore)
+                       (debbugs-gnu-show-reports widget))
+             :keymap debbugs-gnu-widget-map
+             :suppress-done suppress-done
+             :buffer-name (format "*Emacs Bugs*<%d>" i)
+             :bug-ids curr-ids
+             :help-echo (format "%d-%d" (car ids) (car (last curr-ids)))
+             :format " %[%v%]"
+             (number-to-string i))
+            'append)
+           (setq ids (last ids (- (length ids) hits))))
+         (debbugs-gnu-show-reports (car debbugs-gnu-widgets)))
+
+      (debbugs-gnu-show-reports
+       (widget-convert
+       'const
+       :suppress-done suppress-done
+       :buffer-name "*Emacs Bugs*"
+       :bug-ids ids)))))
+
+(defun debbugs-gnu-get-bugs ()
+  "Retrieve bugs numbers from debbugs.gnu.org according search criteria."
   (let ((debbugs-port "gnu.org")
-       (buffer-read-only nil)
-       (ids nil)
-       (default 500))
-    (dolist (severity severities)
-      (setq ids (nconc ids
-                      (debbugs-get-bugs :package (or package "emacs")
-                                        :severity severity
-                                        :archive (if archivedp
-                                                     "1" "0")))))
+       ids)
+    (dolist (severity debbugs-gnu-current-severities)
+      (if (string-equal severity "tagged")
+         (setq ids (nconc ids (copy-sequence debbugs-gnu-local-tags)))
+       (dolist (package debbugs-gnu-current-packages)
+         (setq ids
+               (nconc ids
+                      (debbugs-get-bugs
+                       :package package
+                       :severity severity
+                       :archive debbugs-gnu-current-archive))))))
+    (sort ids '<)))
+
+(defvar debbugs-gnu-current-widget nil)
+
+(defvar widget-mouse-face)
+
+(defun debbugs-gnu-show-reports (widget)
+  "Show bug reports as given in WIDGET property :bug-ids."
+  (pop-to-buffer (get-buffer-create (widget-get widget :buffer-name)))
+  (debbugs-gnu-mode)
+  (let ((inhibit-read-only t)
+       (debbugs-port "gnu.org")
+       (suppress-done (widget-get widget :suppress-done)))
     (erase-buffer)
 
-    (when (> (length ids) default)
-      (let* ((cursor-in-echo-area nil)
-            (input
-             (read-string
-              (format
-               "How many reports (available %d, default %d): "
-               (length ids) default)
-              nil
-              nil
-              (number-to-string default))))
-       (setq ids (last (sort ids '<) (string-to-number input)))))
-
-    (dolist (status (sort (apply 'debbugs-get-status ids)
+    (when debbugs-gnu-widgets
+      (widget-insert "Page:")
+      (mapc
+       (lambda (obj)
+        (if (eq obj widget)
+            (widget-put obj :button-face 'widget-button-pressed)
+          (widget-put obj :button-face 'widget-button-face))
+        (widget-apply obj :create))
+       debbugs-gnu-widgets)
+      (widget-insert "\n\n"))
+
+    (dolist (status (sort (apply 'debbugs-get-status
+                                (widget-get widget :bug-ids))
                          (lambda (s1 s2)
                            (< (cdr (assq 'id s1))
                               (cdr (assq 'id s2))))))
-      (when (or list-done
+      (when (or (not suppress-done)
                (not (equal (cdr (assq 'pending status)) "done")))
-       (let ((address (mail-header-parse-address
-                       (decode-coding-string (cdr (assq 'originator status))
-                                             'utf-8)))
-             (subject (decode-coding-string (cdr (assq 'subject status))
-                                            'utf-8)))
+       (let* ((id (cdr (assq 'id status)))
+              (words
+               (mapconcat
+                'identity
+                (cons (cdr (assq 'severity status))
+                      (cdr (assq 'keywords status)))
+                ","))
+              (face (cond
+                     ((equal (cdr (assq 'pending status)) "done")
+                      'debbugs-gnu-done)
+                     ((= (cdr (assq 'date status))
+                         (cdr (assq 'log_modified status)))
+                      'debbugs-gnu-new)
+                     ((< (- (float-time)
+                            (cdr (assq 'log_modified status)))
+                         (* 60 60 24 4))
+                      'debbugs-gnu-handled)
+                     (t
+                      'debbugs-gnu-stale)))
+              (address (mail-header-parse-address
+                        (decode-coding-string (cdr (assq 'originator status))
+                                              'utf-8)))
+              (owner (if (cdr (assq 'owner status))
+                         (car (mail-header-parse-address
+                               (decode-coding-string (cdr (assq 'owner status))
+                                                     'utf-8)))))
+              (subject (decode-coding-string (cdr (assq 'subject status))
+                                             'utf-8))
+              merged)
+         (unless (equal (cdr (assq 'pending status)) "pending")
+           (setq words
+                 (concat words "," (cdr (assq 'pending status)))))
+         (when (setq merged (cdr (assq 'mergedwith status)))
+           (setq words (format "%s,%s"
+                               (if (numberp merged)
+                                   merged
+                                 (mapconcat 'number-to-string merged ","))
+                               words)))
+         (setq words (propertize words 'face face))
          (setq address
-               ;; Prefer the name over the address.
-               (or (cdr address)
-                   (car address)))
+               (propertize
+                ;; Prefer the name over the address.
+                (or (cdr address)
+                    (car address))
+                'face
+                ;; Mark own submitted bugs.
+                (if (and (stringp (car address))
+                         (string-equal (car address) user-mail-address))
+                    'debbugs-gnu-tagged
+                  'default)))
          (insert
           (format "%5d %-20s [%-23s] %s\n"
-                  (cdr (assq 'id status))
-                  (let ((words
-                         (mapconcat
-                          'identity
-                          (cons (cdr (assq 'severity status))
-                                (cdr (assq 'keywords status)))
-                          ",")))
-                    (unless (equal (cdr (assq 'pending status)) "pending")
-                      (setq words
-                            (concat words "," (cdr (assq 'pending status)))))
-                    (if (> (length words) 20)
-                        (propertize (substring words 0 20) 'help-echo words)
-                      words))
+                  id
+                  (if (> (length words) 20)
+                      (propertize (substring words 0 20) 'help-echo words)
+                    words)
                   (if (> (length address) 23)
                       (propertize (substring address 0 23) 'help-echo address)
                     address)
-                  (propertize subject 'help-echo subject)))
+                  ;; Mark owned bugs.
+                  (if (and (stringp owner)
+                           (string-equal owner user-mail-address))
+                      (propertize subject
+                                  'face 'debbugs-gnu-tagged 'help-echo subject)
+                    (propertize subject 'help-echo subject))))
          (forward-line -1)
+         (put-text-property (point) (1+ (point)) 'debbugs-gnu-status status)
          (put-text-property
-          (+ (point) 5) (+ (point) 26)
-          'face
-          (cond
-           ((equal (cdr (assq 'pending status)) "done")
-            'debbugs-done)
-           ((= (cdr (assq 'date status))
-               (cdr (assq 'log_modified status)))
-            'debbugs-new)
-           ((< (- (float-time)
-                  (cdr (assq 'log_modified status)))
-               (* 60 60 24 4))
-            'debbugs-handled)
-           (t
-            'debbugs-stale)))
-         (forward-line 1)))))
-  (goto-char (point-min)))
+          (point-at-bol) (point-at-eol) 'mouse-face widget-mouse-face)
+         (when (memq id debbugs-gnu-local-tags)
+           (put-text-property
+            (+ (point) (- 5 (length (number-to-string id)))) (+ (point) 5)
+            'face 'debbugs-gnu-tagged))
+         (forward-line 1))))
+
+    (when debbugs-gnu-widgets
+      (widget-insert "\nPage:")
+      (mapc (lambda (obj) (widget-apply obj :create)) debbugs-gnu-widgets)
+      (widget-setup))
+
+    (set-buffer-modified-p nil)
+    (set (make-local-variable 'debbugs-gnu-current-widget)
+        widget)
+    (goto-char (point-min))))
+
+(defvar debbugs-gnu-mode-map
+  (let ((map (make-sparse-keymap)))
+    (define-key map "\r" 'debbugs-gnu-select-report)
+    (define-key map [mouse-1] 'debbugs-gnu-select-report)
+    (define-key map [mouse-2] 'debbugs-gnu-select-report)
+    (define-key map "q" 'bury-buffer)
+    (define-key map "s" 'debbugs-gnu-toggle-sort)
+    (define-key map "t" 'debbugs-gnu-toggle-tag)
+    (define-key map "d" 'debbugs-gnu-display-status)
+    (define-key map "g" 'debbugs-gnu-rescan)
+    (define-key map "x" 'debbugs-gnu-suppress-done)
+    (define-key map "C" 'debbugs-gnu-send-control-message)
+    map))
+
+(defun debbugs-gnu-rescan ()
+  "Rescan the current set of bug reports."
+  (interactive)
+
+  ;; The last page will be provided with new bug ids.
+  ;; TODO: Do it also for the other pages.
+  (when (and debbugs-gnu-widgets
+            (eq debbugs-gnu-current-widget (car (last debbugs-gnu-widgets))))
+    (let ((first-id (car (widget-get debbugs-gnu-current-widget :bug-ids)))
+         (last-id  (car
+                    (last (widget-get debbugs-gnu-current-widget :bug-ids))))
+         (ids (debbugs-gnu-get-bugs)))
+
+      (while (and (<= first-id last-id) (not (memq first-id ids)))
+       (setq first-id (1+ first-id)))
 
-(defvar debbugs-mode-map nil)
-(unless debbugs-mode-map
-  (setq debbugs-mode-map (make-sparse-keymap))
-  (define-key debbugs-mode-map "\r" 'debbugs-select-report)
-  (define-key debbugs-mode-map "q"  'kill-buffer))
+      (when (<= first-id last-id)
+       (widget-put debbugs-gnu-current-widget :bug-ids (memq first-id ids)))))
 
-(defun debbugs-mode ()
+  ;; Refresh the buffer.  `save-excursion' does not work, so we
+  ;; remember the position.
+  (let ((pos (point)))
+    (debbugs-gnu-show-reports debbugs-gnu-current-widget)
+    (goto-char pos)))
+
+(defvar debbugs-gnu-sort-state 'number)
+
+(defun debbugs-gnu-mode ()
   "Major mode for listing bug reports.
 
 All normal editing commands are switched off.
-\\<debbugs-mode-map>
+\\<debbugs-gnu-mode-map>
 
 The following commands are available:
 
-\\{debbugs-mode-map}"
+\\{debbugs-gnu-mode-map}"
   (interactive)
   (kill-all-local-variables)
-  (setq major-mode 'debbugs-mode)
+  (setq major-mode 'debbugs-gnu-mode)
   (setq mode-name "Debbugs")
-  (use-local-map debbugs-mode-map)
+  (use-local-map debbugs-gnu-mode-map)
+  (set (make-local-variable 'debbugs-gnu-sort-state)
+       'number)
   (buffer-disable-undo)
   (setq truncate-lines t)
   (setq buffer-read-only t))
 
-(defun debbugs-select-report ()
+(defvar debbugs-gnu-state-preference
+  '((debbugs-gnu-new . 1)
+    (debbugs-gnu-stale . 2)
+    (debbugs-gnu-handled . 3)
+    (debbugs-gnu-done . 4)))
+
+(defun debbugs-gnu-toggle-sort ()
+  "Toggle sorting by age and by state."
+  (interactive)
+  (beginning-of-line)
+  (let ((buffer-read-only nil)
+       (before-change-functions nil)
+       (current-bug (debbugs-gnu-current-id t))
+       (start-point (point)))
+    (setq debbugs-gnu-sort-state
+         (if (eq debbugs-gnu-sort-state 'number)
+             'state
+           'number))
+    (goto-char (point-min))
+    (while (and (not (eobp))
+               (not (get-text-property (point) 'debbugs-gnu-status)))
+      (forward-line 1))
+    (save-restriction
+      (narrow-to-region
+       (point)
+       (progn
+        (goto-char (point-max))
+        (beginning-of-line)
+        (while (and (not (bobp))
+                    (not (get-text-property (point) 'debbugs-gnu-status)))
+          (forward-line -1))
+        (forward-line 1)
+        (point)))
+      (goto-char (point-min))
+      (sort-subr
+       nil (lambda () (forward-line 1)) 'end-of-line
+       (lambda ()
+        (let ((id (debbugs-gnu-current-id)))
+          (if (eq debbugs-gnu-sort-state 'number)
+              id
+            ;; Sort the tagged ones at the end.
+            (or (and (memq id debbugs-gnu-local-tags)
+                     20)
+                (cdr (assq (get-text-property (+ (point) 7) 'face)
+                           debbugs-gnu-state-preference))
+                10))))))
+    (if (not current-bug)
+       (goto-char start-point)
+      (goto-char (point-min))
+      (re-search-forward (format "^%d" current-bug) nil t))))
+
+(defun debbugs-gnu-toggle-tag ()
+  "Toggle tag of the report in the current line."
+  (interactive)
+  (save-excursion
+    (beginning-of-line)
+    (let ((inhibit-read-only t)
+         (id (debbugs-gnu-current-id)))
+      (if (memq id debbugs-gnu-local-tags)
+         (progn
+           (setq debbugs-gnu-local-tags (delq id debbugs-gnu-local-tags))
+           (put-text-property (point) (+ (point) 5) 'face 'default))
+       (add-to-list 'debbugs-gnu-local-tags id)
+       (put-text-property
+        (+ (point) (- 5 (length (number-to-string id)))) (+ (point) 5)
+        'face 'debbugs-gnu-tagged))))
+  (debbugs-gnu-dump-persistency-file))
+
+(defun debbugs-gnu-suppress-done ()
+  "Suppress bugs marked as done."
+  (interactive)
+  (save-excursion
+    (unless (widget-get debbugs-gnu-current-widget :suppress-done)
+      (let ((inhibit-read-only t))
+       (widget-put debbugs-gnu-current-widget :suppress-done t)
+       (goto-char (point-min))
+       (while (and (not (eobp))
+                   (not (get-text-property (point) 'debbugs-gnu-status)))
+         (forward-line 1))
+       (while  (and (not (eobp))
+                    (get-text-property (point) 'debbugs-gnu-status))
+         (if (equal (cdr (assq 'pending (debbugs-gnu-current-status))) "done")
+             (kill-region (point) (progn (forward-line 1) (point)))
+           (forward-line 1)))))))
+
+(defvar debbugs-gnu-bug-number nil)
+
+(defun debbugs-gnu-current-id (&optional noerror)
+  (or (cdr (assq 'id (debbugs-gnu-current-status)))
+      (and (not noerror)
+          (error "No bug on the current line"))))
+
+(defun debbugs-gnu-current-status ()
+  (get-text-property (line-beginning-position)
+                    'debbugs-gnu-status))
+
+(defun debbugs-gnu-display-status (status)
+  "Display the status of the report on the current line."
+  (interactive (list (debbugs-gnu-current-status)))
+  (pop-to-buffer "*Bug Status*")
+  (erase-buffer)
+  (pp status (current-buffer))
+  (goto-char (point-min)))
+
+(defun debbugs-gnu-select-report ()
   "Select the report on the current line."
   (interactive)
-  (let (id)
-    (save-excursion
-      (beginning-of-line)
-      (if (not (looking-at " *\\([0-9]+\\)"))
-         (error "No bug report on the current line")
-       (setq id (string-to-number (match-string 1)))))
+  ;; We open the report messages.
+  (let* ((status (debbugs-gnu-current-status))
+        (id (cdr (assq 'id status)))
+        (merged (cdr (assq 'mergedwith status))))
     (gnus-read-ephemeral-emacs-bug-group
-     id (cons (current-buffer)
-             (current-window-configuration)))
+     (cons id (if (listp merged)
+                 merged
+               (list merged)))
+     (cons (current-buffer)
+          (current-window-configuration)))
     (with-current-buffer (window-buffer (selected-window))
-      (debbugs-summary-mode 1))))
+      (debbugs-gnu-summary-mode 1)
+      (set (make-local-variable 'debbugs-gnu-bug-number) id))))
 
-(defvar debbugs-summary-mode-map
+(defvar debbugs-gnu-summary-mode-map
   (let ((map (make-sparse-keymap)))
-    (define-key map "C" 'debbugs-send-control-message)
+    (define-key map "C" 'debbugs-gnu-send-control-message)
     map))
 
-(define-minor-mode debbugs-summary-mode
+(defvar gnus-posting-styles)
+
+(define-minor-mode debbugs-gnu-summary-mode
   "Minor mode for providing a debbugs interface in Gnus summary buffers.
 
-\\{debbugs-summary-mode-map}"
-  :lighter " Debbugs" :keymap debbugs-summary-mode-map
+\\{debbugs-gnu-summary-mode-map}"
+  :lighter " Debbugs" :keymap debbugs-gnu-summary-mode-map
   (set (make-local-variable 'gnus-posting-styles)
        '((".*"
          (eval
           (with-current-buffer gnus-article-copy
             (set (make-local-variable 'message-prune-recipient-rules)
                  '((".*@debbugs.*" "emacs-pretest-bug")
-                   (".*@debbugs.*" "bug-gnu-emacs")))
+                   (".*@debbugs.*" "bug-gnu-emacs")
+                   ("[0-9]+@debbugs.*" "submit@debbugs.gnu.org")
+                   ("[0-9]+@debbugs.*" "quiet@debbugs.gnu.org")))
             (set (make-local-variable 'message-alter-recipients-function)
                  (lambda (address)
                    (if (string-match "\\([0-9]+\\)@donarmstrong" (car address))
@@ -198,25 +598,27 @@ The following commands are available:
                          (cons new new))
                      address)))))))))
 
-(defun debbugs-send-control-message (message)
+(defun debbugs-gnu-send-control-message (message &optional reverse)
   "Send a control message for the current bug report.
 You can set the severity or add a tag, or close the report.  If
 you use the special \"done\" MESSAGE, the report will be marked as
-fixed, and then closed."
+fixed, and then closed.
+
+If given a prefix, and given a tag to set, the tag will be
+removed instead."
   (interactive
    (list (completing-read
          "Control message: "
          '("important" "normal" "minor" "wishlist"
-           "done"
+           "done" "donenotabug" "donewontfix"
            "unarchive" "reopen" "close"
            "merge" "forcemerge"
+           "owner" "noowner"
            "patch" "wontfix" "moreinfo" "unreproducible" "fixed" "notabug")
-         nil t)))
-  (let* ((subject (mail-header-subject (gnus-summary-article-header)))
-        (id
-         (if (string-match "bug#\\([0-9]+\\)" subject)
-             (string-to-number (match-string 1 subject))
-           (error "No bug number present")))
+         nil t)
+        current-prefix-arg))
+  (let* ((id (or debbugs-gnu-bug-number        ; Set on group entry.
+                (debbugs-gnu-current-id)))
         (version
          (when (member message '("close" "done"))
            (read-string
@@ -241,19 +643,25 @@ fixed, and then closed."
              (format "Subject: control message for bug #%d\n" id)
              "\n"
              (cond
-              ((member message '("unarchive" "reopen"))
+              ((member message '("unarchive" "reopen" "noowner"))
                (format "%s %d\n" message id))
               ((member message '("merge" "forcemerge"))
                (format "%s %d %s\n" message id
                        (read-string "Merge with bug #: ")))
+              ((equal message "owner")
+               (format "owner %d !\n" id))
               ((equal message "close")
                (format "close %d %s\n" id version))
               ((equal message "done")
                (format "tags %d fixed\nclose %d %s\n" id id version))
+              ((member message '("donenotabug" "donewontfix"))
+               (format "tags %d %s\nclose %d\n" id (substring message 4) id))
               ((member message '("important" "normal" "minor" "wishlist"))
                (format "severity %d %s\n" id message))
               (t
-               (format "tags %d %s\n" id message))))
+               (format "tags %d%s %s\n"
+                       id (if reverse " -" "")
+                       message))))
       (funcall send-mail-function))))
 
 (provide 'debbugs-gnu)