diff options
author | Austin Clements <amdragon@MIT.EDU> | 2012-12-15 15:04:17 -0500 |
---|---|---|
committer | David Bremner <bremner@debian.org> | 2012-12-16 17:04:08 -0400 |
commit | e1d5e881568afd6bcdc145a7f035e207f8ff3884 (patch) | |
tree | 43db3f39738d10156624d23f4df74f0af3b08a59 /emacs/notmuch-lib.el | |
parent | 66c935cff37fcab1ef7b0a5cecf5bc1c62d8d2ed (diff) |
emacs: Improve error handling for notmuch-call-notmuch-json
This checks for non-zero exit status from JSON CLI calls and pops up
an error buffer with stderr and stdout. A consequence of this is that
show and reply now handle errors, rather than ignoring them.
Diffstat (limited to 'emacs/notmuch-lib.el')
-rw-r--r-- | emacs/notmuch-lib.el | 22 |
1 files changed, 15 insertions, 7 deletions
diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el index c3d76d22..dd2c78ba 100644 --- a/emacs/notmuch-lib.el +++ b/emacs/notmuch-lib.el @@ -375,15 +375,23 @@ contents of ERR-FILE will be included in the error message." "Invoke `notmuch-command' with `args' and return the parsed JSON output. The returned output will represent objects using property lists -and arrays as lists." +and arrays as lists. If notmuch exits with a non-zero status, +this will pop up a buffer containing notmuch's output and signal +an error." (with-temp-buffer - (apply #'call-process notmuch-command nil (list t nil) nil args) - (goto-char (point-min)) - (let ((json-object-type 'plist) - (json-array-type 'list) - (json-false 'nil)) - (json-read)))) + (let ((err-file (make-temp-file "nmerr"))) + (unwind-protect + (let ((status (apply #'call-process + notmuch-command nil (list t err-file) nil args))) + (notmuch-check-exit-status status (cons notmuch-command args) + (buffer-string) err-file) + (goto-char (point-min)) + (let ((json-object-type 'plist) + (json-array-type 'list) + (json-false 'nil)) + (json-read))) + (delete-file err-file))))) ;; Compatibility functions for versions of emacs before emacs 23. ;; |