discussion and development of piem
 help / color / mirror / Atom feed
* [PATCH] b4: Remove temporary directory on "missing mbox" error
@ 2020-10-25 19:21 Kyle Meyer
  0 siblings, 0 replies; only message in thread
From: Kyle Meyer @ 2020-10-25 19:21 UTC (permalink / raw)
  To: piem

As of 30defdb (b4: Clean up temporary directories by default,
2020-09-27), piem-b4-am-from-mid is supposed to clean up its temporary
directory unless piem-b4-keep-temp-directory is non-nil.  That commit,
however, missed an error case where the cleanup function needs to be
triggered.
---
 piem-b4.el | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/piem-b4.el b/piem-b4.el
index 31bf6f1..27ecf19 100644
--- a/piem-b4.el
+++ b/piem-b4.el
@@ -101,6 +101,8 @@ (defun piem-b4--get-am-files (mid coderepo args)
                  mbox-cover)
             (if (file-exists-p mbox-am)
                 mbox-am
+              (when clean-fn
+                (funcall clean-fn))
               (error "Expected mbox file does not exist: %s" mbox-am))
             clean-fn))))
 

base-commit: 442472a37da221c7c5f961870a2b6b53b1c18e81
-- 
2.28.0


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-10-25 19:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-25 19:21 [PATCH] b4: Remove temporary directory on "missing mbox" error Kyle Meyer

discussion and development of piem

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.kyleam.com/piem/0 piem/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 piem piem/ https://inbox.kyleam.com/piem \
		piem@inbox.kyleam.com
	public-inbox-index piem

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.emacs.piem


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git