Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original
authorAustin Clements <amdragon@MIT.EDU>
Wed, 22 Feb 2012 16:57:50 +0000 (11:57 +1900)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:44:49 +0000 (09:44 -0800)
4e/8c89a0643f311a3315bc5f95dfc92ce935bd11 [new file with mode: 0644]

diff --git a/4e/8c89a0643f311a3315bc5f95dfc92ce935bd11 b/4e/8c89a0643f311a3315bc5f95dfc92ce935bd11
new file mode 100644 (file)
index 0000000..c620005
--- /dev/null
@@ -0,0 +1,147 @@
+Return-Path: <amdragon@mit.edu>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id BDD85431FB6\r
+       for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 08:57:57 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id woBFToks-w+u for <notmuch@notmuchmail.org>;\r
+       Wed, 22 Feb 2012 08:57:54 -0800 (PST)\r
+Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
+       [18.9.25.12])\r
+       by olra.theworths.org (Postfix) with ESMTP id E86C6431FAE\r
+       for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 08:57:53 -0800 (PST)\r
+X-AuditID: 1209190c-b7fad6d000000920-e2-4f451e91a5b0\r
+Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
+       by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
+       id C4.E1.02336.19E154F4; Wed, 22 Feb 2012 11:57:53 -0500 (EST)\r
+Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
+       by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id q1MGvquv012300; \r
+       Wed, 22 Feb 2012 11:57:53 -0500\r
+Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
+       (authenticated bits=0)\r
+       (User authenticated as amdragon@ATHENA.MIT.EDU)\r
+       by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q1MGvoue012234\r
+       (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
+       Wed, 22 Feb 2012 11:57:52 -0500 (EST)\r
+Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
+       (envelope-from <amdragon@mit.edu>)\r
+       id 1S0FVm-0004xX-6g; Wed, 22 Feb 2012 11:57:50 -0500\r
+Date: Wed, 22 Feb 2012 11:57:50 -0500\r
+From: Austin Clements <amdragon@MIT.EDU>\r
+To: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
+Subject: Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and\r
+       message-cite-original\r
+Message-ID: <20120222165750.GC30513@mit.edu>\r
+References: <1329361957-28493-1-git-send-email-awg+notmuch@xvx.ca>\r
+       <1329361957-28493-8-git-send-email-awg+notmuch@xvx.ca>\r
+       <20120221055920.GA30513@mit.edu>\r
+       <CAMoJFUvo6CZjeNSqeoR_HViKgxz-WS9Tx8qfOgaVgYbmmBWHiw@mail.gmail.com>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=iso-8859-1\r
+Content-Disposition: inline\r
+Content-Transfer-Encoding: 8bit\r
+In-Reply-To:\r
+ <CAMoJFUvo6CZjeNSqeoR_HViKgxz-WS9Tx8qfOgaVgYbmmBWHiw@mail.gmail.com>\r
+User-Agent: Mutt/1.5.21 (2010-09-15)\r
+X-Brightmail-Tracker:\r
+ H4sIAAAAAAAAA+NgFprNKsWRmVeSWpSXmKPExsUixCmqrDtRztXf4NQCcYsje2axW1y/OZPZ\r
+       gcnj2apbzB5NPxazBjBFcdmkpOZklqUW6dslcGXs2fCbtWCVdMXy18vZGhgbxLoYOTgkBEwk\r
+       Vrxy7WLkBDLFJC7cW8/WxcjFISSwj1Gif+olFghnA6NEz4keRgjnJJPE4+s3WCGcJYwSy7bt\r
+       YwPpZxFQlbg0YREjiM0moCGxbf9yMFtEQEvix/qvrCA2s4C0xLffzUwgtrBAnMS371fYQc7g\r
+       FdCRWH7THmLmU0aJh19mM4PU8AoISpyc+YQFoldHYufWO2wg9SBzlv/jgAjLSzRvhSjnFAiU\r
+       mPp/Ddg5ogIqElNObmObwCg8C8mkWUgmzUKYNAvJpAWMLKsYZVNyq3RzEzNzilOTdYuTE/Py\r
+       Uot0DfVyM0v0UlNKNzGC40CSZwfjm4NKhxgFOBiVeHgXsbv6C7EmlhVX5h5ilORgUhLlFQFG\r
+       kRBfUn5KZUZicUZ8UWlOavEhRgkOZiUR3vZTLv5CvCmJlVWpRfkwKWkOFiVxXhWtd35CAumJ\r
+       JanZqakFqUUwWRkODiUJ3ueyQEMFi1LTUyvSMnNKENJMHJwgw3mAhnOALOYtLkjMLc5Mh8if\r
+       YlSUEuf9BNIsAJLIKM2D64WlqVeM4kCvCPMeAqniAaY4uO5XQIOZgAa3/HcEGVySiJCSamBM\r
+       UzozddPK4C8csUqTvC44erk+lfUPljvZENyecG7Z0jOc09lmTLL5W7naQ9npkXmMaIyuX+/U\r
+       t+cUcjmP6cpPvcZokPd818ubcn+cd+UWbpNeu/vc3j+erzcy17Fe5qt3287Fxzb3/ZNI5m18\r
+       07cWH2n/++rZ5f6ppYlbegOC7QS4GLiyw4WVWIozEg21mIuKEwELFylJLgMAAA==\r
+Cc: notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Wed, 22 Feb 2012 16:57:57 -0000\r
+\r
+Quoth Adam Wolfe Gordon on Feb 21 at  9:49 am:\r
+> On Mon, Feb 20, 2012 at 22:59, Austin Clements <amdragon@mit.edu> wrote:\r
+> > I've been thinking about this more.  message-mode's default citation\r
+> > line is really unfortunate and quite possibly insane ("writes" isn't\r
+> > even the right tense and what's up with that extra line break?).  The\r
+> > option to change this is also well hidden (as an experiment, I tried\r
+> > navigating to it through customize and couldn't figure out where it\r
+> > was, even though I knew what I was looking for).  In general, I'm a\r
+> > fan of inheriting as many options from Emacs as possible, but people\r
+> > *are* going to ask how to change this and the default setting *is*\r
+> > going to turn people off of notmuch ("What mail client do you use that\r
+> > generates those quirky citation lines?"  "I use notmuch!"  "Is that,\r
+> > like, from the 80's?").\r
+> \r
+> Agreed. It's a really unfortunate default.\r
+> \r
+> > So, what about adding a notmuch customize option for selecting the\r
+> > citation line format?  It could offer a few const choices, including a\r
+> > default, sane format, plus the option to enter your own or to fall\r
+> > back to whatever message-mode is configured to do.  If we do this,\r
+> > it's probably best done in a follow-up series, but this seemed like an\r
+> > appropriate place to bring it up.\r
+> \r
+> I think there are two options, which have been discussed a bit before [1]:\r
+\r
+Ah, interesting.  I hadn't been following this series at that point.\r
+\r
+> 1) Wrap the citation format with a notmuch customization variable,\r
+> notmuch-citation-line-format or somesuch. Then set the\r
+> message-citation-line-format before calling message-cite-original.\r
+> \r
+> 2) Have notmuch load a user config file (~/.notmuch.el or something)\r
+> on startup, and provide a default file that sets nice defaults for\r
+> things like message-citation-line-format. The default file could even\r
+> be constructed on first run, such that if the user has already\r
+> customized some things (like message-citation-line-format) we can keep\r
+> their settings.\r
+> \r
+> Option 2 is obviously more work, but I think it's the right way to go,\r
+> at least in the long run. In addition to giving a place to provide\r
+> nice defaults for non-notmuch variables, it gives the user a nice\r
+> place to specify notmuch-specific config. For example, I use\r
+> completely separate init files for notmuch and other emacs usage, and\r
+> having a notmuch config file would let me get away from this slightly\r
+> kludgey setup.\r
+\r
+That is an intriguing idea.  My main concern would be\r
+forwards-compatibility.  When we find some other variable that we want\r
+to add to this file, what do we do with already-generated\r
+.notmuch.els?\r
+\r
+I wouldn't dismiss option 1, though.  From a user's perspective, it is\r
+visible that they're using message-mode to edit message drafts and\r
+hence natural that they would customize message-mode to control\r
+editing behavior, but they create a reply by pressing 'r' in a\r
+*notmuch* buffer, which naturally associates the operation with\r
+notmuch, not message-mode.  I would go so far as to say reply's use of\r
+message-mode is an implementation detail.  Hence, I would never expect\r
+an uninformed user to guess that they have to go to message-mode to\r
+customize the citation format.\r
+\r
+> In either case, this can probably come as a separate patch series, but\r
+> it is good to start discussing it here.\r
+> \r
+> [1] id:"m2mx9i3onw.fsf@wal122.wireless-pennnet.upenn.edu"\r