cc/1d54ed22d54fe1932e8a4292634de56a28ab80

   1 Return-Path: <cworth@cworth.org>\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 E8E38431FBF\r
   6         for <notmuch@notmuchmail.org>; Tue, 23 Feb 2010 12:56:17 -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: -3.256\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-3.256 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1.8, AWL=1.143, BAYES_00=-2.599] autolearn=ham\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 xK3r5B46nZjU; Tue, 23 Feb 2010 12:56:17 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id 48E8D431FAE;\r
  18         Tue, 23 Feb 2010 12:56:17 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id 8C9DB25427B; Tue, 23 Feb 2010 12:56:15 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: david@tethera.net, notmuch@notmuchmail.org\r
  23 In-Reply-To: <1262272660-16786-1-git-send-email-david@tethera.net>\r
  24 References: <id:1262249616-sup-3911@elise>\r
  25         <1262272660-16786-1-git-send-email-david@tethera.net>\r
  26 Date: Tue, 23 Feb 2010 12:56:15 -0800\r
  27 Message-ID: <877hq3wu1c.fsf@yoom.home.cworth.org>\r
  28 MIME-Version: 1.0\r
  29 Content-Type: multipart/signed; boundary="=-=-=";\r
  30         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  31 Subject: Re: [notmuch] [PATCH] Add an "--format=(json|text)" command-line\r
  32  option to both notmuch-search and notmuch-show.\r
  33 X-BeenThere: notmuch@notmuchmail.org\r
  34 X-Mailman-Version: 2.1.13\r
  35 Precedence: list\r
  36 List-Id: "Use and development of the notmuch mail system."\r
  37         <notmuch.notmuchmail.org>\r
  38 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  40 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  41 List-Post: <mailto:notmuch@notmuchmail.org>\r
  42 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  43 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  45 X-List-Received-Date: Tue, 23 Feb 2010 20:56:18 -0000\r
  46 \r
  47 --=-=-=\r
  48 Content-Transfer-Encoding: quoted-printable\r
  49 \r
  50 On Thu, 31 Dec 2009 11:17:40 -0400, david@tethera.net wrote:\r
  51 > This is an updated version of Scott Robinson's patch of\r
  52 > id:1261114167-sup-8228@lisa. All of the hard work is Scott's. My\r
  53 > changes to Scotts patch are=20\r
  54 >  - change option to --format from --output\r
  55 >  - make a couple of calls to function points conditional on the=20\r
  56 >    function pointers being non-null\r
  57 >  - updated the built-in documentation and the man page\r
  58 >=20\r
  59 > I also changed the copyright of json.c to Scott. Hope that's OK with\r
  60 > Scott.\r
  61 \r
  62 Well that's embarrassing. I just went through the effort of reviewing\r
  63 the old patch and basically re-constructing that same list of tasks,\r
  64 (only to find out that this updated patch was already sitting in my\r
  65 queue, but later).\r
  66 \r
  67 Anyway, very well done! I've applied this now with one change:\r
  68 \r
  69 I'm now printing "thread" instead of "id" as the JSON name for the\r
  70 field with the thread id in it.\r
  71 \r
  72 I also added a mention to the TODO file that we should fix --format=3Djson\r
  73 to not imply --entire-thread.\r
  74 \r
  75 Thanks again, David and Scott (and we hope that all the backpacking goes\r
  76 well, Scott!).\r
  77 \r
  78 =2DCarl\r
  79 \r
  80 --=-=-=\r
  81 Content-Type: application/pgp-signature\r
  82 \r
  83 -----BEGIN PGP SIGNATURE-----\r
  84 Version: GnuPG v1.4.10 (GNU/Linux)\r
  85 \r
  86 iD8DBQFLhEDv6JDdNq8qSWgRAr/XAJ90FVLYQGjaXoGmLPTdmHDvjSz0qgCeJiHK\r
  87 buGDFeQEnn903cEjgVVOSeU=\r
  88 =6RKI\r
  89 -----END PGP SIGNATURE-----\r
  90 --=-=-=--\r