From debbugs-submit-bounces@debbugs.gnu.org Sat Jun 29 08:57:56 2019 Received: (at submit) by debbugs.gnu.org; 29 Jun 2019 12:57:56 +0000 Received: from localhost ([127.0.0.1]:43587 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hhCvU-0006tj-2y for submit@debbugs.gnu.org; Sat, 29 Jun 2019 08:57:56 -0400 Received: from andalucia.tim-landscheidt.de ([116.203.78.250]:33386 helo=andalucia) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hhCvS-0006tZ-R2 for submit@debbugs.gnu.org; Sat, 29 Jun 2019 08:57:55 -0400 Received: from dslb-090-186-216-122.090.186.pools.vodafone-ip.de ([90.186.216.122]:33428 helo=passepartout.tim-landscheidt.de) by andalucia with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hhCvM-0000px-8B for submit@debbugs.gnu.org; Sat, 29 Jun 2019 14:57:48 +0200 From: Tim Landscheidt To: submit@debbugs.gnu.org (The Gnus Bugfixing Girls + Boys) Subject: gnus-read-ephemeral-bug-group does not handle errors, leaving temporary files behind Organization: http://www.tim-landscheidt.de/ X-Debbugs-Version: 5.13 X-Debbugs-Package: emacs,gnus Date: Sat, 29 Jun 2019 12:57:47 +0000 Message-ID: <87ef3ca6r8.fsf@passepartout.tim-landscheidt.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Score: 3.7 (+++) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: With Emacs 26.1, M-x gnus-read-ephemeral-bug-group RET foobar RET gives "gnus-list-of-unread-articles: Group nndoc+ephemeral:bug#0 couldn’t be activated", while leaving a temporary file with the con [...] Content analysis details: (3.7 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 0.0 URIBL_BLOCKED ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [URIs: gnu.org] 0.0 FSL_HELO_NON_FQDN_1 No description available. -0.0 SPF_PASS SPF: sender matches SPF record 1.5 RCVD_IN_SORBS_WEB RBL: SORBS: sender is an abusable web server [116.203.78.250 listed in dnsbl.sorbs.net] 2.2 HELO_NO_DOMAIN Relay reports its domain incorrectly X-Debbugs-Envelope-To: submit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: 2.7 (++) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: With Emacs 26.1, M-x gnus-read-ephemeral-bug-group RET foobar RET gives "gnus-list-of-unread-articles: Group nndoc+ephemeral:bug#0 couldn’t be activated", while leaving a temporary file with the con [...] Content analysis details: (2.7 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 0.0 URIBL_BLOCKED ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [URIs: gnu.org] 1.5 RCVD_IN_SORBS_WEB RBL: SORBS: sender is an abusable web server [116.203.78.250 listed in dnsbl.sorbs.net] 0.0 FSL_HELO_NON_FQDN_1 No description available. -0.0 SPF_PASS SPF: sender matches SPF record 2.2 HELO_NO_DOMAIN Relay reports its domain incorrectly -1.0 MAILING_LIST_MULTI Multiple indicators imply a widely-seen list manager With Emacs 26.1, M-x gnus-read-ephemeral-bug-group RET foobar RET gives "gnus-list-of-unread-articles: Group nndoc+ephemeral:bug#0 couldn=E2=80=99t be activated", while leaving a temporary file with the contents: | | Error | | An error occurred. | Error was: No bug number | behind in /tmp. This also happens with gnus-read-ephemeral-emacs-bug-group and gnus-read-ephemeral-debian-bug-group. For GNU bugs, fixing this might be made easier by updating debbugs.gnu.org as it currently returns 200 OK for non-ex- isting bugs: | [tim@passepartout ~]$ curl -is 'https://debbugs.gnu.org/cgi/bugreport.cgi= ?bug=3Dfoobar;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 | HTTP/1.1 200 OK | [tim@passepartout ~]$ while bugs.debian.org gives a status code that can be used for diagnostics: | [tim@passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugreport= .cgi?bug=3D595256;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 | HTTP/1.1 200 OK | [tim@passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugreport= .cgi?bug=3D4711;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 | HTTP/1.1 404 Not Found | [tim@passepartout ~]$ From debbugs-submit-bounces@debbugs.gnu.org Sat Jun 29 22:47:20 2019 Received: (at 36433) by debbugs.gnu.org; 30 Jun 2019 02:47:20 +0000 Received: from localhost ([127.0.0.1]:45142 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hhPs7-000882-K0 for submit@debbugs.gnu.org; Sat, 29 Jun 2019 22:47:20 -0400 Received: from mail-ed1-f68.google.com ([209.85.208.68]:34472) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hhPs3-00087e-JJ for 36433@debbugs.gnu.org; Sat, 29 Jun 2019 22:47:17 -0400 Received: by mail-ed1-f68.google.com with SMTP id s49so17305357edb.1 for <36433@debbugs.gnu.org>; Sat, 29 Jun 2019 19:47:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tcd-ie.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version; bh=TtNLPuU8i64fm5+F6o6JbaMW4j2hyx/gUopuP3bMPNw=; b=LF8fVcIjB7m7A4hg5teIQO+2N0EKVVMFk5NpjZ182W1NeOE6eaL85dG+O+BllyauaA OD8LNIepKuWEgYHK5opS0LU7laK+fpXuXy4oixHJvo5tcK4eIORexpIm7BCXabGiNSHT YsA6kt8yU59oKonztYcH47RzLjfWnyVa6qwF+m82N2i77iErDhz2/WdRWj5S+T0GORMg AEiSK6Gt5gBwi1oqH94vb9lQtAaRqxETWCP8YwRreRQ34ENLbm48H0T/1SryaQVEzlRq EvMt125uk3R5H4XCjVAzhwt3+mMHsXXP2l+a/kyDtP72DCBSyR3ieQDvonEre7VyYJLl 4MXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:references:date:in-reply-to :message-id:user-agent:mime-version; bh=TtNLPuU8i64fm5+F6o6JbaMW4j2hyx/gUopuP3bMPNw=; b=fQ4xJsTOeZu6cvEmuwow3tZsnftvRqA8W8RY9bslrOSk+UyiLQeosCYhQb9U9Teohg NO8c6E0+YiLrJ20vmcnTtEM1hmhEddgWM1GIEdJp7pvlQBkOc94WengFhmsd0jl8jA36 kWfxDT2zyQEoZ6Gr5EsnTgy/E9ckFDAji7ZzezdkG9C4Z0MYDxJMWPrr9wiBFno9qO6O +k6tQx41394GLv9zsNH4HMDu/CpX9femqDGWpN56TyXP/ngwPfT1tDpJpoeAqIcNNlZj uOAgLwbrCH7xmzlxose/XimiJPu6cFy2V5fe/ZpeI+NFPpG96bAH6Go4SDxek0n4t1fd csGQ== X-Gm-Message-State: APjAAAWoHdvhHL5vPcGzw9oPwkO03I8VktGzKcQk8WRsX21I16fNa9YC ewOB7/0k/pvYmkueuowmQ7RNXQ== X-Google-Smtp-Source: APXvYqwvD5Ww8rHLSpr48Ujh7mqlUn7ydKIZk8aGYrUkLROAG7oErZ3ehbbliCS6KbH1xf6u+MSc1A== X-Received: by 2002:a17:906:9256:: with SMTP id c22mr16184330ejx.172.1561862829477; Sat, 29 Jun 2019 19:47:09 -0700 (PDT) Received: from localhost ([2a02:8084:20e2:c380:92bd:1bfd:38fc:fae2]) by smtp.gmail.com with ESMTPSA id i26sm1319529eja.14.2019.06.29.19.47.08 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Sat, 29 Jun 2019 19:47:08 -0700 (PDT) From: "Basil L. Contovounesios" To: Tim Landscheidt Subject: Re: bug#36433: gnus-read-ephemeral-bug-group does not handle errors, leaving temporary files behind References: <87ef3ca6r8.fsf@passepartout.tim-landscheidt.de> Date: Sun, 30 Jun 2019 03:47:02 +0100 In-Reply-To: <87ef3ca6r8.fsf@passepartout.tim-landscheidt.de> (Tim Landscheidt's message of "Sat, 29 Jun 2019 12:57:47 +0000") Message-ID: <878stj4wnt.fsf@tcd.ie> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: 36433 Cc: 36433@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) --=-=-= Content-Type: text/plain tags 36433 + patch quit --=-=-= Content-Type: text/x-diff Content-Disposition: attachment; filename=0001-Improve-Gnus-ephemeral-bug-group-browsing.patch >From e7988593af6f424df64bafb6d8948f2b75aa7ba7 Mon Sep 17 00:00:00 2001 From: "Basil L. Contovounesios" Date: Sat, 29 Jun 2019 22:26:56 +0100 Subject: [PATCH] Improve Gnus ephemeral bug group browsing * doc/misc/gnus.texi (Foreign Groups): Update description of gnus-read-ephemeral-emacs-bug-group for multiple bug IDs (bug#11961). * lisp/gnus/gnus-group.el (gnus-bug-group-download-format-alist): Use HTTPS for Debian's bug tracker. (gnus-group--read-bug-ids): New function for reading multiple bug IDs in the minibuffer. Improves on previous brittle approach of word-at-point -> read-string -> string-to-number by a) defaulting to the more accurate bug-reference-bug-regexp or number-at-point without using the intrusive INITIAL-INPUT argument, and b) not attempting to parse bug IDs. (gnus-read-ephemeral-bug-group): Use it. Extend docstring and commentary. Fix handling of multiple bug IDs as either numbers or strings. Hoist some string consing out of inner loop. Delete temporary file even on error. Throw more informative error when url-insert-file-contents successfully returns an error (bug#36433). (gnus-read-ephemeral-debian-bug-group): Use gnus-group--read-bug-ids and fix docstring for multiple bug IDs. Accept optional WINDOW-CONF like other ephemeral bug group commands. (gnus-read-ephemeral-emacs-bug-group): Use gnus-group--read-bug-ids and fix string/numeric ID conversions. Try loading debbugs-gnu before testing for debbugs-gnu-summary-mode. --- doc/misc/gnus.texi | 7 +- lisp/gnus/gnus-group.el | 162 +++++++++++++++++++++++----------------- 2 files changed, 98 insertions(+), 71 deletions(-) diff --git a/doc/misc/gnus.texi b/doc/misc/gnus.texi index 28f000c489..5e41ff2e4c 100644 --- a/doc/misc/gnus.texi +++ b/doc/misc/gnus.texi @@ -2716,9 +2716,10 @@ Foreign Groups @item gnus-read-ephemeral-emacs-bug-group @findex gnus-read-ephemeral-emacs-bug-group -Read an Emacs bug report in an ephemeral group. Gnus will prompt for a -bug number. The default is the number at point. The @acronym{URL} is -specified in @code{gnus-bug-group-download-format-alist}. +Read an Emacs bug report in an ephemeral group. Gnus will prompt for +multiple bug numbers. The default is the number at point. The +@acronym{URL} template is specified in +@code{gnus-bug-group-download-format-alist}. @item gnus-read-ephemeral-debian-bug-group @findex gnus-read-ephemeral-debian-bug-group diff --git a/lisp/gnus/gnus-group.el b/lisp/gnus/gnus-group.el index f49ed16443..ea695aaa97 100644 --- a/lisp/gnus/gnus-group.el +++ b/lisp/gnus/gnus-group.el @@ -2423,7 +2423,7 @@ gnus-read-ephemeral-gmane-group-url (defcustom gnus-bug-group-download-format-alist '((emacs . "https://debbugs.gnu.org/cgi/bugreport.cgi?bug=%s;mboxmaint=yes;mboxstat=yes") (debian - . "http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=%s&mbox=yes;mboxmaint=yes")) + . "https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=%s&mbox=yes;mboxmaint=yes")) "Alist of symbols for bug trackers and the corresponding URL format string. The URL format string must contain a single \"%s\", specifying the bug number, and browsing the URL must return mbox output." @@ -2434,90 +2434,116 @@ gnus-bug-group-download-format-alist :version "24.1" :type '(repeat (cons (symbol) (string :tag "URL format string")))) +(autoload 'thing-at-point-looking-at "thingatpt") +(defvar bug-reference-bug-regexp) + +(defun gnus-group--read-bug-ids () + "Return a list of bug IDs read in the minibuffer." + (require 'bug-reference) + (let ((def (cond ((thing-at-point-looking-at bug-reference-bug-regexp 500) + (match-string 2)) + ((number-at-point))))) + ;; Pass DEF as the value of COLLECTION instead of DEF because: + ;; a) null input should not cause DEF to be returned and + ;; b) TAB and M-n still work this way. + (completing-read-multiple "Bug IDs: " (and def (list (format "%s" def)))))) + (defun gnus-read-ephemeral-bug-group (ids mbox-url &optional window-conf) - "Browse bug NUMBER as ephemeral group." - (interactive (list (read-string "Enter bug number: " - (thing-at-point 'word) nil) - ;; FIXME: Add completing-read from - ;; `gnus-emacs-bug-group-download-format' ... - (cdr (assoc 'emacs gnus-bug-group-download-format-alist)))) - (when (stringp ids) - (setq ids (string-to-number ids))) - (unless (listp ids) - (setq ids (list ids))) + "Browse bug reports with IDS in an ephemeral group. +IDS can be either a single bug ID (a number or string), or a list +thereof. MBOX-URL is a URL format string identifying the bug +tracker; see `gnus-bug-group-download-format-alist' for details. +Interactively, read multiple bug IDS in the minibuffer and +default to the MBOX-URL for the Emacs bug tracker. WINDOW-CONF +is the name of the Gnus window configuration to use when exiting +the ephemeral group." + (interactive + (list (gnus-group--read-bug-ids) + (alist-get 'emacs gnus-bug-group-download-format-alist))) + (or ids (user-error "No bug IDs specified")) + (setq ids (mapcar (lambda (id) (format "%s" id)) + (if (consp ids) ids (list ids)))) (let ((tmpfile (make-temp-file "gnus-temp-group-"))) - (let ((coding-system-for-write 'binary) - (coding-system-for-read 'binary)) - (with-temp-file tmpfile - (mm-disable-multibyte) - (dolist (id ids) - (let ((file (format "~/.emacs.d/debbugs-cache/%s" id))) - (if (and (not gnus-plugged) - (file-exists-p file)) - (insert-file-contents file) - (url-insert-file-contents (format mbox-url id) t)))) + (unwind-protect ;; Add the debbugs address so that we can respond to reports easily. - (let ((address - (format "%s@%s" (car ids) - (url-host (url-generic-parse-url mbox-url))))) - (goto-char (point-min)) - (while (re-search-forward (concat "^" message-unix-mail-delimiter) - nil t) - (narrow-to-region (point) - (if (search-forward "\n\n" nil t) - (1- (point)) - (point-max))) - (unless (string-match (concat "\\(?:\\`\\|[ ,<]\\)" - (regexp-quote address) - "\\(?:\\'\\|[ ,>]\\)") - (concat (message-fetch-field "to") " " - (message-fetch-field "cc"))) + (let* ((address (format "%s@%s" (car ids) + (url-host (url-generic-parse-url mbox-url)))) + (address-re (concat "\\(?:\\`\\|[ ,<]\\)" + (regexp-quote address) + "\\(?:\\'\\|[ ,>]\\)")) + (delim (concat "^" message-unix-mail-delimiter))) + (let ((coding-system-for-write 'binary) + (coding-system-for-read 'binary)) + (with-temp-file tmpfile + (mm-disable-multibyte) + (dolist (id ids) + (let ((file (concat "~/.emacs.d/debbugs-cache/" id))) + (if (and (not gnus-plugged) + (file-exists-p file)) + (insert-file-contents file) + ;; Pass non-nil VISIT to avoid errors with non-nil + ;; `url-automatic-caching' (bug#26063, bug#29008) + ;; and immediately unvisit. + ;; FIXME: This masks real errors! + (url-insert-file-contents (format mbox-url id) t) + (setq buffer-file-name nil)))) (goto-char (point-min)) - (if (re-search-forward "^To:" nil t) - (progn + ;; Throw an informative error early instead of passing nonsense + ;; to `gnus-group-read-ephemeral-group' (bug#36433). + (unless (save-excursion (re-search-forward delim nil t)) + (error "Invalid mbox format for bug IDs: %s" + (string-join ids ", "))) + (while (re-search-forward delim nil t) + (narrow-to-region (point) + (if (search-forward "\n\n" nil t) + (1- (point)) + (point-max))) + (unless (string-match-p address-re + (concat (message-fetch-field "to") " " + (message-fetch-field "cc"))) + (goto-char (point-min)) + (if (not (re-search-forward "^To:" nil t)) + (insert "To: " address "\n") (message-next-header) (skip-chars-backward "\t\n ") - (insert ", " address)) - (insert "To: " address "\n"))) - (goto-char (point-max)) - (widen))) - ;; `url-insert-file-contents' sets this because of the 2nd arg. - (setq buffer-file-name nil))) - (gnus-group-read-ephemeral-group - (format "nndoc+ephemeral:bug#%s" - (mapconcat 'number-to-string ids ",")) - `(nndoc ,tmpfile - (nndoc-article-type mbox)) - nil window-conf) - (delete-file tmpfile))) + (insert ", " address))) + (goto-char (point-max)) + (widen)))) + (gnus-group-read-ephemeral-group + (concat "nndoc+ephemeral:bug#" (string-join ids ",")) + `(nndoc ,tmpfile + (nndoc-article-type mbox)) + nil window-conf)) + (delete-file tmpfile)))) -(defun gnus-read-ephemeral-debian-bug-group (number) - "Browse Debian bug NUMBER as ephemeral group." - (interactive (list (read-string "Enter bug number: " - (thing-at-point 'word) nil))) +(defun gnus-read-ephemeral-debian-bug-group (ids &optional window-conf) + "Browse Debian bug reports with IDS in an ephemeral group. +The arguments have the same meaning as those of +`gnus-read-ephemeral-bug-group', which see." + (interactive (list (gnus-group--read-bug-ids))) (gnus-read-ephemeral-bug-group - number - (cdr (assoc 'debian gnus-bug-group-download-format-alist)))) + ids + (alist-get 'debian gnus-bug-group-download-format-alist) + window-conf)) (defvar debbugs-gnu-bug-number) ; debbugs-gnu (defun gnus-read-ephemeral-emacs-bug-group (ids &optional window-conf) - "Browse Emacs bugs IDS as an ephemeral group." - (interactive (list (string-to-number - (read-string "Enter bug number: " - (thing-at-point 'word) nil)))) - (when (stringp ids) - (setq ids (string-to-number ids))) - (unless (listp ids) - (setq ids (list ids))) + "Browse Emacs bug reports with IDS in an ephemeral group. +The arguments have the same meaning as those of +`gnus-read-ephemeral-bug-group', which see." + (interactive (list (gnus-group--read-bug-ids))) (gnus-read-ephemeral-bug-group ids - (cdr (assoc 'emacs gnus-bug-group-download-format-alist)) + (alist-get 'emacs gnus-bug-group-download-format-alist) window-conf) - (when (fboundp 'debbugs-gnu-summary-mode) + (when (and (require 'debbugs-gnu nil t) + (fboundp 'debbugs-gnu-summary-mode)) (with-current-buffer (window-buffer (selected-window)) (debbugs-gnu-summary-mode 1) - (set (make-local-variable 'debbugs-gnu-bug-number) (car ids))))) + (let ((id (or (car-safe ids) ids))) + (if (stringp id) (setq id (string-to-number id))) + (setq-local debbugs-gnu-bug-number id))))) (defun gnus-group-jump-to-group (group &optional prompt) "Jump to newsgroup GROUP. -- 2.20.1 --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Tim Landscheidt writes: > With Emacs 26.1, M-x gnus-read-ephemeral-bug-group RET > foobar RET gives "gnus-list-of-unread-articles: Group > nndoc+ephemeral:bug#0 couldn=E2=80=99t be activated", while leaving > a temporary file with the contents: > > | > | Error > | > | An error occurred. > | Error was: No bug number > | > > behind in /tmp. > > This also happens with gnus-read-ephemeral-emacs-bug-group > and gnus-read-ephemeral-debian-bug-group. The attached patch fixes this for Emacs 27. It also improves the reading and handling of multiple string/numeric bug IDs, which is currently quite brittle, and extends some of the relevant documentation. WDYT? > For GNU bugs, fixing this might be made easier by updating > debbugs.gnu.org as it currently returns 200 OK for non-ex- > isting bugs: > > | [tim@passepartout ~]$ curl -is 'https://debbugs.gnu.org/cgi/bugreport.c= gi?bug=3Dfoobar;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 > | HTTP/1.1 200 OK > | [tim@passepartout ~]$ > > while bugs.debian.org gives a status code that can be used > for diagnostics: > > | [tim@passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugrepo= rt.cgi?bug=3D595256;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 > | HTTP/1.1 200 OK > | [tim@passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugrepo= rt.cgi?bug=3D4711;mboxmaint=3Dyes;mboxstat=3Dyes' | head -1 > | HTTP/1.1 404 Not Found > | [tim@passepartout ~]$ I think the attached patch is as good as gnus-read-ephemeral-bug-group can do for now, without fixing several known deficiencies of the url.el library and debbugs.gnu.org instance (each of which deserves its own bug ticket if it doesn't already have one). Thanks, --=20 Basil --=-=-=-- From debbugs-submit-bounces@debbugs.gnu.org Sat Jul 06 10:33:27 2019 Received: (at 36433) by debbugs.gnu.org; 6 Jul 2019 14:33:27 +0000 Received: from localhost ([127.0.0.1]:55700 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hjlkk-0002TH-Ui for submit@debbugs.gnu.org; Sat, 06 Jul 2019 10:33:27 -0400 Received: from quimby.gnus.org ([80.91.231.51]:53480) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hjlki-0002T6-43 for 36433@debbugs.gnu.org; Sat, 06 Jul 2019 10:33:25 -0400 Received: from cm-84.212.202.86.getinternet.no ([84.212.202.86] helo=marnie) by quimby.gnus.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hjlkb-0005KP-8X; Sat, 06 Jul 2019 16:33:19 +0200 From: Lars Ingebrigtsen To: "Basil L. Contovounesios" Subject: Re: bug#36433: gnus-read-ephemeral-bug-group does not handle errors, leaving temporary files behind References: <87ef3ca6r8.fsf@passepartout.tim-landscheidt.de> <878stj4wnt.fsf@tcd.ie> Date: Sat, 06 Jul 2019 16:33:16 +0200 In-Reply-To: <878stj4wnt.fsf@tcd.ie> (Basil L. Contovounesios's message of "Sun, 30 Jun 2019 03:47:02 +0100") Message-ID: <87lfxbdyhf.fsf@mouse.gnus.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Report: Spam detection software, running on the system "quimby.gnus.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see @@CONTACT_ADDRESS@@ for details. Content preview: "Basil L. Contovounesios" writes: > The attached patch fixes this for Emacs 27. It also improves the > reading and handling of multiple string/numeric bug IDs, which is > currently quite brittle, and extends some of the relevant docum [...] Content analysis details: (-2.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: 36433 Cc: 36433@debbugs.gnu.org, Tim Landscheidt X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) "Basil L. Contovounesios" writes: > The attached patch fixes this for Emacs 27. It also improves the > reading and handling of multiple string/numeric bug IDs, which is > currently quite brittle, and extends some of the relevant documentation. > WDYT? Looks good to me, and it works fine, so I've applied it to the trunk. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no From debbugs-submit-bounces@debbugs.gnu.org Sat Jul 06 10:33:34 2019 Received: (at control) by debbugs.gnu.org; 6 Jul 2019 14:33:34 +0000 Received: from localhost ([127.0.0.1]:55703 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hjlks-0002Tc-7y for submit@debbugs.gnu.org; Sat, 06 Jul 2019 10:33:34 -0400 Received: from quimby.gnus.org ([80.91.231.51]:53496) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hjlkq-0002TT-Mv for control@debbugs.gnu.org; Sat, 06 Jul 2019 10:33:33 -0400 Received: from cm-84.212.202.86.getinternet.no ([84.212.202.86] helo=marnie) by quimby.gnus.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1hjlko-0005KX-4X for control@debbugs.gnu.org; Sat, 06 Jul 2019 16:33:32 +0200 Date: Sat, 06 Jul 2019 16:33:29 +0200 Message-Id: <87k1cvdyh2.fsf@mouse.gnus.org> To: control@debbugs.gnu.org From: Lars Ingebrigtsen Subject: control message for bug #36433 X-Spam-Report: Spam detection software, running on the system "quimby.gnus.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see @@CONTACT_ADDRESS@@ for details. Content preview: tags 36433 fixed close 36433 27.1 quit Content analysis details: (-2.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: control X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) tags 36433 fixed close 36433 27.1 quit From unknown Wed Jun 18 00:21:57 2025 Received: (at fakecontrol) by fakecontrolmessage; To: internal_control@debbugs.gnu.org From: Debbugs Internal Request Subject: Internal Control Message-Id: bug archived. Date: Sun, 04 Aug 2019 11:24:05 +0000 User-Agent: Fakemail v42.6.9 # This is a fake control message. # # The action: # bug archived. thanks # This fakemail brought to you by your local debbugs # administrator