emacs: poll: return useful errors when poll fails.
authorMark Walters <markwalters1009@gmail.com>
Wed, 28 Oct 2015 18:28:39 +0000 (18:28 +0000)
committerDavid Bremner <david@tethera.net>
Mon, 23 Nov 2015 12:37:19 +0000 (08:37 -0400)
Previously poll called from emacs would fail silently. This makes it
return a useful error message.

In the non-deprecated case of notmuch new and appropriate hooks, it
uses notmuch-call-notmuch-process which gives an error and
additionally puts the stdout/stderr etc in the *Notmuch errors*
buffer.

In the deprecated case of a custom poll script it only returns an
error message.

Commit based on a bug report, and a potential fix, by Ketil Malde.

emacs/notmuch-lib.el

index 201d7ec8593f376c1ae9f20a3671eff52535e4bc..dedcbeb865562903a2595e0a5883ef0e01a2d8c4 100644 (file)
@@ -240,8 +240,9 @@ depending on the value of `notmuch-poll-script'."
   (interactive)
   (if (stringp notmuch-poll-script)
       (unless (string= notmuch-poll-script "")
-       (call-process notmuch-poll-script nil nil))
-    (call-process notmuch-command nil nil nil "new")))
+       (unless (equal (call-process notmuch-poll-script nil nil) 0)
+         (error "Notmuch: poll script `%s' failed!" notmuch-poll-script)))
+    (notmuch-call-notmuch-process "new")))
 
 (defun notmuch-bury-or-kill-this-buffer ()
   "Undisplay the current buffer.