tag sharing [was: Re: release-candidate/0.6 redux]

2011-06-08 Thread Jesse Rosenthal
On Wed, 08 Jun 2011 10:46:57 -0700, Jameson Graef Rollins wrote: > Did you guys try to address the issue of tag removal at all? I've been > trying to decide if this is something we need to worry about or not. > For instance, if cworth pushed a tag ".needs-review", you would probably > want to hav

Re: [PATCH 00/10] Fix 'notmuch new' atomicity issues

2011-06-08 Thread Carl Worth
On Sat, 28 May 2011 22:51:10 -0400, Austin Clements wrote: > Rebased to current master (cb8418) as atomic-new-v4 (aka > for-review/atomic-new-v4). Hi Austin, Thanks so much for sending this series (and 4 times, even!). I *really* like the new robustness provided by this series, and I especially

[PATCH 00/10] Fix 'notmuch new' atomicity issues

2011-06-08 Thread Carl Worth
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/20110608/b6c54405/attachment-0001.pgp>

[PATCH 3/4] test: test for absence of "Non-text part: application/pgp-*" lines in reply

2011-06-08 Thread Jameson Graef Rollins
In reply, the quoted text does not need to mention that the original message had "application/pgp-signed" or "application/pgp-encrypted" parts. --- test/crypto|2 -- test/multipart |1 - 2 files changed, 0 insertions(+), 3 deletions(-) diff --git a/test/crypto b/test/crypto index 8e92

tweaks for handling parts in reply

2011-06-08 Thread Jameson Graef Rollins
This is a small set of tweaks to remove unneccessary "Non-text part:" lines in reply, for parts that really don't need to be mentioned. This patch set is not really related to the series that it is being sent in reply to, but it has been worked on top of the message/rfc822 rework of the multipart

[PATCH 1/4] test/multipart: test for absence of "Non-text part:" lines in reply for multipart/* and message/rfc822 parts

2011-06-08 Thread Jameson Graef Rollins
There's no reason to output "Non-text part:" lines for parts that are not leaf nodes, eg. multipart/* and message/rfc822. We fix the text here to test for their absence. The next patch will fix reply accordingly. --- test/multipart |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) di

[PATCH 2/4] Fix notmuch-reply to not output "Non-text part:" lines for non-leafnode parts.

2011-06-08 Thread Jameson Graef Rollins
These lines are just cruft in this case, and can be removed. --- notmuch-reply.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index a19eb19..7a76ba3 100644 --- a/notmuch-reply.c +++ b/notmuch-reply.c @@ -93,7 +93,12 @@ reply_p

[PATCH 4/4] Ignore "application/pgp-*" parts in reply.

2011-06-08 Thread Jameson Graef Rollins
The quoted text doesn't need to mention that the message being replied to had these crufty parts. --- notmuch-reply.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index 7a76ba3..0eb295e 100644 --- a/notmuch-reply.c +++ b/notmuch-rep

[PATCH 4/4] Ignore "application/pgp-*" parts in reply.

2011-06-08 Thread Jameson Graef Rollins
The quoted text doesn't need to mention that the message being replied to had these crufty parts. --- notmuch-reply.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index 7a76ba3..0eb295e 100644 --- a/notmuch-reply.c +++ b/notmuch-rep

[PATCH 3/4] test: test for absence of "Non-text part: application/pgp-*" lines in reply

2011-06-08 Thread Jameson Graef Rollins
In reply, the quoted text does not need to mention that the original message had "application/pgp-signed" or "application/pgp-encrypted" parts. --- test/crypto|2 -- test/multipart |1 - 2 files changed, 0 insertions(+), 3 deletions(-) diff --git a/test/crypto b/test/crypto index 8e92

[PATCH 2/4] Fix notmuch-reply to not output "Non-text part:" lines for non-leafnode parts.

2011-06-08 Thread Jameson Graef Rollins
These lines are just cruft in this case, and can be removed. --- notmuch-reply.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index a19eb19..7a76ba3 100644 --- a/notmuch-reply.c +++ b/notmuch-reply.c @@ -93,7 +93,12 @@ reply_p

[PATCH 1/4] test/multipart: test for absence of "Non-text part:" lines in reply for multipart/* and message/rfc822 parts

2011-06-08 Thread Jameson Graef Rollins
There's no reason to output "Non-text part:" lines for parts that are not leaf nodes, eg. multipart/* and message/rfc822. We fix the text here to test for their absence. The next patch will fix reply accordingly. --- test/multipart |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) di

tweaks for handling parts in reply

2011-06-08 Thread Jameson Graef Rollins
This is a small set of tweaks to remove unneccessary "Non-text part:" lines in reply, for parts that really don't need to be mentioned. This patch set is not really related to the series that it is being sent in reply to, but it has been worked on top of the message/rfc822 rework of the multipart

Re: tag sharing [was: Re: release-candidate/0.6 redux]

2011-06-08 Thread Jesse Rosenthal
On Wed, 08 Jun 2011 10:46:57 -0700, Jameson Graef Rollins wrote: > Did you guys try to address the issue of tag removal at all? I've been > trying to decide if this is something we need to worry about or not. > For instance, if cworth pushed a tag ".needs-review", you would probably > want to ha

Re: tag sharing [was: Re: release-candidate/0.6 redux]

2011-06-08 Thread Jameson Graef Rollins
On Mon, 06 Jun 2011 13:20:00 -0400, Jesse Rosenthal wrote: > After a conversation with David last year about bug-tracking, I worked > up a rough python-based prototype of this. It worked in terms of > namespaces, so Carl could associate the namespace "public" with a list > of tags he publishes to

tag sharing [was: Re: release-candidate/0.6 redux]

2011-06-08 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/20110608/22059ca1/attachment.pgp>