11/380b5a3c28916293251634718be360deadae80

   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 9834F431FBC\r
   6         for <notmuch@notmuchmail.org>; Tue,  6 Mar 2012 10:43:11 -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 q10vUcaOeGqM for <notmuch@notmuchmail.org>;\r
  16         Tue,  6 Mar 2012 10:43:09 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-2.mit.edu (DMZ-MAILSEC-SCANNER-2.MIT.EDU\r
  18         [18.9.25.13])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 474C8431FAE\r
  20         for <notmuch@notmuchmail.org>; Tue,  6 Mar 2012 10:43:09 -0800 (PST)\r
  21 X-AuditID: 1209190d-b7fbf6d0000008ba-2b-4f565abc6f43\r
  22 Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
  23         by dmz-mailsec-scanner-2.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id E7.31.02234.CBA565F4; Tue,  6 Mar 2012 13:43:08 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id q26Ih7Ep015091; \r
  27         Tue, 6 Mar 2012 13:43:08 -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 q26Ih6Hu014839\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Tue, 6 Mar 2012 13:43:07 -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 1S4zLm-0001qo-Bn; Tue, 06 Mar 2012 13:43:06 -0500\r
  37 Date: Tue, 6 Mar 2012 13:43:06 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
  40 Subject: Re: [PATCH 5/5] show: Convert raw format to the new self-recursive\r
  41         style\r
  42 Message-ID: <20120306184306.GA5457@mit.edu>\r
  43 References: <1330752025-2542-1-git-send-email-amdragon@mit.edu>\r
  44         <1330752025-2542-6-git-send-email-amdragon@mit.edu>\r
  45         <87zkbxqr09.fsf@servo.finestructure.net>\r
  46 MIME-Version: 1.0\r
  47 Content-Type: text/plain; charset=us-ascii\r
  48 Content-Disposition: inline\r
  49 In-Reply-To: <87zkbxqr09.fsf@servo.finestructure.net>\r
  50 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  51 X-Brightmail-Tracker:\r
  52  H4sIAAAAAAAAA+NgFmpkleLIzCtJLcpLzFFi42IR4hRV1t0TFeZv8Gm1vsWefV4W12/OZHZg\r
  53         8rh7msvj2apbzAFMUVw2Kak5mWWpRfp2CVwZ1xd+Yy54IVpx5V5BA+N+wS5GTg4JAROJVe/P\r
  54         M0HYYhIX7q1n62Lk4hAS2McosWnmF3YIZz2jxNbfV1hBqoQETjBJLH3rD5FYwiixbeF2FpAE\r
  55         i4CKxM2mbkYQm01AQ2Lb/uVgtoiAmUTPlz9gNrOAtMS3381g64QFgiU6//8As3kFtCX+L/rP\r
  56         DDF0AaPE+ZdToBKCEidnPmGBaNaSuPHvJVCcA2zQ8n8cIGFOAVOJ9vXfwUpEgW6YcnIb2wRG\r
  57         oVlIumch6Z6F0L2AkXkVo2xKbpVubmJmTnFqsm5xcmJeXmqRrpFebmaJXmpK6SZGcEhL8u5g\r
  58         fHdQ6RCjAAejEg9vlW6YvxBrYllxZe4hRkkOJiVR3pvhQCG+pPyUyozE4oz4otKc1OJDjBIc\r
  59         zEoivN/eh/oL8aYkVlalFuXDpKQ5WJTEeVW13vkJCaQnlqRmp6YWpBbBZGU4OJQkeC2AsSsk\r
  60         WJSanlqRlplTgpBm4uAEGc4DNNwRpIa3uCAxtzgzHSJ/ilFRSpz3WSRQQgAkkVGaB9cLSzmv\r
  61         GMWBXhHmFQNp5wGmK7juV0CDmYAGtykHgwwuSURISTUwLmhzXbum9e/1t4cKHI2fP9gyUfR6\r
  62         gE3AjRuhnUF6y5QCKzw7hV7k1n5LF3/8e9oODe32Mtf4V20rf9u3rhPnT7VOu5nVp+sr+vq6\r
  63         s4aO2Gm55gv9Vxuse7bVh/9Walwxze+glNu/1mm/2tdq773F9ZP7QKzu1bo5K857zRF4mxKf\r
  64         fqK9Sd9AiaU4I9FQi7moOBEAIImrrhQDAAA=\r
  65 Cc: notmuch@notmuchmail.org\r
  66 X-BeenThere: notmuch@notmuchmail.org\r
  67 X-Mailman-Version: 2.1.13\r
  68 Precedence: list\r
  69 List-Id: "Use and development of the notmuch mail system."\r
  70         <notmuch.notmuchmail.org>\r
  71 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  72         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  73 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  74 List-Post: <mailto:notmuch@notmuchmail.org>\r
  75 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  76 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  77         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  78 X-List-Received-Date: Tue, 06 Mar 2012 18:43:11 -0000\r
  79 \r
  80 TL;DR: I had a MIMEpiphany and I'm sending a new version of this\r
  81 series that I think will address your concerns.\r
  82 \r
  83 Quoth Jameson Graef Rollins on Mar 03 at  2:05 pm:\r
  84 > Hey, Austin.  As always, thank you so much for your hard work on this\r
  85 > rewrite.  It looks like things are definitely moving the right\r
  86 > direction.\r
  87 > \r
  88 > I haven't done a full review of this patch set, and I've been pretty out\r
  89 > of the loop on this stuff recently, but I do notice that there are some\r
  90 > changes to the tests that don't look right to me.\r
  91 > \r
  92 > On Sat,  3 Mar 2012 00:20:25 -0500, Austin Clements <amdragon@MIT.EDU> wrote:\r
  93 > > This is fully compatible for root and leaf parts, but drops support\r
  94 > > for interior parts.  Showing interior parts in raw has always been\r
  95 > > braindead broken, so I don't think anyone will miss this.  Tests have\r
  96 > > been updated to reflect this.\r
  97 > \r
  98 > I think I'm confused about this "drop support for interior parts".  What\r
  99 > constitutes an "interior part"?  Aren't all parts interior?  It looks\r
 100 > From the patch that maybe you're referring specifically to rfc822 parts?\r
 101 \r
 102 Interior parts are any part that isn't a root part (the whole message)\r
 103 or a leaf part.\r
 104 \r
 105 I had originally planned to simply deprecate raw in favor of two new\r
 106 formats: "body", which outputted just bodies with transfer decoding\r
 107 (which, for rfc822 parts, would include the attached message headers,\r
 108 but not the envelope headers) and "source", which outputted full parts\r
 109 with headers.  I had been thinking that "source" would be necessary\r
 110 for fetching the entire message and "body" for fetching attachments\r
 111 and other leaf parts, but then it dawned on me that even trivial\r
 112 messages have two parts already: part 0 is an rfc822 part representing\r
 113 the whole message and part 1 is the body.  Hence, modulo specialty\r
 114 uses like crypto, "body" was already sufficient for both uses *and*\r
 115 compatible with "raw" for root and leaf parts.\r
 116 \r
 117 I've updated my series to reflect this.  Raw now works for any part.\r
 118 \r
 119 > >  test_begin_subtest "--format=raw --part=4, rfc822's html part"\r
 120 > > -notmuch show --format=raw --part=4 'id:87liy5ap00.fsf@yoom.home.cworth.org' >OUTPUT\r
 121 > > +notmuch show --format=raw --part=4 'id:87liy5ap00.fsf@yoom.home.cworth.org' >&OUTPUT\r
 122 > >  cat <<EOF >EXPECTED\r
 123 > > -<p>This is an embedded message, with a multipart/alternative part.</p>\r
 124 > > -This is an embedded message, with a multipart/alternative part.\r
 125 > > +Error: Raw only supports root and leaf parts\r
 126 > >  EOF\r
 127 > >  test_expect_equal_file OUTPUT EXPECTED\r
 128 > \r
 129 > Maybe this is ultimately a limitation of what we can expect the raw\r
 130 > formatter to do, but isn't this a leaf part?\r
 131 \r
 132 Actually that was a typo in the test name.  Part 4 is a multipart, not\r
 133 a leaf part.  The new series includes a patch for this.\r