49/bb3fe6f10fcf624887243d16cd29ab5696ac2a

   1 Return-Path: <jrollins@finestructure.net>\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 27A09431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon, 18 Jun 2012 10:15:09 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 8Xe1dAM26OhX for <notmuch@notmuchmail.org>;\r
  16         Mon, 18 Jun 2012 10:15:07 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 2F833431FAE\r
  20         for <notmuch@notmuchmail.org>; Mon, 18 Jun 2012 10:15:07 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 75C452E50BE9;\r
  23         Mon, 18 Jun 2012 10:15:04 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (unknown [76.89.192.57])\r
  26         (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id 7B3892E50D6C;\r
  28         Mon, 18 Jun 2012 10:15:01 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id B793039B; Mon, 18 Jun 2012 10:15:00 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: Notmuch Pick\r
  34 In-Reply-To: <87395ump0d.fsf@qmul.ac.uk>\r
  35 References: <87395ump0d.fsf@qmul.ac.uk>\r
  36 User-Agent: Notmuch/0.13.2+53~g1567997 (http://notmuchmail.org) Emacs/23.4.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Mon, 18 Jun 2012 10:14:58 -0700\r
  39 Message-ID: <87zk80ilt9.fsf@servo.finestructure.net>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Mon, 18 Jun 2012 17:15:09 -0000\r
  56 \r
  57 --=-=-=\r
  58 \r
  59 On Sat, Jun 16 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  60 > Since I have had various requests for notmuch pick\r
  61 > (id:"1329096015-8078-2-git-send-email-markwalters1009@gmail.com") so I\r
  62 > have started a git repository at\r
  63 > git://github.com/markwalters1009/notmuch.git\r
  64 >\r
  65 > The branch pick-6 is the current version. My intention is to start a new\r
  66 > branch each time I rebase on to current master but this may change. (I\r
  67 > suggest that people do not rely on a consistent history for this repository)\r
  68 \r
  69 Hey, Mark.  I took a look at this and I have a couple of comments:\r
  70 \r
  71 726e11aff69e10499a9855e0ac2f15e518985c1f\r
  72     cli: notmuch-show with framing newlines between threads in JSON.\r
  73 \r
  74 This patch introduces a change in the json output format, but there is\r
  75 no subsequent update of the test suite, so it's causing a lot of test\r
  76 failures.  Obviously this needs to be fixed, but it would probably be\r
  77 nice to include a couple of other tests for the pick output itself.  At\r
  78 the very least a sanity test to check that it's working at all would be\r
  79 sufficient initially.\r
  80 \r
  81 Would it also be useful to make this same change for the search out, for\r
  82 consistency?  I notice the search output now uses newlines between all\r
  83 fields, which should help for asynchronous processing, but it might be\r
  84 nice to put newline separators between the initial and final brackets as\r
  85 well.\r
  86 \r
  87 df97df62b70b884a1cd367360ed6ff7eda0e8af6\r
  88     cli: add --headers_only option to notmuch-show.c\r
  89 \r
  90 Your comment in this patch is very interesting:\r
  91 \r
  92     This is used by notmuch-pick.el (although not needed) because it gives a\r
  93     speed-up of at least a factor of a two; moreover it reduces the memory\r
  94     usage in emacs hugely.\r
  95 \r
  96 The only difference between the regular show json output and the\r
  97 --headers-only output, as far as I can tell, is the presence of the\r
  98 content of text/plain parts if they exist in the message.  We previously\r
  99 had a discussion about the show output not including any part bodies at\r
 100 all, but we decided that the inclusion of text/plain bodies shouldn't\r
 101 affect anything, so why not include them.  If they actually do, then I\r
 102 argue we should just move to having show json not include any body parts\r
 103 at all by default, and just have them be retrieved individually like we\r
 104 do currently for non-text/plain parts.  This would make things cleaner,\r
 105 and would get rid of the need to have this extra option, which really\r
 106 doesn't produce a significantly different output.\r
 107 \r
 108 jamie.\r
 109 \r
 110 --=-=-=\r
 111 Content-Type: application/pgp-signature\r
 112 \r
 113 -----BEGIN PGP SIGNATURE-----\r
 114 Version: GnuPG v1.4.12 (GNU/Linux)\r
 115 \r
 116 iQIcBAEBCAAGBQJP32ISAAoJEO00zqvie6q8JAcP+wYJihHLRGZ4Vd6NrOk2AqxU\r
 117 6zksedeFhMpZKAONSpIGP68REo/BKT5sh8C4bIX3kLoXfLBIarnfOTYXCHxRi8SH\r
 118 nc4zAXxMDvR3Wn/2FhD4mvlwo9ywfB+T8uKxfiCyk35UiSADd8MQC1Mun9AGBn9t\r
 119 a8jlzqLqLIO3L5tXbJPNDUl6/EWJgEBRrkyL3I2lhWcCi4KKLNwBLO4lp3ihvGhC\r
 120 1qX5fWmvgAJaElmkHGCR3rZlIH5mFHo6Riq56bwI1CdE5a9YxrLqDix48CG30NLV\r
 121 +YGxNfGQir9e4PSH2659xMvRYvBXsGW8gV8CmhuaAsft2PfquDFH8aglCeq+8sCT\r
 122 +M3Y6AtmIYfc0qiSTksX8URnwUi+SdZ4s+3RYTzQDndAk/nJM1sSW7NsmBydUSYj\r
 123 B59eBXl4YFOQLTVk+MjPscU2TRUOSJ76pzUiDk8G+jdw10VFTgoZW92Pkeg4Tsgp\r
 124 zGMakUREYkC5nm+lLLj154k7YUu3eeXIscCrufS90Ezpy3IBkNHIu4HF1dBFYZ0v\r
 125 ywdVSwaC3I4D4Pa49nCzCX6S/pwk+1kIZf4Klg7dYmwYPrh2QkCLEZOCmx359v0c\r
 126 IGx5sqeCDVQEOkrp10m/QfQg8qjD9PqUAQ1CeWV5McZXezyD1UcbGlzLodd//QKe\r
 127 It3lBMIDPQNvz0QXch5h\r
 128 =xfcy\r
 129 -----END PGP SIGNATURE-----\r
 130 --=-=-=--\r