Re: MIME restructuring [was: Re: release-candidate/0.6]

diff --git a/b4/b9f36ab4a9323eea6c4423711a46c2f9650b08 b/b4/b9f36ab4a9323eea6c4423711a46c2f9650b08
new file mode 100644 (file)
index 0000000..50d33be
--- /dev/null
+++ b/b4/b9f36ab4a9323eea6c4423711a46c2f9650b08
@@ -0,0 +1,112 @@
+Return-Path: <cworth@cworth.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id C2945431FD0\r
+       for <notmuch@notmuchmail.org>; Tue, 17 May 2011 16:05:56 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0.01\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
+       tests=[T_MIME_NO_TEXT=0.01] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id dgeHQwNA--5w for <notmuch@notmuchmail.org>;\r
+       Tue, 17 May 2011 16:05:56 -0700 (PDT)\r
+Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
+       by olra.theworths.org (Postfix) with ESMTP id F0D95431FB6\r
+       for <notmuch@notmuchmail.org>; Tue, 17 May 2011 16:05:55 -0700 (PDT)\r
+Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
+       by arlo.cworth.org (Postfix) with ESMTP id 40E8E29A053;\r
+       Tue, 17 May 2011 16:05:54 -0700 (PDT)\r
+Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
+       id 2B47B254183; Tue, 17 May 2011 16:05:54 -0700 (PDT)\r
+From: Carl Worth <cworth@cworth.org>\r
+To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
+Subject: Re: MIME restructuring [was: Re: release-candidate/0.6]\r
+In-Reply-To: <87liy6bbug.fsf@yoom.home.cworth.org>\r
+References: <8762pn7gth.fsf@servo.factory.finestructure.net>\r
+       <874o4zczr8.fsf@yoom.home.cworth.org>\r
+       <87wrhvyqfn.fsf@servo.factory.finestructure.net>\r
+       <87r57ybcna.fsf@yoom.home.cworth.org>\r
+       <4DD18DFE.7090206@fifthhorseman.net>\r
+       <87liy6bbug.fsf@yoom.home.cworth.org>\r
+User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
+       (i486-pc-linux-gnu)\r
+Date: Tue, 17 May 2011 16:05:48 -0700\r
+Message-ID: <87ei3x9bcj.fsf@yoom.home.cworth.org>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/signed; boundary="=-=-=";\r
+       micalg=pgp-sha1; protocol="application/pgp-signature"\r
+Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Tue, 17 May 2011 23:05:56 -0000\r
+\r
+--=-=-=\r
+Content-Transfer-Encoding: quoted-printable\r
+\r
+On Mon, 16 May 2011 13:59:51 -0700, Carl Worth <cworth@cworth.org> wrote:\r
+> So what I'd love to see from here is a commit with a description like\r
+> the above, and then a test case looking like your example.\r
+>=20\r
+> From there, I'd next like a new version of the commit that gets the\r
+> intended behavior with less code duplication.\r
+>=20\r
+> I'll work on each of the above unless someone beats me to any of it. Let\r
+> me know.\r
+\r
+I've now pushed out my own version of the MIME restructuring feature.\r
+\r
+It differs from what was presented here in avoiding the code\r
+duplication.\r
+\r
+It also provides properly nested mutlipart/* output for the\r
+=2D-format=3Dtext case. That's not that I expect anybody to *do* anything\r
+with that nested output, just that it was cleaner and easier to fix both\r
+text/json at the same time, (and avoiding doing that is perhaps what led\r
+to the original code duplication).\r
+\r
+I also cherry-picked in a piece of later patch from Jamie so that the\r
+existing emacs tests still pass. And I updated the documentation for\r
+this new feature.\r
+\r
+So I'm happy with this new feature now, (which I know provides an\r
+essential part of the basis for the rest of the crypto branch).\r
+\r
+From=20here, I'm hoping that my review of the rest of Jamie's\r
+release-candidate branch goes faster. The general shape of the commits\r
+and commit messages looks pretty good to me, so I think it will.\r
+\r
+I think there are still features added here and there without\r
+corresponding test cases (multipart/alternative is one that comes to\r
+mind) and perhaps without updated documentation (--decrypt is at least\r
+documented---but I do think it's strange that it's documented to only\r
+work for json output).\r
+\r
+More from me tomorrow.\r
+\r
+=2DCarl\r
+\r
+--=-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.11 (GNU/Linux)\r
+\r
+iEYEARECAAYFAk3S/0wACgkQ6JDdNq8qSWh8NgCfTJYFQXSq70KNgDoPLsv9iduj\r
+/wkAn2rHXxF15uhiwVdy9VENwbjeb/px\r
+=yNBm\r
+-----END PGP SIGNATURE-----\r
+--=-=-=--\r