[PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply"

2012-01-06 Thread David Bremner
On Thu, 5 Jan 2012 22:25:12 +0200, Jani Nikula wrote: > notmuch_show_params_t params is only initialized partially in > notmuch_reply_command(). The only field that is used uninitialized is > params.decrypt. It is usually non-zero, making "notmuch reply" on encrypted > messages work by

[PATCH] run notmuch-hello-mode-hook at the end of the hello mode setup

2012-01-06 Thread Jani Nikula
Hi, looks like a similar patch has been applied as commit db352df09e92171293eacf159342cd950ae596d0. BR, Jani. On Mon, 8 Aug 2011 12:26:58 +0200, Antoine Beaupr? wrote: > i mostly use this to start offlineimap.el, but it could be used for > other purposes. i have not added hooks to other

[PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Tomi Ollila
On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupr? wrote: > > For the curious, those (other) patches are: > > * lib: add 'safe' setting for flags > * lib: Add back the synchronization of 'T' flag with deleted tag > * run notmuch-hello-mode-hook at the end of the hello mode setup To add

[PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Jani Nikula
Hi Antoine - On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupr? wrote: > On Tue, 03 Jan 2012 21:57:22 +0200, Jani Nikula wrote: > > I wouldn't use it either, and I don't use "deleted" tag in the first > > place. And even if I used it, I'd still like to keep the distinction > > between

[PATCH] emacs: call "notmuch tag" only once when archiving a thread

2012-01-06 Thread Tomi Ollila
On Thu, 05 Jan 2012 22:58:30 +0200, Jani Nikula wrote: > On Thu, 05 Jan 2012 12:38:18 -0800, Jameson Graef Rollins finestructure.net> wrote: > > On Thu, 05 Jan 2012 22:32:16 +0200, Jani Nikula wrote: > > > In the search view it does exactly this. > > > > I worry about race conditions in this

[PATCH] emacs: Mark the quoted region during reply.

2012-01-06 Thread Tomi Ollila
On Fri, 6 Jan 2012 10:03:40 +, David Edmondson wrote: > Mark the quoted region of text during a reply, making it easy for the > user to delete it quickly. > --- +1 Tomi > emacs/notmuch-mua.el |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git

[PATCH] emacs: Mark the quoted region during reply.

2012-01-06 Thread Xavier Maillard
On Fri, 6 Jan 2012 10:03:40 +, David Edmondson wrote: > Mark the quoted region of text during a reply, making it easy for the > user to delete it quickly. LGTM ! (That's also a desirable feature). /Xavier

[PATCH v3 1/4] emacs: Let the user choose where to compose new mails

2012-01-06 Thread Thomas Jost
at all with the latest versions of this patch. I'll try to fix that and post a new patch in a few days. Best regards, -- Thomas/Schnouki ------ next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 489 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/1449754d/attachment.pgp>

[RFC][PATCH] emacs: Use text properties rather than overlays in various places.

2012-01-06 Thread David Edmondson
ype: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/fc8c6118/attachment.pgp>

[PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread David Bremner
On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupr? wrote: > On Tue, 03 Jan 2012 21:57:22 +0200, Jani Nikula wrote: > * lib: add 'safe' setting for flags > * lib: Add back the synchronization of 'T' flag with deleted tag > * run notmuch-hello-mode-hook at the end of the hello mode setup >

[PATCH] Perform mail polling asynchronously

2012-01-06 Thread David Edmondson
ubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/235851e2/attachment-0001.pgp>

[PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Antoine Beaupré
attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/8fb6af40/attachment.pgp>

[PATCH 0/4] notmuch reply bugfix & reply to sender only

2012-01-06 Thread Jani Nikula
Hi Mark - On Thu, 05 Jan 2012 22:01:30 +, Mark Walters wrote: > > Bikeshedding topic #1: How about making replying to just the sender the > > default > > in "notmuch reply", and having --reply-all option (instead of > > --no-reply-all)? > > In id:"87pqn5cg4g.fsf at yoom.home.cworth.org"

[PATCH 0/4] notmuch reply bugfix & reply to sender only

2012-01-06 Thread Mark Walters
Hello I have now compared the two patch sets. They seem very similar but I think yours is a little bit nicer in all respects except I prefer my use of "g_mime_message_get_all_recipients (reply) == NULL". However, one case that is slightly less clear with my approach is what to do about

[PATCH] Perform mail polling asynchronously

2012-01-06 Thread Antono Vasiljev
On Tue, 2012-01-03 at 16:02 +, David Edmondson wrote: > On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev > wrote: > > * reimplemented notmuch-poll( callback) so that it > > calls process asynchronously and run callback after process > > finished > > * changed usage of

[PATCH 4/4] add tests for reply-to-sender

2012-01-06 Thread Mark Walters
--- test/notmuch-test|1 + test/reply-to-sender | 209 ++ 2 files changed, 210 insertions(+), 0 deletions(-) create mode 100755 test/reply-to-sender diff --git a/test/notmuch-test b/test/notmuch-test index e40ef86..6a99ae3 100755 ---

[PATCH 3/4] Emacs changes for reply-to-sender

2012-01-06 Thread Mark Walters
Reply to sender is currently bound to R --- emacs/notmuch-mua.el |8 +--- emacs/notmuch-show.el |8 +++- emacs/notmuch.el |2 +- 3 files changed, 13 insertions(+), 5 deletions(-) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 7114e48..7171210 100644

[PATCH 2/4] Update the man page for notmuch-reply to give the syntax for the --reply-to option.

2012-01-06 Thread Mark Walters
--- man/man1/notmuch-reply.1 | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-reply.1 b/man/man1/notmuch-reply.1 index 099d808..13cc72c 100644 --- a/man/man1/notmuch-reply.1 +++ b/man/man1/notmuch-reply.1 @@ -45,6 +45,19 @@ Includes subject

[PATCH 1/4] Add the option "--reply-to" to notmuch reply.

2012-01-06 Thread Mark Walters
Possible values for this option are "sender" which replies just to sender and "all" (the default). More precisely reply to sender follows these rules: reply only to sender unless it was the user reply only to all people on the "to" line unless they were all the user reply

[PATCH 0/4] notmuch reply bugfix & reply to sender only

2012-01-06 Thread Mark Walters
> That is a very good point, and one that my patch fails to address. > > > (*) I have a version of that patch-set which applies to master if that > > would be useful to anyone, and I recently started writing tests in > > preparation for re-submitting. > > Ah, it's old, no wonder I didn't know

[PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Jameson Graef Rollins
bbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/8d55ef64/attachment.pgp>

[PATCH] Perform mail polling asynchronously

2012-01-06 Thread David Edmondson
c io sucks. > Just drop it if you do not like. I don't use notmuch anymore so i dont > care. Antono, thanks for replying. I'll mark the patch as 'obsolete'. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature

Exporting mime email

2012-01-06 Thread David Edmondson
RL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/d0edeeb4/attachment.pgp>

Exporting mime email

2012-01-06 Thread Mark Walters
Hi > Presumably something is converting it from 8bit to base64 along the > path. If it makes a difference my setup is gmail->offlineimap->notmuch > I scouted around for other messages that include ?? (which is what I > presume causes the encoding), but the ones I have are all still 8bit. A

Exporting mime email

2012-01-06 Thread David Edmondson
e: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/6c895bf3/attachment.pgp>

Exporting mime email

2012-01-06 Thread Mark Walters
Hello On Fri, 06 Jan 2012 08:35:27 +, David Edmondson wrote: > On Fri, 06 Jan 2012 00:29:39 +, Mark Walters gmail.com> wrote: > > What is the best way of exporting/saving as text a mime-encoded email > > from the emacs interface (e.g, if I want to save a mime-encoded email > >

[PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply"

2012-01-06 Thread Jani Nikula
On Thu, 05 Jan 2012 23:22:34 -0400, David Bremner wrote: > On Thu, 5 Jan 2012 22:25:12 +0200, Jani Nikula wrote: > > -notmuch_show_params_t params; > > +notmuch_show_params_t params = { .part = -1 }; > > > > reply_format_func = notmuch_reply_format_default; > > -params.part =

[PATCH] emacs: Mark the quoted region during reply.

2012-01-06 Thread David Edmondson
Mark the quoted region of text during a reply, making it easy for the user to delete it quickly. --- emacs/notmuch-mua.el |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 7114e48..32e2e30 100644 --- a/emacs/notmuch-mua.el

[PATCH v2] emacs: Helpers for notmuch developers.

2012-01-06 Thread David Edmondson
--- - Prefix the branch name with 'review/' - Avoid `shell-command', which also results in better error reporting when 'git-am' fails. emacs/notmuch-dev.el | 129 ++ 1 files changed, 129 insertions(+), 0 deletions(-) create mode 100644

[PATCH 2/2] test: Add tests for advance/rewind.

2012-01-06 Thread Jameson Graef Rollins
could provide more info? jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/47e8b664/attachment.pgp>

[RFC][PATCH] emacs: Use text properties rather than overlays in various places.

2012-01-06 Thread Jameson Graef Rollins
- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/41e8fde6/attachment.pgp>

[PATCH 0/4] notmuch reply bugfix & reply to sender only

2012-01-06 Thread David Edmondson
ture Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/5e57f5bc/attachment-0001.pgp>

Exporting mime email

2012-01-06 Thread David Edmondson
:"1325599946-5936-1-git-send-email-dme at dme.org" instead :-)) -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/94de68fe/attachment.pgp>

[PATCH] emacs: Helpers for notmuch developers.

2012-01-06 Thread David Edmondson
vide any help (or even notification) that there are outstanding things to do. It also blocks a future git-am. Currently thinking about an alternative to using git-am that is more magit friendly... -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120106/d2339bce/attachment.pgp>

[PATCH] NEWS: consistent 2-space indentation

2012-01-06 Thread David Bremner
On Wed, 4 Jan 2012 15:26:52 +0200, Tomi Ollila wrote: > --- > This patch is submitted on release branch. > NEWS | 16 > 1 files changed, 8 insertions(+), 8 deletions(-) Hi Tomi; It seems like applying this to release is just asking for trouble. It is already quite an ordeal

[PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply"

2012-01-06 Thread David Bremner
On Fri, 06 Jan 2012 10:11:42 +0200, Jani Nikula wrote: > IMHO the first is cleanest and unaffected by changes in > notmuch_show_params_t, though might be surprising if you're not (yet) > used to C99 designated initializers. Obviously not ;). It would nice to have a consensus about the use of

[PATCH] Perform mail polling asynchronously

2012-01-06 Thread Jesse Rosenthal
On Tue, 03 Jan 2012 16:02:43 +, David Edmondson wrote: > On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev > wrote: > > If the poll script runs asynchronously, won't the notmuch-search output > refresh and jump into my face when I'm busy doing something else? And, > if I'm not busy doing

Exporting mime email

2012-01-06 Thread Mark Walters
Hi On Thu, 05 Jan 2012 16:44:10 -0800, Jameson Graef Rollins wrote: > On Fri, 06 Jan 2012 00:29:39 +, Mark Walters gmail.com> wrote: > > What is the best way of exporting/saving as text a mime-encoded email > > from the emacs interface (e.g, if I want to save a mime-encoded > > email

Exporting mime email

2012-01-06 Thread Mark Walters
Hello What is the best way of exporting/saving as text a mime-encoded email from the emacs interface (e.g, if I want to save a mime-encoded email containing a patch)? For non-mime emails View-raw is fine, but with mime encoded emails such as (picked pretty much at random)

Re: [PATCH 1/4] cli: fix use of uninitialized variable in notmuch reply

2012-01-06 Thread Jani Nikula
On Thu, 05 Jan 2012 23:22:34 -0400, David Bremner da...@tethera.net wrote: On Thu, 5 Jan 2012 22:25:12 +0200, Jani Nikula j...@nikula.org wrote: -notmuch_show_params_t params; +notmuch_show_params_t params = { .part = -1 }; reply_format_func = notmuch_reply_format_default;

Re: [PATCH] emacs: Helpers for notmuch developers.

2012-01-06 Thread David Edmondson
On Thu, 05 Jan 2012 23:39:30 -0400, David Bremner da...@tethera.net wrote: General management (i.e. keeping up to date) of the repository it uses is your responsibility, as is cleaning out old branches. You can, of course, just delete the temporary repository after using it - the code will

Re: Exporting mime email

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 00:29:39 +, Mark Walters markwalters1...@gmail.com wrote: What is the best way of exporting/saving as text a mime-encoded email from the emacs interface (e.g, if I want to save a mime-encoded email containing a patch)? For non-mime emails View-raw is fine, but with

Re: [PATCH 0/4] notmuch reply bugfix reply to sender only

2012-01-06 Thread David Edmondson
On Thu, 5 Jan 2012 22:25:11 +0200, Jani Nikula j...@nikula.org wrote: Bikeshedding topic #1: How about making replying to just the sender the default in notmuch reply, and having --reply-all option (instead of --no-reply-all)? No strong opinion (I hardly ever use the CLI directly).

[PATCH v2] emacs: Helpers for notmuch developers.

2012-01-06 Thread David Edmondson
--- - Prefix the branch name with 'review/' - Avoid `shell-command', which also results in better error reporting when 'git-am' fails. emacs/notmuch-dev.el | 129 ++ 1 files changed, 129 insertions(+), 0 deletions(-) create mode 100644

[PATCH] emacs: Mark the quoted region during reply.

2012-01-06 Thread David Edmondson
Mark the quoted region of text during a reply, making it easy for the user to delete it quickly. --- emacs/notmuch-mua.el |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 7114e48..32e2e30 100644 --- a/emacs/notmuch-mua.el

Re: Exporting mime email

2012-01-06 Thread Mark Walters
Hello On Fri, 06 Jan 2012 08:35:27 +, David Edmondson d...@dme.org wrote: On Fri, 06 Jan 2012 00:29:39 +, Mark Walters markwalters1...@gmail.com wrote: What is the best way of exporting/saving as text a mime-encoded email from the emacs interface (e.g, if I want to save a

Re: Exporting mime email

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 10:27:16 +, Mark Walters markwalters1...@gmail.com wrote: Does saving the message like this give you a file you can apply directly with the patch command? When I save it (using | or from view-raw) I get a message which looks like Ahh. This particular message is a bad

Re: Exporting mime email

2012-01-06 Thread Mark Walters
Hi Presumably something is converting it from 8bit to base64 along the path. If it makes a difference my setup is gmail-offlineimap-notmuch I scouted around for other messages that include © (which is what I presume causes the encoding), but the ones I have are all still 8bit. A

Re: Exporting mime email

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 10:51:47 +, Mark Walters markwalters1...@gmail.com wrote: Presumably something is converting it from 8bit to base64 along the path. If it makes a difference my setup is gmail-offlineimap-notmuch Mine is the same for the relevant messages. I scouted around for

Re: [PATCH] Perform mail polling asynchronously

2012-01-06 Thread Antono Vasiljev
On Tue, 2012-01-03 at 16:02 +, David Edmondson wrote: On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev s...@antono.info wrote: * reimplemented notmuch-poll(optional callback) so that it calls process asynchronously and run callback after process finished * changed

Re: [PATCH] Perform mail polling asynchronously

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 14:09:57 +0300, Antono Vasiljev s...@antono.info wrote: On Tue, 2012-01-03 at 16:02 +, David Edmondson wrote: On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev s...@antono.info wrote: * reimplemented notmuch-poll(optional callback) so that it calls

Re: [PATCH 1/4] cli: fix use of uninitialized variable in notmuch reply

2012-01-06 Thread David Bremner
On Fri, 06 Jan 2012 10:11:42 +0200, Jani Nikula j...@nikula.org wrote: IMHO the first is cleanest and unaffected by changes in notmuch_show_params_t, though might be surprising if you're not (yet) used to C99 designated initializers. Obviously not ;). It would nice to have a consensus about

Re: [PATCH] NEWS: consistent 2-space indentation

2012-01-06 Thread David Bremner
On Wed, 4 Jan 2012 15:26:52 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: --- This patch is submitted on release branch. NEWS | 16 1 files changed, 8 insertions(+), 8 deletions(-) Hi Tomi; It seems like applying this to release is just asking for trouble. It is already

Re: [PATCH 0/4] notmuch reply bugfix reply to sender only

2012-01-06 Thread Jani Nikula
Hi Mark - On Thu, 05 Jan 2012 22:01:30 +, Mark Walters markwalters1...@gmail.com wrote: Bikeshedding topic #1: How about making replying to just the sender the default in notmuch reply, and having --reply-all option (instead of --no-reply-all)? In

Re: [PATCH 0/4] notmuch reply bugfix reply to sender only

2012-01-06 Thread Mark Walters
That is a very good point, and one that my patch fails to address. (*) I have a version of that patch-set which applies to master if that would be useful to anyone, and I recently started writing tests in preparation for re-submitting. Ah, it's old, no wonder I didn't know about it.

[PATCH 1/4] Add the option --reply-to to notmuch reply.

2012-01-06 Thread Mark Walters
Possible values for this option are sender which replies just to sender and all (the default). More precisely reply to sender follows these rules: reply only to sender unless it was the user reply only to all people on the to line unless they were all the user reply to all

[PATCH 3/4] Emacs changes for reply-to-sender

2012-01-06 Thread Mark Walters
Reply to sender is currently bound to R --- emacs/notmuch-mua.el |8 +--- emacs/notmuch-show.el |8 +++- emacs/notmuch.el |2 +- 3 files changed, 13 insertions(+), 5 deletions(-) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 7114e48..7171210 100644

[PATCH 4/4] add tests for reply-to-sender

2012-01-06 Thread Mark Walters
--- test/notmuch-test|1 + test/reply-to-sender | 209 ++ 2 files changed, 210 insertions(+), 0 deletions(-) create mode 100755 test/reply-to-sender diff --git a/test/notmuch-test b/test/notmuch-test index e40ef86..6a99ae3 100755 ---

Re: [PATCH] Perform mail polling asynchronously

2012-01-06 Thread Jesse Rosenthal
On Tue, 03 Jan 2012 16:02:43 +, David Edmondson d...@dme.org wrote: On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev s...@antono.info wrote: If the poll script runs asynchronously, won't the notmuch-search output refresh and jump into my face when I'm busy doing something else? And, if

Re: [PATCH] Perform mail polling asynchronously

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 06:45:59 -0800, Jesse Rosenthal jrosent...@jhu.edu wrote: On Tue, 03 Jan 2012 16:02:43 +, David Edmondson d...@dme.org wrote: On Mon, 5 Sep 2011 07:14:36 +0300, Antono Vasiljev s...@antono.info wrote: If the poll script runs asynchronously, won't the

Re: [RFC][PATCH] emacs: Use text properties rather than overlays in various places.

2012-01-06 Thread Jameson Graef Rollins
On Wed, 28 Dec 2011 14:31:31 +, David Edmondson d...@dme.org wrote: Probably just for aficionados at the moment... Actually, I think that all the rest of these comments should be included in the log. There's useful discussion here. Given that clashes between overlays and text properties

Re: [PATCH 2/2] test: Add tests for advance/rewind.

2012-01-06 Thread Jameson Graef Rollins
On Thu, 29 Dec 2011 16:05:39 +, David Edmondson d...@dme.org wrote: What's the general feeling about this approach to producing tests for the emacs UI? (That is, code the test in a .el file and call the relevant function(s) from the test harness.) I think it's a nice idea. It seems much

Re: [RFC][PATCH] emacs: Use text properties rather than overlays in various places.

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 08:58:31 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Wed, 28 Dec 2011 14:31:31 +, David Edmondson d...@dme.org wrote: Probably just for aficionados at the moment... Actually, I think that all the rest of these comments should be included in the

Re: [PATCH 2/2] test: Add tests for advance/rewind.

2012-01-06 Thread David Edmondson
On Fri, 06 Jan 2012 09:10:17 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: It makes it simpler to develop and maintain the test (because you can do more work with traditional emacs support for editing elisp), but might make interpreting failures more difficult (the test

Re: [PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Antoine Beaupré
On Tue, 03 Jan 2012 21:57:22 +0200, Jani Nikula j...@nikula.org wrote: I wouldn't use it either, and I don't use deleted tag in the first place. And even if I used it, I'd still like to keep the distinction between deleted after reading and deleted unread, which this patch loses by removing

Re: [PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Jameson Graef Rollins
On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupré anar...@koumbit.org wrote: Most patches I have submitted here haven't been accepted and I have to painfully reroll my own packages every time there's a new release, which has been a very frustrating experience. To see such a trivial patch

Re: [PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread David Bremner
On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupré anar...@koumbit.org wrote: On Tue, 03 Jan 2012 21:57:22 +0200, Jani Nikula j...@nikula.org wrote: * lib: add 'safe' setting for flags * lib: Add back the synchronization of 'T' flag with deleted tag * run notmuch-hello-mode-hook at the

Re: [PATCH] emacs: Mark the quoted region during reply.

2012-01-06 Thread Tomi Ollila
On Fri, 6 Jan 2012 10:03:40 +, David Edmondson d...@dme.org wrote: Mark the quoted region of text during a reply, making it easy for the user to delete it quickly. --- +1 Tomi emacs/notmuch-mua.el |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git

Re: [PATCH 1/2] add notmuch keybinding 'd'

2012-01-06 Thread Tomi Ollila
On Fri, 06 Jan 2012 15:10:57 -0500, Antoine Beaupré anar...@koumbit.org wrote: For the curious, those (other) patches are: * lib: add 'safe' setting for flags * lib: Add back the synchronization of 'T' flag with deleted tag * run notmuch-hello-mode-hook at the end of the hello mode

Re: [PATCH] run notmuch-hello-mode-hook at the end of the hello mode setup

2012-01-06 Thread Jani Nikula
Hi, looks like a similar patch has been applied as commit db352df09e92171293eacf159342cd950ae596d0. BR, Jani. On Mon, 8 Aug 2011 12:26:58 +0200, Antoine Beaupré anar...@koumbit.org wrote: i mostly use this to start offlineimap.el, but it could be used for other purposes. i have not added

Re: [PATCH 0/4] notmuch reply bugfix reply to sender only

2012-01-06 Thread Tomi Ollila
On Thu, 5 Jan 2012 17:01:44 -0700, Adam Wolfe Gordon awg+notm...@xvx.ca wrote: On Thu, Jan 5, 2012 at 13:25, Jani Nikula j...@nikula.org wrote: Bikeshedding topic #1: How about making replying to just the sender the default in notmuch reply, and having --reply-all option (instead of

Re: [PATCH 2/2] lib: add 'safe' setting for flags

2012-01-06 Thread Jani Nikula
On Sat, 16 Jul 2011 23:56:13 -0400, Antoine Beaupré anar...@koumbit.org wrote: the 'safe' setting needs to be 'true' for flags to be manipulated by notmuch new/tag/restore. for now, only the (T)rash tag is configurable and set to false (by default) but this could be extended to allow the

Re: [PATCH 1/4] cli: fix use of uninitialized variable in notmuch reply

2012-01-06 Thread David Bremner
On Thu, 5 Jan 2012 22:25:12 +0200, Jani Nikula j...@nikula.org wrote: notmuch_show_params_t params is only initialized partially in notmuch_reply_command(). The only field that is used uninitialized is params.decrypt. It is usually non-zero, making notmuch reply on encrypted messages work by

Re: [PATCH 1/4] cli: fix use of uninitialized variable in notmuch reply

2012-01-06 Thread Jani Nikula
On Jan 7, 2012 5:52 AM, David Bremner da...@tethera.net wrote: On Thu, 5 Jan 2012 22:25:12 +0200, Jani Nikula j...@nikula.org wrote: notmuch_show_params_t params is only initialized partially in notmuch_reply_command(). The only field that is used uninitialized is params.decrypt. It is