Re: [notmuch] [PATCH] Add an "--format=(json|text)" command-line option to both notmu...

diff --git a/ad/ea517065e6168fe30b13401084ab7c690d7413 b/ad/ea517065e6168fe30b13401084ab7c690d7413
new file mode 100644 (file)
index 0000000..85e9698
--- /dev/null
+++ b/ad/ea517065e6168fe30b13401084ab7c690d7413
@@ -0,0 +1,72 @@
+Return-Path: <scott@quadhome.com>\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 42748431FBC\r
+       for <notmuch@notmuchmail.org>; Wed, 10 Feb 2010 09:54:49 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.246\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.246 tagged_above=-999 required=5\r
+       tests=[AWL=-0.247, BAYES_50=0.001] autolearn=ham\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 bNcYRH47tI+d for <notmuch@notmuchmail.org>;\r
+       Wed, 10 Feb 2010 09:54:48 -0800 (PST)\r
+Received: from julia.quadhome.com (quadhome.xen.prgmr.com [68.68.99.250])\r
+       by olra.theworths.org (Postfix) with ESMTP id 9F865431FAE\r
+       for <notmuch@notmuchmail.org>; Wed, 10 Feb 2010 09:54:48 -0800 (PST)\r
+Received: from elise.quadhome.com (150.59.148.190.dsl.intelnet.net.gt\r
+       [190.148.59.150])\r
+       by julia.quadhome.com (Postfix) with ESMTPSA id B77DE5FA92\r
+       for <notmuch@notmuchmail.org>; Wed, 10 Feb 2010 17:54:47 +0000 (UTC)\r
+Content-Type: text/plain; charset=UTF-8\r
+From: Scott Robinson <scott@quadhome.com>\r
+To: notmuch <notmuch@notmuchmail.org>\r
+In-reply-to: <87mxzhjj0s.fsf@rocinante.cs.unb.ca>\r
+References: <id:1262249616-sup-3911@elise>\r
+       <1262272660-16786-1-git-send-email-david@tethera.net>\r
+       <87mxzhjj0s.fsf@rocinante.cs.unb.ca>\r
+Date: Wed, 10 Feb 2010 11:54:45 -0600\r
+Message-Id: <1265824234-sup-8219@elise>\r
+User-Agent: Sup/0.10.2\r
+Content-Transfer-Encoding: 8bit\r
+Subject: Re: [notmuch] [PATCH] Add an "--format=(json|text)" command-line\r
+       option to both notmuch-search and notmuch-show.\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, 10 Feb 2010 17:54:49 -0000\r
+\r
+Excerpts from David Bremner's message of Wed Feb 10 07:51:47 -0600 2010:\r
+> Dear Scott and other interested parties;\r
+> \r
+> As I play with json output a bit more, it seems like having\r
+> --entire-thread optional would be useful for quite a few applications.\r
+> For example, if I want to get the From address for a given message-ID, I\r
+> with the current setup I have to post process the output. This is\r
+> doable, but I wonder if the better option isn't to make --entire-thread\r
+> optional for --format=json as well. \r
+> \r
+> Any thoughts?\r
+\r
+I made --entire-thread forced becuase it kept the JSON patch minimal. The way\r
+the thread structure is calculated is closely tied into the message tree\r
+recursion. It would have required a much more significant refactoring to get\r
+both ways; and, I wanted the patch to have as few barriers to being merged.\r
+\r
+However, as patch still hasn't been merged and probably needs to be rebased,\r
+there doesn't seem to be any good reason anymore to avoid the work.\r
+\r
+Except, I can't do it. As I noted in an earlier e-mail, I have to abandon this\r
+piece of code. I'm a bit busy backpacking through Central America. :-)\r