c4/0c2eb800c791181fbb66a42110a2732e82e7e9

   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 EC499431FD0\r
   6         for <notmuch@notmuchmail.org>; Mon, 16 May 2011 14:20:18 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 nnW7d8o1Hn-H for <notmuch@notmuchmail.org>;\r
  16         Mon, 16 May 2011 14:20:18 -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 32E3D431FB6\r
  19         for <notmuch@notmuchmail.org>; Mon, 16 May 2011 14:20:18 -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 E174C29A056;\r
  22         Mon, 16 May 2011 14:20:16 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id C7736254183; Mon, 16 May 2011 14:20:14 -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: <4DD18DFE.7090206@fifthhorseman.net>\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 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  35         (i486-pc-linux-gnu)\r
  36 Date: Mon, 16 May 2011 14:20:07 -0700\r
  37 Message-ID: <87ei3ybawo.fsf@yoom.home.cworth.org>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: multipart/signed; boundary="=-=-=";\r
  40         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  41 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Mon, 16 May 2011 21:20:19 -0000\r
  55 \r
  56 --=-=-=\r
  57 Content-Type: text/plain; charset=utf-8\r
  58 Content-Transfer-Encoding: quoted-printable\r
  59 \r
  60 On Mon, 16 May 2011 16:50:06 -0400, Daniel Kahn Gillmor <dkg@fifthhorseman.=\r
  61 net> wrote:\r
  62 > So a message like this:\r
  63 >=20\r
  64 > A=E2=94=94=E2=94=AC=E2=95=B4multipart/signed 355339 bytes\r
  65 > B =E2=94=9C=E2=94=AC=E2=95=B4multipart/mixed 353462 bytes\r
  66 > C =E2=94=82=E2=94=9C=E2=95=B4text/plain 235 bytes\r
  67 > D =E2=94=82=E2=94=94=E2=95=B4image/jpeg attachment [foo.jpg] 352752 bytes\r
  68 > E =E2=94=94=E2=95=B4application/pgp-signature attachment [signature.asc] =\r
  69 1030 bytes\r
  70 \r
  71 I tried creating a message like that but mine came out slightly\r
  72 differently:\r
  73 \r
  74 > A=E2=94=94=E2=94=AC=E2=95=B4multipart/mixed\r
  75 > B =E2=94=9C=E2=94=AC=E2=95=B4multipart/signed\r
  76 > C =E2=94=82=E2=94=9C=E2=95=B4text/plain\r
  77 > D =E2=94=82=E2=94=94=E2=95=B4application/pgp-signature\r
  78 > E =E2=94=94=E2=95=B4application/octet-stream\r
  79 \r
  80 I'll have to learn better how to control the emacs mail composer in\r
  81 order to understand how to get signatures to cover attachments if I want\r
  82 to do that kind of thing.\r
  83 \r
  84 > would come out with three parts:\r
  85 >=20\r
  86 >  1) C\r
  87 >  2) D\r
  88 >  3) E\r
  89 \r
  90 That is indeed the behavior I see with master (for both text and json\r
  91 output).\r
  92 \r
  93 > the new code assigns this message to 5 parts:\r
  94 >=20\r
  95 >  1) A\r
  96 >  2) B\r
  97 >  3) C\r
  98 >  4) D\r
  99 >  5) E\r
 100 \r
 101 Interestingly, this is not quite the behavior I get (with commit\r
 102 373f352). With --format=3Dtext I'm now seeing:\r
 103 \r
 104 2) C\r
 105 3) D\r
 106 4) E\r
 107 \r
 108 and with --format=3Djson I'm seeing (I think this structure is right):\r
 109 \r
 110 1) A\r
 111   3) B\r
 112     5) C\r
 113     7) D\r
 114   9) E\r
 115 \r
 116 So that explains some of my confusion. The behavioral change of this\r
 117 commit is really only impacting the json format, and not the text. That\r
 118 wasn't clear from the commit message (and I had only been doing my\r
 119 testing with the text backend).\r
 120 \r
 121 This seems to be justifying my fears about the code duplication---the\r
 122 two code paths are already divergent, (which means that things like\r
 123 notmuch part-number identifiers cannot be used between the different\r
 124 formats). I'd like to fix that by preventing the code duplication.\r
 125 \r
 126 Also, both paths seem to be suffering from some excess part-number\r
 127 incrementing somewhere.\r
 128 \r
 129 All of this should be easy to get right with a careful test case or two.\r
 130 \r
 131 =2DCarl\r
 132 \r
 133 --=-=-=\r
 134 Content-Type: application/pgp-signature\r
 135 \r
 136 -----BEGIN PGP SIGNATURE-----\r
 137 Version: GnuPG v1.4.11 (GNU/Linux)\r
 138 \r
 139 iEYEARECAAYFAk3RlQgACgkQ6JDdNq8qSWhXhACfSwbiYbc7Mtru+JM2/tvfoLE2\r
 140 j1MAnjdHapGaRs5uDmPjdW0V8eOGc77L\r
 141 =+LYf\r
 142 -----END PGP SIGNATURE-----\r
 143 --=-=-=--\r