[PATCH v2 0/6] reply to sender

Subject: [PATCH v2 0/6] reply to sender

Date: Sun, 8 Jan 2012 23:48:29 +0200

To: notmuch@notmuchmail.org

Cc:

From: Jani Nikula


Hi all, here's v2 of the reply-to-sender series.

Patches 1 & 2 are exactly the same as before.

Changes since first version:

- Patch 3: Settle on --reply-to=(all|sender) option for "notmuch
  reply". This was originally Carl Worth's suggestion, predates this
  patch set, and, as you'd expect, is in line with the rest of the
  CLI. Acked by David on IRC.

- Patch 3: Handle replying to user's own messages gracefully. Credits
  to Mark Walters for his earlier work on this.

- Patch 3: Update man page.

- Patch 4: Change emacs implementation to keep old function names for
  old reply-to-all functionality, and add new reply-to-sender versions
  for new functionality, instead of vice versa. Suggested by Mark
  Walters.

- Add patch 5 to change emacs keybindings, 'r' for reply-to-sender,
  'R' for reply-to-all. I think everyone was in favour of this.

- Add patch 6, written by Mark Winters, to test the cli.

Comments are, as always, welcome.


BR,
Jani.

Jani Nikula (5):
  cli: fix use of uninitialized variable in "notmuch reply"
  cli: convert "notmuch reply" to use the new argument parser
  cli: add support for replying just to the sender in "notmuch reply"
  emacs: add support for replying just to the sender
  emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all

Mark Walters (1):
  test: add tests for "notmuch reply" --reply-to=sender

 emacs/notmuch-mua.el     |    9 ++-
 emacs/notmuch-show.el    |   12 ++-
 emacs/notmuch.el         |   11 ++-
 man/man1/notmuch-reply.1 |   28 +++++-
 notmuch-reply.c          |  155 ++++++++++++++++++++++------------
 test/notmuch-test        |    1 +
 test/reply-to-sender     |  209 ++++++++++++++++++++++++++++++++++++++++++++++
 7 files changed, 356 insertions(+), 69 deletions(-)
 create mode 100755 test/reply-to-sender

-- 
1.7.5.4


Thread: