8b/262b085609310ae2a2a7bbf03a7028e6ee647b

   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 05790431FD0\r
   6         for <notmuch@notmuchmail.org>; Thu, 12 May 2011 15:36:52 -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 QeQctZdnLNup for <notmuch@notmuchmail.org>;\r
  16         Thu, 12 May 2011 15:36:51 -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 42D19431FB6\r
  19         for <notmuch@notmuchmail.org>; Thu, 12 May 2011 15:36:51 -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 AC80C29A039;\r
  22         Thu, 12 May 2011 15:36:49 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 6DC05254183; Thu, 12 May 2011 15:36:49 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  27         Notmuch Mail <notmuch@notmuchmail.org>\r
  28 Subject: Re: release-candidate/0.6\r
  29 In-Reply-To: <8762pn7gth.fsf@servo.factory.finestructure.net>\r
  30 References: <8762pn7gth.fsf@servo.factory.finestructure.net>\r
  31 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  32         (i486-pc-linux-gnu)\r
  33 Date: Thu, 12 May 2011 15:36:43 -0700\r
  34 Message-ID: <874o4zczr8.fsf@yoom.home.cworth.org>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: multipart/signed; boundary="=-=-=";\r
  37         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  38 Cc: David Edmondson <dme@dme.org>\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Thu, 12 May 2011 22:36:52 -0000\r
  52 \r
  53 --=-=-=\r
  54 Content-Transfer-Encoding: quoted-printable\r
  55 \r
  56 On Fri, 06 May 2011 12:46:34 -0700, Jameson Graef Rollins <jrollins@finestr=\r
  57 ucture.net> wrote:\r
  58 > Hi, folks.  As some of you already know, I am trying to put together a\r
  59 > release candidate for a 0.6 release that we can present to cworth for\r
  60 > approval.\r
  61 \r
  62 Hi Jameson,\r
  63 \r
  64 This is really great! I appreciate you collecting useful patches for\r
  65 review like this. I hope this helps us get things moving once again.\r
  66 \r
  67 > So far, this release candidate includes a couple of patch series that\r
  68 > are not currently on cworth's master branch:\r
  69 >=20\r
  70 > json structure now replicates mime structure\r
  71 > * dme's json-fully-reflects-MIME-structure improvements\r
  72 \r
  73 I got stalled on the first commit on this branch:\r
  74 \r
  75         commit 5a5aae66bf41d3c621c412da711fec9b33f37dcc\r
  76         Author: David Edmondson <dme@dme.org>\r
  77         Date:   Mon May 10 10:25:15 2010 +0100\r
  78 \r
  79             Improved MIME support.\r
  80 =20=20=20=20\r
  81         Signed-off-by: Jameson Rollins <jrollins@finestructure.net>\r
  82 \r
  83 It's probably the same issue that stalled me when David first sent this\r
  84 patch series for review over a year ago[!]. I'm guessing I didn't do a\r
  85 good job of stating the issue then, so I'll try to do a better job here.\r
  86 \r
  87 This first patch appears to be doing multiple things:\r
  88 \r
  89    * It changes how the emacs code inserts MIME parts when constructing\r
  90      a mail view.\r
  91 \r
  92    * It changes something in the json formatting. (Just extra newlines?)\r
  93 \r
  94    * It changes something about the MIME part counting.\r
  95 \r
  96    * It appears to be doing new emission of multipart part when doing\r
  97      json output, (and for this has some hard-coded printf("]}\n") stuff\r
  98      rather than using the formatter).\r
  99 \r
 100    * It has new explicit support for an embedded Message as a MIME part,\r
 101      (inserting the headers of the enclosed message).\r
 102 \r
 103 So that's five seemingly independent changes. I'd really like to see\r
 104 those split up into separate commits as much as possible. Or, excepting\r
 105 that, there should be some explanation/justification in the commit\r
 106 message for why some must be combined.\r
 107 \r
 108 The worst part is that not a single change is actually described in the\r
 109 commit message. All we have is "Improved MIME support". Improved how?\r
 110 What changed? Why? What impact does it have? Does it fix bugs? Lay\r
 111 groundwork for future changes?\r
 112 \r
 113 What's really changing here and why?\r
 114 \r
 115 If the patch series were sufficiently-well described in the commit\r
 116 message, then my review process could be more or less:\r
 117 \r
 118   * Read the commit message. Does it describe a desirable change?\r
 119 \r
 120   * If so, read the patch content. Does it do what the commit message\r
 121     describes? Accurately? Without doing unrelated things?\r
 122 \r
 123   * If so, accept the patch.\r
 124 \r
 125 Does anyone want to attempt to fix up this first patch? (It doesn't\r
 126 necessarily have to be David).\r
 127 \r
 128 From=20a quick skim, many later patches in the series appear to be better\r
 129 in this regard.\r
 130 \r
 131 =2DCarl\r
 132 \r
 133 [!] Has it really been that long?\r
 134 \r
 135 --=-=-=\r
 136 Content-Type: application/pgp-signature\r
 137 \r
 138 -----BEGIN PGP SIGNATURE-----\r
 139 Version: GnuPG v1.4.11 (GNU/Linux)\r
 140 \r
 141 iEYEARECAAYFAk3MYPsACgkQ6JDdNq8qSWir9QCgnK926GAwX/KVMgLX2CDYBPhS\r
 142 eAgAn1VqV/IFi7REPZDD6RGX28j3Xvh1\r
 143 =BM8E\r
 144 -----END PGP SIGNATURE-----\r
 145 --=-=-=--\r