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

2012-01-07 Thread Mueen Nawaz
Jameson Graef Rollins writes: > On Thu, 5 Jan 2012 22:25:11 +0200, Jani Nikula 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)? >> >> Bikeshedding topic #2: How

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

2012-01-07 Thread Tomi Ollila
On Thu, 5 Jan 2012 17:01:44 -0700, Adam Wolfe Gordon wrote: > On Thu, Jan 5, 2012 at 13:25, Jani Nikula 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 0/4] notmuch reply bugfix reply to sender only

2012-01-07 Thread Mueen Nawaz
Jameson Graef Rollins jroll...@finestructure.net writes: 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)?

[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 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 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 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). > Bikeshedding

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).

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.

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

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

2012-01-05 Thread Jani Nikula
Hi all - Patch 1 is a bugfix worth considering for release branch independently. Included in the same patchset as the following ones depend on it. The rest of the patches add reply-to-sender (as opposed to reply-to-all) functionality to "notmuch reply" and emacs. Bikeshedding topic #1: How

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

2012-01-05 Thread Mark Walters
Hello > 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" cworth suggested using --reply-to=sender vs. --reply-to=all (keeping the

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

2012-01-05 Thread Adam Wolfe Gordon
On Thu, Jan 5, 2012 at 13:25, Jani Nikula 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)? > > Bikeshedding topic #2: How about binding 'r' to reply to just the sender by >

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

2012-01-05 Thread Jameson Graef Rollins
On Thu, 5 Jan 2012 22:25:11 +0200, Jani Nikula 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)? > > Bikeshedding topic #2: How about binding 'r' to reply to just the

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

2012-01-05 Thread Jani Nikula
Hi all - Patch 1 is a bugfix worth considering for release branch independently. Included in the same patchset as the following ones depend on it. The rest of the patches add reply-to-sender (as opposed to reply-to-all) functionality to notmuch reply and emacs. Bikeshedding topic #1: How about

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

2012-01-05 Thread Jameson Graef Rollins
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)? Bikeshedding topic #2: How about binding 'r' to reply to just

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

2012-01-05 Thread Mark Walters
Hello 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@yoom.home.cworth.org cworth suggested using --reply-to=sender vs. --reply-to=all (keeping the latter the

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

2012-01-05 Thread Adam Wolfe Gordon
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 --no-reply-all)? Bikeshedding topic #2: How about binding 'r' to reply to just the