[PATCH 1/9] lib: read "property" terms from messages.
[notmuch-archives.git] / 80 / daf64d4e0e92a66bde45a150df6e20aec79c84
1 Return-Path: <awg@xvx.ca>\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 DB9D4431FAF\r
6         for <notmuch@notmuchmail.org>; Sun,  5 Feb 2012 22:27:18 -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 x90mRUWQFEOM for <notmuch@notmuchmail.org>;\r
16         Sun,  5 Feb 2012 22:27:18 -0800 (PST)\r
17 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
18         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id 0AACD431FAE\r
21         for <notmuch@notmuchmail.org>; Sun,  5 Feb 2012 22:27:17 -0800 (PST)\r
22 Received: by bke11 with SMTP id 11so5168695bke.26\r
23         for <notmuch@notmuchmail.org>; Sun, 05 Feb 2012 22:27:15 -0800 (PST)\r
24 MIME-Version: 1.0\r
25 Received: by 10.204.130.89 with SMTP id r25mr7568699bks.49.1328509635078; Sun,\r
26         05 Feb 2012 22:27:15 -0800 (PST)\r
27 Sender: awg@xvx.ca\r
28 Received: by 10.204.104.13 with HTTP; Sun, 5 Feb 2012 22:27:14 -0800 (PST)\r
29 X-Originating-IP: [96.52.216.56]\r
30 In-Reply-To: <20120206034413.GO10898@mit.edu>\r
31 References: <1326995217-27423-1-git-send-email-awg+notmuch@xvx.ca>\r
32         <1326995217-27423-3-git-send-email-awg+notmuch@xvx.ca>\r
33         <20120206034413.GO10898@mit.edu>\r
34 Date: Sun, 5 Feb 2012 23:27:14 -0700\r
35 X-Google-Sender-Auth: nq553jjXy9Js6O6Y7CcBKxcFcq8\r
36 Message-ID:\r
37  <CAMoJFUtw=HD7gPKvuYSO3OhZ97kaMEroea6UmkQwYWMBc8WS_A@mail.gmail.com>\r
38 Subject: Re: [PATCH v3 2/5] reply: Add a JSON reply format.\r
39 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
40 To: Austin Clements <amdragon@mit.edu>\r
41 Content-Type: text/plain; charset=ISO-8859-1\r
42 Content-Transfer-Encoding: quoted-printable\r
43 Cc: notmuch@notmuchmail.org\r
44 X-BeenThere: notmuch@notmuchmail.org\r
45 X-Mailman-Version: 2.1.13\r
46 Precedence: list\r
47 List-Id: "Use and development of the notmuch mail system."\r
48         <notmuch.notmuchmail.org>\r
49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
52 List-Post: <mailto:notmuch@notmuchmail.org>\r
53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
56 X-List-Received-Date: Mon, 06 Feb 2012 06:27:19 -0000\r
57 \r
58 On Sun, Feb 5, 2012 at 20:44, Austin Clements <amdragon@mit.edu> wrote:\r
59 > Sorry for coming late to the party. =A0I really like this idea, but it\r
60 > seems like your implementation is duplicating a lot of the work of\r
61 > notmuch show. =A0This makes me wonder if it would be better to return\r
62 > reply header information in the JSON (which is definitely the way to\r
63 > go) but to fetch the part body from the UI via show (and maybe reuse\r
64 > some of the show-mode logic, if it makes sense to do so). =A0If this has\r
65 > already been discussed, just point me at the thread and I'll catch\r
66 > myself up.\r
67 \r
68 Thanks for taking a look. Dmitry noted on IRC that inlining the HTML\r
69 in JSON could cause issues with non-UTF8 character sets. Right now I'm\r
70 working on essentially what you've suggested - having the CLI produce\r
71 only headers, and then using show to get the quotable body.\r
72 \r
73 Something else that was mentioned on IRC is using some of the notmuch\r
74 show logic to produce the show JSON format as part of reply. I looked\r
75 into this, but it would take some serious refactoring (to make the\r
76 show JSON stuff accessible to reply), and since emacs will need to end\r
77 up calling show anyway, I'm not sure it's worth it. I do like the idea\r
78 of different CLI commands being able to produce standardized formats\r
79 through some shared interface, I'm just not sure it's necessary here,\r
80 and not sure what the interface should look like.\r