c5/86a9d09f064625d837dd1ac1d7778f6c3b9146

   1 Return-Path: <amdragon@mit.edu>\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 D3F26431E84\r
   6         for <notmuch@notmuchmail.org>; Wed, 15 Feb 2012 10:17:28 -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 cSAM6RVT38Tk for <notmuch@notmuchmail.org>;\r
  16         Wed, 15 Feb 2012 10:17:25 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-8.mit.edu (DMZ-MAILSEC-SCANNER-8.MIT.EDU\r
  18         [18.7.68.37])\r
  19         by olra.theworths.org (Postfix) with ESMTP id F1238431E82\r
  20         for <notmuch@notmuchmail.org>; Wed, 15 Feb 2012 10:17:24 -0800 (PST)\r
  21 X-AuditID: 12074425-b7f4a6d0000008e0-c7-4f3bf6b402cb\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-8.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 31.97.02272.4B6FB3F4; Wed, 15 Feb 2012 13:17:24 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q1FIHNRM017457; \r
  27         Wed, 15 Feb 2012 13:17:23 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q1FIHLq3010035\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Wed, 15 Feb 2012 13:17:22 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1RxjOJ-00081j-Ek; Wed, 15 Feb 2012 13:15:43 -0500\r
  37 Date: Wed, 15 Feb 2012 13:15:43 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Tomi Ollila <tomi.ollila@iki.fi>\r
  40 Subject: Re: [PATCH 0/8] Rewrite JSON show format\r
  41 Message-ID: <20120215181543.GV27039@mit.edu>\r
  42 References: <1329240823-7856-1-git-send-email-amdragon@mit.edu>\r
  43         <m2obt0dtv8.fsf@guru.guru-group.fi>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 Content-Disposition: inline\r
  47 In-Reply-To: <m2obt0dtv8.fsf@guru.guru-group.fi>\r
  48 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  49 X-Brightmail-Tracker:\r
  50  H4sIAAAAAAAAA+NgFmpmleLIzCtJLcpLzFFi42IR4hTV1t3yzdrf4O1zYYvrN2cyW7xZOY/V\r
  51         gcnj8NeFLB7PVt1iDmCK4rJJSc3JLEst0rdL4MpoPfeaueAcb8X3+x9YGhi3c3UxcnBICJhI\r
  52         /G4U6GLkBDLFJC7cW8/WxcjFISSwj1Hi/5xVjBDOBkaJZwduQzknmSQWbT7IBOEsYZRY/OIN\r
  53         E0g/i4CqxLkzexhBbDYBDYlt+5eD2SICKhIP2tazgtjMAtIS3343g9ULCxhKHJ65mAXE5hXQ\r
  54         kVg9+R0biC0kkCpx6MdkJoi4oMTJmU9YIHq1JG78e8kEcjbInOX/OEDCnAIGEv0TJoKtEgVa\r
  55         NeXkNrYJjEKzkHTPQtI9C6F7ASPzKkbZlNwq3dzEzJzi1GTd4uTEvLzUIl0LvdzMEr3UlNJN\r
  56         jKCwZndR3cE44ZDSIUYBDkYlHl6he1b+QqyJZcWVuYcYJTmYlER573yx9hfiS8pPqcxILM6I\r
  57         LyrNSS0+xCjBwawkwqs9HSjHm5JYWZValA+TkuZgURLn1dR65yckkJ5YkpqdmlqQWgSTleHg\r
  58         UJLgvfQVqFGwKDU9tSItM6cEIc3EwQkynAdouMc3kOHFBYm5xZnpEPlTjIpS4rw/QZoFQBIZ\r
  59         pXlwvbC084pRHOgVYV4OkHYeYMqC634FNJgJaLD5CyuQwSWJCCmpBsZp1zmC19qVx0xKfy6y\r
  60         L3PR1RP32f8YVzrFfDMqeMbhlHphVeuK6Qe23o+/89gian9532n/iN5UnYNGs5rnrBG+Nud9\r
  61         /8dH7F+yZ37++rlI4k3SqhlrNTTsbx4W8tLJfHNl9WHWwgm/3yUtVmC/5/Tz8xP2FpHFApMX\r
  62         XJWrnVY2Qf7R7k2PMloilFiKMxINtZiLihMB1t1VehYDAAA=\r
  63 Cc: notmuch@notmuchmail.org\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Wed, 15 Feb 2012 18:17:29 -0000\r
  77 \r
  78 Quoth Tomi Ollila on Feb 15 at  5:00 pm:\r
  79 > On Tue, 14 Feb 2012 12:33:35 -0500, Austin Clements <amdragon@MIT.EDU> wrote:\r
  80 > > The saga continues.  As for the text format, this first shifts lots of\r
  81 > > code around without changing its semantics, then it dives in and\r
  82 > > simplifies a lot of things.  Don't be put off by the number of\r
  83 > > patches; most of them are straightforward.\r
  84 > > \r
  85 > > As an added bonus, I documented (!) the JSON format for both show and\r
  86 > > search.\r
  87 > \r
  88 > I got 2 trailing whitespace "errors" when applied (schemata) patches.\r
  89 \r
  90 Oops.  Will fix in v2.\r
  91 \r
  92 > \r
  93 > Emacs MUA works for me.\r
  94 > \r
  95 > Most json-related tests pass (for me), but I get this:\r
  96 > \r
  97 >  FAIL   notmuch-show for message with invalid From\r
  98 >         --- emacs.10.expected   2012-02-15 14:28:48.000000000 +0000\r
  99 >         +++ emacs.10.output     2012-02-15 14:28:48.000000000 +0000\r
 100 >         @@ -1,4 +1,4 @@\r
 101 >         -"Invalid " (2001-01-05) (inbox)\r
 102 >         +Invalid  From <test_suite@notmuchmail.org> <q> (2001-01-05) (inbox)\r
 103 > \r
 104 > If this works for others then the problem is in my setup\r
 105 > (I have 8 other tests failing for various other reasons)\r
 106 > \r
 107 > The patch in id:"1329240823-7856-5-git-send-email-amdragon@mit.edu"\r
 108 > changes this -- the + line is almost like it was (without that <q>).\r
 109 > Maybe the reason is within gmime-2.4.20-1.el5.rf I'm having in this\r
 110 > machine -- have to try on another...\r
 111 > \r
 112 > ...YES. other machine has (self-compiled) gmime 2.4.25 and there I got\r
 113 \r
 114 The from header is now being parsed by gmime, so if gmime's parsing of\r
 115 invalid headers has changed, it's quite possible it would affect this\r
 116 test.\r
 117 \r
 118 > All 370 tests behaved as expected (2 expected failures).\r
 119 > \r
 120 > Code LGTM.\r
 121 > \r
 122 > +1\r
 123 > \r
 124 > Tomi\r