emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[elpa] master 87be44e: Fix debbugs-gnu-announce-commit for ELPA bugs


From: Noam Postavsky
Subject: [elpa] master 87be44e: Fix debbugs-gnu-announce-commit for ELPA bugs
Date: Sat, 27 Apr 2019 10:46:35 -0400 (EDT)

branch: master
commit 87be44e0f524816c7d70de34fdc196df7c1ec917
Author: Noam Postavsky <address@hidden>
Commit: Noam Postavsky <address@hidden>

    Fix debbugs-gnu-announce-commit for ELPA bugs
    
    * packages/debbugs/debbugs-gnu.el (debbugs-gnu-make-control-message):
    Add NOVERSION parameter.
    (debbugs-gnu-announce-commit): Use it when no emacs-version is found
    in the repository where the commit came from.
---
 packages/debbugs/debbugs-gnu.el | 29 ++++++++++++++++-------------
 1 file changed, 16 insertions(+), 13 deletions(-)

diff --git a/packages/debbugs/debbugs-gnu.el b/packages/debbugs/debbugs-gnu.el
index 5020d96..c2309a8 100644
--- a/packages/debbugs/debbugs-gnu.el
+++ b/packages/debbugs/debbugs-gnu.el
@@ -1572,12 +1572,16 @@ removed instead."
                                       (string-to-number
                                       (match-string 1 addr)))))))))))))
 
-(defun debbugs-gnu-make-control-message (message bugid &optional reverse 
buffer)
+(defun debbugs-gnu-make-control-message
+    (message bugid &optional reverse buffer noversion)
   "Make a control message for the current bug report.
 The message is inserted into BUFFER, and mail headers are adjust
 so that it will be sent to address@hidden (via Bcc if
 there is already a To address).  If BUFFER omitted, create and
-display a new buffer.
+display a new buffer.  If optional NOVERSION is non-nil, suppress
+query for version number on \"close\", \"fixed\", etc messages.
+Otherwise, the version is queried for bugs whose package is
+\"emacs\".
 
 When called interactively, choose the current buffer if it is in
 `message-mode', or create a new buffer otherwise.
@@ -1609,6 +1613,7 @@ removed instead."
                      (car (debbugs-get-status bugid))))
          (version
           (if (and
+               (not noversion)
                (member message '("close" "done"
                                  "fixed" "notfixed" "found" "notfound"))
                (member "emacs" (cdr (assq 'package status))))
@@ -1618,8 +1623,7 @@ removed instead."
                  (pcase (nbutlast (version-to-list emacs-version)
                                   ;; Chop off build number, if needed.
                                   (if (boundp 'emacs-build-number)
-                                      0
-                                    1))
+                                      0 1))
                    (`(,major ,minor ,_micro) ; Development version.
                     (format "%d.%d" major
                             (if (member
@@ -1860,17 +1864,16 @@ Optionally call `debbugs-gnu-make-control-message' to 
close BUGNUM."
      commit-range)
     (when (y-or-n-p "Close bug? ")
       (let ((emacs-version
-             (or (and (member "emacs" packages)
-                      (file-exists-p "configure.ac")
-                      (with-temp-buffer
-                        (insert-file-contents "configure.ac")
-                        (and (re-search-forward "\
+             (and (member "emacs" packages)
+                  (file-exists-p "configure.ac")
+                  (with-temp-buffer
+                    (insert-file-contents "configure.ac")
+                    (and (re-search-forward "\
 ^ *AC_INIT(GNU Emacs, *\\([0-9.]+\\), address@hidden"
-                                                nil t)
-                             (match-string 1))))
-                 "")))
+                                            nil t)
+                         (match-string 1))))))
         (debbugs-gnu-make-control-message
-         "done" bugnum nil (current-buffer))))))
+         "done" bugnum nil (current-buffer) (not emacs-version))))))
 
 (defun debbugs-gnu-post-patch (commit-range bugnum &optional format-patch-args)
   "Attach COMMIT-RANGE as patches into current message.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]