[PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-18 Thread David Bremner
On Tue, 13 Mar 2012 22:30:05 -0600, Adam Wolfe Gordon  
wrote:
> Hi everyone,
> 
> This is mostly a re-send of last night's series [1]. The only changes are:
> 
> * Fixed the emacs patch, where I had a rebase issue.
> * Fixed the commit that re-added test/test-lib by accident.
> * Fixed devel/schemata per Austin's review of the previous version.

It looks like 04/11 doesn't apply anymore after the rest of the pending
patches were applied. It looked like the patches before that didn't
really make sense in isolation so I'm going to tag the whole series
stale.  When you rebase, no need for "needs-review" tags.

d


Re: [PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-18 Thread David Bremner
On Tue, 13 Mar 2012 22:30:05 -0600, Adam Wolfe Gordon  
wrote:
> Hi everyone,
> 
> This is mostly a re-send of last night's series [1]. The only changes are:
> 
> * Fixed the emacs patch, where I had a rebase issue.
> * Fixed the commit that re-added test/test-lib by accident.
> * Fixed devel/schemata per Austin's review of the previous version.

It looks like 04/11 doesn't apply anymore after the rest of the pending
patches were applied. It looked like the patches before that didn't
really make sense in isolation so I'm going to tag the whole series
stale.  When you rebase, no need for "needs-review" tags.

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


[PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-14 Thread Austin Clements
Quoth Adam Wolfe Gordon on Mar 13 at 10:30 pm:
> Hi everyone,
> 
> This is mostly a re-send of last night's series [1]. The only changes are:
> 
> * Fixed the emacs patch, where I had a rebase issue.
> * Fixed the commit that re-added test/test-lib by accident.
> * Fixed devel/schemata per Austin's review of the previous version.
> 
> Since this series seems to be getting close, I figured I'd add a NEWS entry
> for good measure.
> 
> Thanks for all the reviews so far, and I hope this is almost ready!
> 
> [1] id:"1331525142-30539-1-git-send-email-awg+notmuch at xvx.ca"

LGTM.


Re: [PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-14 Thread Austin Clements
Quoth Adam Wolfe Gordon on Mar 13 at 10:30 pm:
> Hi everyone,
> 
> This is mostly a re-send of last night's series [1]. The only changes are:
> 
> * Fixed the emacs patch, where I had a rebase issue.
> * Fixed the commit that re-added test/test-lib by accident.
> * Fixed devel/schemata per Austin's review of the previous version.
> 
> Since this series seems to be getting close, I figured I'd add a NEWS entry
> for good measure.
> 
> Thanks for all the reviews so far, and I hope this is almost ready!
> 
> [1] id:"1331525142-30539-1-git-send-email-awg+notm...@xvx.ca"

LGTM.
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


[PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-13 Thread Adam Wolfe Gordon
Hi everyone,

This is mostly a re-send of last night's series [1]. The only changes are:

* Fixed the emacs patch, where I had a rebase issue.
* Fixed the commit that re-added test/test-lib by accident.
* Fixed devel/schemata per Austin's review of the previous version.

Since this series seems to be getting close, I figured I'd add a NEWS entry
for good measure.

Thanks for all the reviews so far, and I hope this is almost ready!

[1] id:"1331525142-30539-1-git-send-email-awg+notmuch at xvx.ca"

Adam Wolfe Gordon (11):
  test: Add broken test for the new JSON reply format.
  reply: Factor out reply creation
  TODO: Add replying to multiple messages
  reply: Add a JSON reply format.
  schemata: Add documentation for JSON reply format.
  man: Update notmuch-reply man page for JSON format.
  man: Add --decrypt to reply flags
  emacs: Factor out useful functions into notmuch-lib
  test: Add broken tests for new emacs reply functionality
  emacs: Use the new JSON reply format and message-cite-original
  NEWS: news for reply enhancements

 NEWS |   20 ++
 devel/TODO   |8 +++
 devel/schemata   |   27 -
 emacs/notmuch-lib.el |   63 +++
 emacs/notmuch-mua.el |  124 +
 emacs/notmuch-show.el|   55 +++--
 man/man1/notmuch-reply.1 |   20 ++-
 notmuch-client.h |   14 +++-
 notmuch-reply.c  |  153 +-
 notmuch-show.c   |   29 ++---
 test/emacs   |  101 ++-
 test/multipart   |   52 
 12 files changed, 518 insertions(+), 148 deletions(-)

-- 
1.7.5.4



[PATCH v7.1 00/11] Reply enhancements, second attempt

2012-03-13 Thread Adam Wolfe Gordon
Hi everyone,

This is mostly a re-send of last night's series [1]. The only changes are:

* Fixed the emacs patch, where I had a rebase issue.
* Fixed the commit that re-added test/test-lib by accident.
* Fixed devel/schemata per Austin's review of the previous version.

Since this series seems to be getting close, I figured I'd add a NEWS entry
for good measure.

Thanks for all the reviews so far, and I hope this is almost ready!

[1] id:"1331525142-30539-1-git-send-email-awg+notm...@xvx.ca"

Adam Wolfe Gordon (11):
  test: Add broken test for the new JSON reply format.
  reply: Factor out reply creation
  TODO: Add replying to multiple messages
  reply: Add a JSON reply format.
  schemata: Add documentation for JSON reply format.
  man: Update notmuch-reply man page for JSON format.
  man: Add --decrypt to reply flags
  emacs: Factor out useful functions into notmuch-lib
  test: Add broken tests for new emacs reply functionality
  emacs: Use the new JSON reply format and message-cite-original
  NEWS: news for reply enhancements

 NEWS |   20 ++
 devel/TODO   |8 +++
 devel/schemata   |   27 -
 emacs/notmuch-lib.el |   63 +++
 emacs/notmuch-mua.el |  124 +
 emacs/notmuch-show.el|   55 +++--
 man/man1/notmuch-reply.1 |   20 ++-
 notmuch-client.h |   14 +++-
 notmuch-reply.c  |  153 +-
 notmuch-show.c   |   29 ++---
 test/emacs   |  101 ++-
 test/multipart   |   52 
 12 files changed, 518 insertions(+), 148 deletions(-)

-- 
1.7.5.4

___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch