b4/b9f36ab4a9323eea6c4423711a46c2f9650b08

   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 C2945431FD0\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 May 2011 16:05:56 -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: 0.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[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 dgeHQwNA--5w for <notmuch@notmuchmail.org>;\r
  16         Tue, 17 May 2011 16:05:56 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id F0D95431FB6\r
  19         for <notmuch@notmuchmail.org>; Tue, 17 May 2011 16:05:55 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id 40E8E29A053;\r
  22         Tue, 17 May 2011 16:05:54 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 2B47B254183; Tue, 17 May 2011 16:05:54 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
  27 Subject: Re: MIME restructuring [was: Re: release-candidate/0.6]\r
  28 In-Reply-To: <87liy6bbug.fsf@yoom.home.cworth.org>\r
  29 References: <8762pn7gth.fsf@servo.factory.finestructure.net>\r
  30         <874o4zczr8.fsf@yoom.home.cworth.org>\r
  31         <87wrhvyqfn.fsf@servo.factory.finestructure.net>\r
  32         <87r57ybcna.fsf@yoom.home.cworth.org>\r
  33         <4DD18DFE.7090206@fifthhorseman.net>\r
  34         <87liy6bbug.fsf@yoom.home.cworth.org>\r
  35 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  36         (i486-pc-linux-gnu)\r
  37 Date: Tue, 17 May 2011 16:05:48 -0700\r
  38 Message-ID: <87ei3x9bcj.fsf@yoom.home.cworth.org>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: multipart/signed; boundary="=-=-=";\r
  41         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  42 Cc: Notmuch Mail <notmuch@notmuchmail.org>\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: Tue, 17 May 2011 23:05:56 -0000\r
  56 \r
  57 --=-=-=\r
  58 Content-Transfer-Encoding: quoted-printable\r
  59 \r
  60 On Mon, 16 May 2011 13:59:51 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  61 > So what I'd love to see from here is a commit with a description like\r
  62 > the above, and then a test case looking like your example.\r
  63 >=20\r
  64 > From there, I'd next like a new version of the commit that gets the\r
  65 > intended behavior with less code duplication.\r
  66 >=20\r
  67 > I'll work on each of the above unless someone beats me to any of it. Let\r
  68 > me know.\r
  69 \r
  70 I've now pushed out my own version of the MIME restructuring feature.\r
  71 \r
  72 It differs from what was presented here in avoiding the code\r
  73 duplication.\r
  74 \r
  75 It also provides properly nested mutlipart/* output for the\r
  76 =2D-format=3Dtext case. That's not that I expect anybody to *do* anything\r
  77 with that nested output, just that it was cleaner and easier to fix both\r
  78 text/json at the same time, (and avoiding doing that is perhaps what led\r
  79 to the original code duplication).\r
  80 \r
  81 I also cherry-picked in a piece of later patch from Jamie so that the\r
  82 existing emacs tests still pass. And I updated the documentation for\r
  83 this new feature.\r
  84 \r
  85 So I'm happy with this new feature now, (which I know provides an\r
  86 essential part of the basis for the rest of the crypto branch).\r
  87 \r
  88 From=20here, I'm hoping that my review of the rest of Jamie's\r
  89 release-candidate branch goes faster. The general shape of the commits\r
  90 and commit messages looks pretty good to me, so I think it will.\r
  91 \r
  92 I think there are still features added here and there without\r
  93 corresponding test cases (multipart/alternative is one that comes to\r
  94 mind) and perhaps without updated documentation (--decrypt is at least\r
  95 documented---but I do think it's strange that it's documented to only\r
  96 work for json output).\r
  97 \r
  98 More from me tomorrow.\r
  99 \r
 100 =2DCarl\r
 101 \r
 102 --=-=-=\r
 103 Content-Type: application/pgp-signature\r
 104 \r
 105 -----BEGIN PGP SIGNATURE-----\r
 106 Version: GnuPG v1.4.11 (GNU/Linux)\r
 107 \r
 108 iEYEARECAAYFAk3S/0wACgkQ6JDdNq8qSWh8NgCfTJYFQXSq70KNgDoPLsv9iduj\r
 109 /wkAn2rHXxF15uhiwVdy9VENwbjeb/px\r
 110 =yNBm\r
 111 -----END PGP SIGNATURE-----\r
 112 --=-=-=--\r