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 be
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 be
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. I'm
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. I
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
pgpJ9aFYIkO7n.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuc
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 h
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 ha
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: application/pgp-signature
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
> [..
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
> [..
On Thu, 17 Dec 2009 21:33:54 -0800, Scott Robinson 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-search and notmuch
On Thu, 17 Dec 2009 21:33:54 -0800, Scott Robinson
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-search and notmuc
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
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
14 matches
Mail list logo