From: Katsumi Yamaoka <yamaoka@jpl.org>
To: ivan@kanis.fr
Cc: 23069@debbugs.gnu.org
Subject: bug#23069: 25.1.50; in Gnus temporary browse files are always deleted
Date: Wed, 13 Jul 2016 17:42:39 +0900 [thread overview]
Message-ID: <b4mpoqi3p4w.fsf@jpl.org> (raw)
In-Reply-To: <87bn69uog6.fsf@kanis.fr>
On Mon, 11 Jul 2016 20:58:11 +0200, Ivan Kanis <ivan@kanis.fr> wrote:
> I have read again the documentation of gnus-article-browse-delete-temp.
> Here is the bit that interests me:
> "If nil, don't delete temporary files"
> I understand that if the variable is set to nil the temporary files
> should not be deleted.
I can understand it so, too.
> I am trying to keep those files so I am guessing the bit you quoted
> Reiner Steib will not help me.
;;; gnus-art.el ;;;
(defun gnus-article-browse-html-parts (list &optional header)
[...]
(add-hook 'gnus-summary-prepare-exit-hook
'gnus-article-browse-delete-temp-files)
(add-hook 'gnus-exit-gnus-hook
(lambda ()
(gnus-article-browse-delete-temp-files t)))
These two hooks are what Reiner added over a decade ago, and
the last `t' forces it to delete temp files regardless of
`gnus-article-browse-delete-temp'. I think there are three
ways to fix this issue:
1. Add a note, like the following, to the documentation of
`gnus-article-browse-delete-temp':
Note that temporary files will be deleted when exiting Gnus
no matter what the value of this variable is.
2. Don't pass `t' to `gnus-article-browse-delete-temp-files'
that `gnus-exit-gnus-hook' runs.
3. Don't add `gnus-article-browse-delete-temp-files' to
`gnus-exit-gnus-hook'.
I vote for 3.
`gnus-summary-prepare-exit-hook' is sufficient for those who
want to delete temp files automatically, I think.
WDYT?
next prev parent reply other threads:[~2016-07-13 8:42 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-20 16:03 bug#23069: 25.1.50; in Gnus temporary browse files are always deleted Ivan Kanis
2016-03-22 2:42 ` Katsumi Yamaoka
2016-07-11 18:58 ` Ivan Kanis
2016-07-13 8:42 ` Katsumi Yamaoka [this message]
2016-07-13 14:14 ` Ted Zlatanov
2016-07-13 23:27 ` Katsumi Yamaoka
2016-08-25 9:57 ` Ivan Kanis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b4mpoqi3p4w.fsf@jpl.org \
--to=yamaoka@jpl.org \
--cc=23069@debbugs.gnu.org \
--cc=ivan@kanis.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).