discussion and development of piem
 help / color / mirror / code / 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 related	[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

Code repositories for project(s) associated with this public inbox

	https://git.kyleam.com/piem/

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).