4e/8c89a0643f311a3315bc5f95dfc92ce935bd11

   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 BDD85431FB6\r
   6         for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 08:57:57 -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 woBFToks-w+u for <notmuch@notmuchmail.org>;\r
  16         Wed, 22 Feb 2012 08:57:54 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
  18         [18.9.25.12])\r
  19         by olra.theworths.org (Postfix) with ESMTP id E86C6431FAE\r
  20         for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 08:57:53 -0800 (PST)\r
  21 X-AuditID: 1209190c-b7fad6d000000920-e2-4f451e91a5b0\r
  22 Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
  23         by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id C4.E1.02336.19E154F4; Wed, 22 Feb 2012 11:57:53 -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 q1MGvquv012300; \r
  27         Wed, 22 Feb 2012 11:57:53 -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 q1MGvoue012234\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Wed, 22 Feb 2012 11:57:52 -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 1S0FVm-0004xX-6g; Wed, 22 Feb 2012 11:57:50 -0500\r
  37 Date: Wed, 22 Feb 2012 11:57:50 -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: <20120222165750.GC30513@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         <20120221055920.GA30513@mit.edu>\r
  46         <CAMoJFUvo6CZjeNSqeoR_HViKgxz-WS9Tx8qfOgaVgYbmmBWHiw@mail.gmail.com>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=iso-8859-1\r
  49 Content-Disposition: inline\r
  50 Content-Transfer-Encoding: 8bit\r
  51 In-Reply-To:\r
  52  <CAMoJFUvo6CZjeNSqeoR_HViKgxz-WS9Tx8qfOgaVgYbmmBWHiw@mail.gmail.com>\r
  53 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  54 X-Brightmail-Tracker:\r
  55  H4sIAAAAAAAAA+NgFprNKsWRmVeSWpSXmKPExsUixCmqrDtRztXf4NQCcYsje2axW1y/OZPZ\r
  56         gcnj2apbzB5NPxazBjBFcdmkpOZklqUW6dslcGXs2fCbtWCVdMXy18vZGhgbxLoYOTgkBEwk\r
  57         Vrxy7WLkBDLFJC7cW8/WxcjFISSwj1Gif+olFghnA6NEz4keRgjnJJPE4+s3WCGcJYwSy7bt\r
  58         YwPpZxFQlbg0YREjiM0moCGxbf9yMFtEQEvix/qvrCA2s4C0xLffzUwgtrBAnMS371fYQc7g\r
  59         FdCRWH7THmLmU0aJh19mM4PU8AoISpyc+YQFoldHYufWO2wg9SBzlv/jgAjLSzRvhSjnFAiU\r
  60         mPp/Ddg5ogIqElNObmObwCg8C8mkWUgmzUKYNAvJpAWMLKsYZVNyq3RzEzNzilOTdYuTE/Py\r
  61         Uot0DfVyM0v0UlNKNzGC40CSZwfjm4NKhxgFOBiVeHgXsbv6C7EmlhVX5h5ilORgUhLlFQFG\r
  62         kRBfUn5KZUZicUZ8UWlOavEhRgkOZiUR3vZTLv5CvCmJlVWpRfkwKWkOFiVxXhWtd35CAumJ\r
  63         JanZqakFqUUwWRkODiUJ3ueyQEMFi1LTUyvSMnNKENJMHJwgw3mAhnOALOYtLkjMLc5Mh8if\r
  64         YlSUEuf9BNIsAJLIKM2D64WlqVeM4kCvCPMeAqniAaY4uO5XQIOZgAa3/HcEGVySiJCSamBM\r
  65         UzozddPK4C8csUqTvC44erk+lfUPljvZENyecG7Z0jOc09lmTLL5W7naQ9npkXmMaIyuX+/U\r
  66         t+cUcjmP6cpPvcZokPd818ubcn+cd+UWbpNeu/vc3j+erzcy17Fe5qt3287Fxzb3/ZNI5m18\r
  67         07cWH2n/++rZ5f6ppYlbegOC7QS4GLiyw4WVWIozEg21mIuKEwELFylJLgMAAA==\r
  68 Cc: notmuch@notmuchmail.org\r
  69 X-BeenThere: notmuch@notmuchmail.org\r
  70 X-Mailman-Version: 2.1.13\r
  71 Precedence: list\r
  72 List-Id: "Use and development of the notmuch mail system."\r
  73         <notmuch.notmuchmail.org>\r
  74 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  76 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  77 List-Post: <mailto:notmuch@notmuchmail.org>\r
  78 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  79 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  80         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  81 X-List-Received-Date: Wed, 22 Feb 2012 16:57:57 -0000\r
  82 \r
  83 Quoth Adam Wolfe Gordon on Feb 21 at  9:49 am:\r
  84 > On Mon, Feb 20, 2012 at 22:59, Austin Clements <amdragon@mit.edu> wrote:\r
  85 > > I've been thinking about this more.  message-mode's default citation\r
  86 > > line is really unfortunate and quite possibly insane ("writes" isn't\r
  87 > > even the right tense and what's up with that extra line break?).  The\r
  88 > > option to change this is also well hidden (as an experiment, I tried\r
  89 > > navigating to it through customize and couldn't figure out where it\r
  90 > > was, even though I knew what I was looking for).  In general, I'm a\r
  91 > > fan of inheriting as many options from Emacs as possible, but people\r
  92 > > *are* going to ask how to change this and the default setting *is*\r
  93 > > going to turn people off of notmuch ("What mail client do you use that\r
  94 > > generates those quirky citation lines?"  "I use notmuch!"  "Is that,\r
  95 > > like, from the 80's?").\r
  96 > \r
  97 > Agreed. It's a really unfortunate default.\r
  98 > \r
  99 > > So, what about adding a notmuch customize option for selecting the\r
 100 > > citation line format?  It could offer a few const choices, including a\r
 101 > > default, sane format, plus the option to enter your own or to fall\r
 102 > > back to whatever message-mode is configured to do.  If we do this,\r
 103 > > it's probably best done in a follow-up series, but this seemed like an\r
 104 > > appropriate place to bring it up.\r
 105 > \r
 106 > I think there are two options, which have been discussed a bit before [1]:\r
 107 \r
 108 Ah, interesting.  I hadn't been following this series at that point.\r
 109 \r
 110 > 1) Wrap the citation format with a notmuch customization variable,\r
 111 > notmuch-citation-line-format or somesuch. Then set the\r
 112 > message-citation-line-format before calling message-cite-original.\r
 113 > \r
 114 > 2) Have notmuch load a user config file (~/.notmuch.el or something)\r
 115 > on startup, and provide a default file that sets nice defaults for\r
 116 > things like message-citation-line-format. The default file could even\r
 117 > be constructed on first run, such that if the user has already\r
 118 > customized some things (like message-citation-line-format) we can keep\r
 119 > their settings.\r
 120 > \r
 121 > Option 2 is obviously more work, but I think it's the right way to go,\r
 122 > at least in the long run. In addition to giving a place to provide\r
 123 > nice defaults for non-notmuch variables, it gives the user a nice\r
 124 > place to specify notmuch-specific config. For example, I use\r
 125 > completely separate init files for notmuch and other emacs usage, and\r
 126 > having a notmuch config file would let me get away from this slightly\r
 127 > kludgey setup.\r
 128 \r
 129 That is an intriguing idea.  My main concern would be\r
 130 forwards-compatibility.  When we find some other variable that we want\r
 131 to add to this file, what do we do with already-generated\r
 132 .notmuch.els?\r
 133 \r
 134 I wouldn't dismiss option 1, though.  From a user's perspective, it is\r
 135 visible that they're using message-mode to edit message drafts and\r
 136 hence natural that they would customize message-mode to control\r
 137 editing behavior, but they create a reply by pressing 'r' in a\r
 138 *notmuch* buffer, which naturally associates the operation with\r
 139 notmuch, not message-mode.  I would go so far as to say reply's use of\r
 140 message-mode is an implementation detail.  Hence, I would never expect\r
 141 an uninformed user to guess that they have to go to message-mode to\r
 142 customize the citation format.\r
 143 \r
 144 > In either case, this can probably come as a separate patch series, but\r
 145 > it is good to start discussing it here.\r
 146 > \r
 147 > [1] id:"m2mx9i3onw.fsf@wal122.wireless-pennnet.upenn.edu"\r