[PATCH] emacs: process crypto for reply only when specified

2014-04-14 Thread Tomi Ollila
On Mon, Apr 14 2014, Tomi Ollila wrote: > On Mon, Apr 14 2014, Jameson Graef Rollins > wrote: > >> >> In any event, if the mml tag is present, it's no longer in notmuch's >> hands; emacs's mail processing is handling things and calling gpg-agent >> to sign/encrypt the message. >> >> Can you cla

[PATCH] emacs: process crypto for reply only when specified

2014-04-14 Thread Tomi Ollila
On Mon, Apr 14 2014, Jameson Graef Rollins wrote: > On Sun, Apr 13 2014, Tomi Ollila wrote: >>> Perhaps people with no ability to sign are less likely to have >>> "notmuch-crypto-process-mime" set? Or we can add another configuration >>> variable initialized from notmuch-crypto-process-mime, b

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-14 Thread Tomi Ollila
On Mon, Apr 14 2014, Tomi Ollila wrote: > On Mon, Apr 14 2014, Jameson Graef Rollins wrote: > >> >> In any event, if the mml tag is present, it's no longer in notmuch's >> hands; emacs's mail processing is handling things and calling gpg-agent >> to sign/encrypt the message. >> >> Can you clarif

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-14 Thread Tomi Ollila
On Mon, Apr 14 2014, Jameson Graef Rollins wrote: > On Sun, Apr 13 2014, Tomi Ollila wrote: >>> Perhaps people with no ability to sign are less likely to have >>> "notmuch-crypto-process-mime" set? Or we can add another configuration >>> variable initialized from notmuch-crypto-process-mime, bu

[PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Tomi Ollila
On Sun, Apr 13 2014, David Bremner wrote: > Tomi Ollila writes: > >> >> Code looks OK. +1. I'm interested to see whether replies to signed emails >> work on this system where sign/(de|en)cryption just doesn't work (out of >> the box) :D >> > > It's a good point. I need to insert my smartcard to s

[PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread David Bremner
Tomi Ollila writes: > > Well, I set notmuch-crypto-process-mime to nil -- it still wants to > sign the message and runs gpg... > Was it nil when you replied, i.e. when the mml tags were created? d

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Jameson Graef Rollins
On Sun, Apr 13 2014, Tomi Ollila wrote: >> Perhaps people with no ability to sign are less likely to have >> "notmuch-crypto-process-mime" set? Or we can add another configuration >> variable initialized from notmuch-crypto-process-mime, but allowing >> people to shut this off. > > Well, I set no

[PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Jameson Graef Rollins
On Sun, Apr 13 2014, Tomi Ollila wrote: >> Perhaps people with no ability to sign are less likely to have >> "notmuch-crypto-process-mime" set? Or we can add another configuration >> variable initialized from notmuch-crypto-process-mime, but allowing >> people to shut this off. > > Well, I set no

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread David Bremner
Tomi Ollila writes: > > Well, I set notmuch-crypto-process-mime to nil -- it still wants to > sign the message and runs gpg... > Was it nil when you replied, i.e. when the mml tags were created? d ___ notmuch mailing list notmuch@notmuchmail.org http:

[PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Tomi Ollila
On Sun, Apr 13 2014, David Bremner wrote: > Jameson Graef Rollins writes: > >> This is a tweak to patch "emacs: sign/encrypt replies to >> signed/encrypted messages" to only add mml crypto flags for replys >> when crypto processing has been activated. >> > > The (merged) patch seems straightforw

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Tomi Ollila
On Sun, Apr 13 2014, David Bremner wrote: > Tomi Ollila writes: > >> >> Code looks OK. +1. I'm interested to see whether replies to signed emails >> work on this system where sign/(de|en)cryption just doesn't work (out of >> the box) :D >> > > It's a good point. I need to insert my smartcard to s

[PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread David Bremner
Tomi Ollila writes: > > Code looks OK. +1. I'm interested to see whether replies to signed emails > work on this system where sign/(de|en)cryption just doesn't work (out of > the box) :D > It's a good point. I need to insert my smartcard to sign things, which I'm sometimes too lazy to do. In my

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread David Bremner
Tomi Ollila writes: > > Code looks OK. +1. I'm interested to see whether replies to signed emails > work on this system where sign/(de|en)cryption just doesn't work (out of > the box) :D > It's a good point. I need to insert my smartcard to sign things, which I'm sometimes too lazy to do. In my

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-13 Thread Tomi Ollila
On Sun, Apr 13 2014, David Bremner wrote: > Jameson Graef Rollins writes: > >> This is a tweak to patch "emacs: sign/encrypt replies to >> signed/encrypted messages" to only add mml crypto flags for replys >> when crypto processing has been activated. >> > > The (merged) patch seems straightforw

[PATCH] emacs: process crypto for reply only when specified

2014-04-12 Thread David Bremner
Jameson Graef Rollins writes: > This is a tweak to patch "emacs: sign/encrypt replies to > signed/encrypted messages" to only add mml crypto flags for replys > when crypto processing has been activated. > The (merged) patch seems straightforward and seems to work. I'll probably push it tomorrow

Re: [PATCH] emacs: process crypto for reply only when specified

2014-04-12 Thread David Bremner
Jameson Graef Rollins writes: > This is a tweak to patch "emacs: sign/encrypt replies to > signed/encrypted messages" to only add mml crypto flags for replys > when crypto processing has been activated. > The (merged) patch seems straightforward and seems to work. I'll probably push it tomorrow

[PATCH] emacs: process crypto for reply only when specified

2014-04-06 Thread Jameson Graef Rollins
This is a tweak to patch "emacs: sign/encrypt replies to signed/encrypted messages" to only add mml crypto flags for replys when crypto processing has been activated. --- Thanks to mjw1009 for implementation suggestions. Jani, you might consider squashing this with your original for a v2. Pushin

[PATCH] emacs: process crypto for reply only when specified

2014-04-06 Thread Jameson Graef Rollins
This is a tweak to patch "emacs: sign/encrypt replies to signed/encrypted messages" to only add mml crypto flags for replys when crypto processing has been activated. --- Thanks to mjw1009 for implementation suggestions. Jani, you might consider squashing this with your original for a v2. Pushin