8c/80e3ac7e09b4e62265e6118bbd2c6e62484853

   1 Return-Path: <amdragon@mit.edu>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 3C066431FAF\r
   6         for <notmuch@notmuchmail.org>; Mon, 20 Feb 2012 21:59:24 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id 8CgtjXtMHxST for <notmuch@notmuchmail.org>;\r
  16         Mon, 20 Feb 2012 21:59:23 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-5.mit.edu (DMZ-MAILSEC-SCANNER-5.MIT.EDU\r
  18         [18.7.68.34])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 929DC431FAE\r
  20         for <notmuch@notmuchmail.org>; Mon, 20 Feb 2012 21:59:23 -0800 (PST)\r
  21 X-AuditID: 12074422-b7fd66d0000008f9-de-4f4332badb61\r
  22 Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
  23         by dmz-mailsec-scanner-5.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 2F.17.02297.AB2334F4; Tue, 21 Feb 2012 00:59:23 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id q1L5xMQx027803; \r
  27         Tue, 21 Feb 2012 00:59:22 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q1L5xKev014435\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Tue, 21 Feb 2012 00:59:21 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1Rziky-0001bB-LG; Tue, 21 Feb 2012 00:59:20 -0500\r
  37 Date: Tue, 21 Feb 2012 00:59:20 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
  40 Subject: Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and\r
  41         message-cite-original\r
  42 Message-ID: <20120221055920.GA30513@mit.edu>\r
  43 References: <1329361957-28493-1-git-send-email-awg+notmuch@xvx.ca>\r
  44         <1329361957-28493-8-git-send-email-awg+notmuch@xvx.ca>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: text/plain; charset=us-ascii\r
  47 Content-Disposition: inline\r
  48 In-Reply-To: <1329361957-28493-8-git-send-email-awg+notmuch@xvx.ca>\r
  49 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  50 X-Brightmail-Tracker:\r
  51  H4sIAAAAAAAAA+NgFmpmleLIzCtJLcpLzFFi42IR4hRV1t1t5OxvMOUMk8WRPbPYLa7fnMns\r
  52         wOTxbNUtZo+mH4tZA5iiuGxSUnMyy1KL9O0SuDI+zVjDUrBOoGLnwscsDYwzeLsYOTkkBEwk\r
  53         2pdOZIWwxSQu3FvP1sXIxSEksI9R4uiPblYIZwOjxJsJX5lBqoQETjJJvL/gB5FYwijxpO0I\r
  54         WDuLgKpE64597CA2m4CGxLb9yxlBbBEBLYkf67+C1TALSEt8+93MBGILC8RJfPt+BayeV0BH\r
  55         YtGemawQC6olJvefYYaIC0qcnPmEBaJXS+LGv5dAvRxgc5b/4wAJcwo4SzS9+A22SlRARWLK\r
  56         yW1sExiFZiHpnoWkexZC9wJG5lWMsim5Vbq5iZk5xanJusXJiXl5qUW6pnq5mSV6qSmlmxhB\r
  57         Yc3uorSD8edBpUOMAhyMSjy8nKVO/kKsiWXFlbmHGCU5mJREeVcYOPsL8SXlp1RmJBZnxBeV\r
  58         5qQWH2KU4GBWEuHdpAmU401JrKxKLcqHSUlzsCiJ86prvfMTEkhPLEnNTk0tSC2CycpwcChJ\r
  59         8JoC41dIsCg1PbUiLTOnBCHNxMEJMpwHaLg6SA1vcUFibnFmOkT+FKOilDhEQgAkkVGaB9cL\r
  60         SzuvGMWBXhHmNQSp4gGmLLjuV0CDmYAGt/x3BBlckoiQkmpgLLrA2cNf1Pb/qWfop2oPqZza\r
  61         8M8dras2rgu0kO5fLX7qJftPZ5WjGiUikrZsXVOMt6dpFWWUf76y/afZjVvXWPo7zb/ybtq4\r
  62         50v2hfLCevv5s/u6szl23Rdg+yb02jpuq1Ou/tv0NRY3bpVUL6+zdJEs+SoqvifmJvOE0z6V\r
  63         uZOZ2mdH7VilxFKckWioxVxUnAgAdhFT5hYDAAA=\r
  64 Cc: notmuch@notmuchmail.org\r
  65 X-BeenThere: notmuch@notmuchmail.org\r
  66 X-Mailman-Version: 2.1.13\r
  67 Precedence: list\r
  68 List-Id: "Use and development of the notmuch mail system."\r
  69         <notmuch.notmuchmail.org>\r
  70 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  72 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  73 List-Post: <mailto:notmuch@notmuchmail.org>\r
  74 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  75 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  76         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  77 X-List-Received-Date: Tue, 21 Feb 2012 05:59:24 -0000\r
  78 \r
  79 Quoth Adam Wolfe Gordon on Feb 15 at  8:12 pm:\r
  80 > Use the new JSON reply format to create replies in emacs. Quote HTML\r
  81 > parts nicely by using mm-display-part to turn them into displayable\r
  82 > text, then quoting them with message-cite-original. This is very\r
  83 > useful for users who regularly receive HTML-only email.\r
  84 > \r
  85 > Use message-mode's message-cite-original function to create the\r
  86 > quoted body for reply messages. In order to make this act like the\r
  87 > existing notmuch defaults, you will need to set the following in\r
  88 > your emacs configuration:\r
  89 > \r
  90 > message-citation-line-format "On %a, %d %b %Y, %f wrote:"\r
  91 > message-citation-line-function 'message-insert-formatted-citation-line\r
  92 \r
  93 I've been thinking about this more.  message-mode's default citation\r
  94 line is really unfortunate and quite possibly insane ("writes" isn't\r
  95 even the right tense and what's up with that extra line break?).  The\r
  96 option to change this is also well hidden (as an experiment, I tried\r
  97 navigating to it through customize and couldn't figure out where it\r
  98 was, even though I knew what I was looking for).  In general, I'm a\r
  99 fan of inheriting as many options from Emacs as possible, but people\r
 100 *are* going to ask how to change this and the default setting *is*\r
 101 going to turn people off of notmuch ("What mail client do you use that\r
 102 generates those quirky citation lines?"  "I use notmuch!"  "Is that,\r
 103 like, from the 80's?").\r
 104 \r
 105 So, what about adding a notmuch customize option for selecting the\r
 106 citation line format?  It could offer a few const choices, including a\r
 107 default, sane format, plus the option to enter your own or to fall\r
 108 back to whatever message-mode is configured to do.  If we do this,\r
 109 it's probably best done in a follow-up series, but this seemed like an\r
 110 appropriate place to bring it up.\r
 111 \r
 112 > The tests have been updated to reflect the (ugly) emacs default.\r
 113 > ---\r
 114 >  emacs/notmuch-lib.el |    6 ++\r
 115 >  emacs/notmuch-mua.el |  127 +++++++++++++++++++++++++++++++++++---------------\r
 116 >  test/emacs           |    8 ++--\r
 117 >  3 files changed, 100 insertions(+), 41 deletions(-)\r