Re: [PATCH] emacs: don't start processes stopped

2020-01-03 Thread David Edmondson
On Friday, 2020-01-03 at 09:04:00 -08, Steven Allen wrote:

> It causes this function to fail with:
>
> let: Wrong type argument: null, t
>
> Support for this was removed from Emacs in April
> 2019 (5c5e309527e6b582e2c04b83e7af45f3144863ac) because it never
> worked correctly (apparently).
>
> This also shouldn't be necessary as sentinels will not be called
> unless emacs is idle or waiting for input. Therefore, the
> `process-put' calls immediately following the `make-process' call
> should always complete before the sentinel is first called.

Reviewed-by: David Edmondson 

> ---
>  emacs/notmuch-crypto.el | 9 ++---
>  1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/emacs/notmuch-crypto.el b/emacs/notmuch-crypto.el
> index f4585d5e..4035ee37 100644
> --- a/emacs/notmuch-crypto.el
> +++ b/emacs/notmuch-crypto.el
> @@ -222,16 +222,11 @@ corresponding key when the status button is pressed."
>:buffer buffer
>:stderr buffer
>:command (list notmuch-crypto-gpg-program 
> "--recv-keys" keyid)
> -  :sentinel #'notmuch-crypto--async-key-sentinel
> -  ;; Create the process stopped so that
> -  ;; we have time to store the key id,
> -  ;; etc. on it.
> -  :stop t)))
> +  :sentinel 
> #'notmuch-crypto--async-key-sentinel)))
>   (process-put p :gpg-key-id keyid)
>   (process-put p :notmuch-show-buffer (current-buffer))
>   (process-put p :notmuch-show-point (point))
> - (message "Getting the GPG key %s asynchronously..." keyid)
> - (continue-process p)))
> + (message "Getting the GPG key %s asynchronously..." keyid)))
>  
>(let ((window (display-buffer buffer)))
>   (with-selected-window window
> -- 
> 2.24.1
>
> ___
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

dme.
-- 
We can dance if we want to, we can leave your friends behind.
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


[wish] notmuch-emacs: allow operation on marked messages in notmuch-tree-mode

2020-01-03 Thread Gregor Zattler
Dear notmuch developers,

it would be helpful if it was possible to mark individual
messages in notmuch-tree-mode and later apply a command to all
marked messages (UI as in dired).

I'm used to this for managing emails in mutt.


Ciao; Gregor
--
 -... --- .-. . -.. ..--.. ...-.-

___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


[PATCH] emacs: don't start processes stopped

2020-01-03 Thread Steven Allen
It causes this function to fail with:

let: Wrong type argument: null, t

Support for this was removed from Emacs in April
2019 (5c5e309527e6b582e2c04b83e7af45f3144863ac) because it never
worked correctly (apparently).

This also shouldn't be necessary as sentinels will not be called
unless emacs is idle or waiting for input. Therefore, the
`process-put' calls immediately following the `make-process' call
should always complete before the sentinel is first called.
---
 emacs/notmuch-crypto.el | 9 ++---
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/emacs/notmuch-crypto.el b/emacs/notmuch-crypto.el
index f4585d5e..4035ee37 100644
--- a/emacs/notmuch-crypto.el
+++ b/emacs/notmuch-crypto.el
@@ -222,16 +222,11 @@ corresponding key when the status button is pressed."
 :buffer buffer
 :stderr buffer
 :command (list notmuch-crypto-gpg-program 
"--recv-keys" keyid)
-:sentinel #'notmuch-crypto--async-key-sentinel
-;; Create the process stopped so that
-;; we have time to store the key id,
-;; etc. on it.
-:stop t)))
+:sentinel 
#'notmuch-crypto--async-key-sentinel)))
(process-put p :gpg-key-id keyid)
(process-put p :notmuch-show-buffer (current-buffer))
(process-put p :notmuch-show-point (point))
-   (message "Getting the GPG key %s asynchronously..." keyid)
-   (continue-process p)))
+   (message "Getting the GPG key %s asynchronously..." keyid)))
 
   (let ((window (display-buffer buffer)))
(with-selected-window window
-- 
2.24.1

___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch