[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-31 Thread David Bremner
On Thu, 31 Dec 2009 00:54:55 -0800, Scott Robinson wrote: > Excerpts from Carl Worth's message of Tue Dec 22 21:48:34 -0800 2009: > > On Fri, 18 Dec 2009 10:47:33 -0800, Scott Robinson > > wrote: > > > Resubmit a full patch, or submit another one on top of it? > > > > A new full patch would

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-31 Thread Scott Robinson
Excerpts from Carl Worth's message of Tue Dec 22 21:48:34 -0800 2009: > On Fri, 18 Dec 2009 10:47:33 -0800, Scott Robinson > wrote: > > Resubmit a full patch, or submit another one on top of it? > > A new full patch would be great, thanks! Sadly, I won't be able to make the changes requested.

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-25 Thread David Bremner
On Fri, 18 Dec 2009 10:47:33 -0800, Scott Robinson wrote: > Resubmit a full patch, or submit another one on top of it? My own feeling is that in general if a patch hasn't been merged into master, and the changes don't form a separate logical change (i.e. "oh, if you take that patch, you could

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-22 Thread Carl Worth
On Fri, 18 Dec 2009 10:47:33 -0800, Scott Robinson wrote: > Resubmit a full patch, or submit another one on top of it? A new full patch would be great, thanks! -Carl -- next part -- A non-text attachment was scrubbed... Name: not available Type:

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-18 Thread Scott Robinson
Excerpts from Carl Worth's message of Fri Dec 18 09:31:39 -0800 2009: > [...] > I don't know why, but I think I'd prefer --format for the name here. ACK > [...] > It looks like the new documentation is missing that point, (and the man > page in notmuch.1 is missing an update as well). ACK >

Re: [notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-18 Thread Carl Worth
On Thu, 17 Dec 2009 21:33:54 -0800, Scott Robinson sc...@quadhome.com wrote: I took an earlier suggestion and didn't use cJSON, instead writing custom code for emitting the new format. Nice! I have a few comments below. Added an --output=(json|text|) command-line option to both

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-17 Thread Scott Robinson
Excerpts from David Bremner's message of Mon Dec 14 15:10:12 -0800 2009: > OK, I'll wait for Scott's patch, and see how he is using cJSON. Just > stealing a few functions might be the way to go. > > The cJSON code is less horrifying after running through indent. Now all > we need is "indent

[notmuch] [PATCH] JSON output for notmuch-search and notmuch-show.

2009-12-17 Thread Scott Robinson
Excerpts from David Bremner's message of Mon Dec 14 15:10:12 -0800 2009: OK, I'll wait for Scott's patch, and see how he is using cJSON. Just stealing a few functions might be the way to go. The cJSON code is less horrifying after running through indent. Now all we need is indent --carl :)