c4/a3d18f7a5c90e46a21b355ed9594e44608315c

   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 AF357429E21\r
   6         for <notmuch@notmuchmail.org>; Mon, 25 Apr 2011 13:52:06 -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.99\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.99 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, 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 xzbdbZQWBJBx; Mon, 25 Apr 2011 13:52:06 -0700 (PDT)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id 6C64B431FB6;\r
  18         Mon, 25 Apr 2011 13:52:05 -0700 (PDT)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id E5FA72543F1; Mon, 25 Apr 2011 13:52:04 -0700 (PDT)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Austin Clements <amdragon@mit.edu>\r
  23 Subject: Re: [PATCH 0/5] Fetch all message metadata in a single pass\r
  24 In-Reply-To: <AANLkTi=uZXTacQFiUUk8k5h17NrV1wTo5OR7T=3UguE_@mail.gmail.com>\r
  25 References: <1291928396-27937-1-git-send-email-amdragon@mit.edu>\r
  26         <AANLkTiknTgHruLKKYsVo-r9319VHxmuXcw5oxOLPzHQ_@mail.gmail.com>\r
  27         <87hbba8ggp.fsf@yoom.home.cworth.org>\r
  28         <AANLkTi=uZXTacQFiUUk8k5h17NrV1wTo5OR7T=3UguE_@mail.gmail.com>\r
  29 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.2.1\r
  30         (i486-pc-linux-gnu)\r
  31 Date: Mon, 25 Apr 2011 13:52:04 -0700\r
  32 Message-ID: <87oc3u3vgr.fsf@yoom.home.cworth.org>\r
  33 MIME-Version: 1.0\r
  34 Content-Type: multipart/signed; boundary="=-=-=";\r
  35         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  36 Cc: notmuch@notmuchmail.org\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Mon, 25 Apr 2011 20:52:06 -0000\r
  50 \r
  51 --=-=-=\r
  52 Content-Transfer-Encoding: quoted-printable\r
  53 \r
  54 On Mon, 21 Mar 2011 02:56:05 -0400, Austin Clements <amdragon@mit.edu> wrot=\r
  55 e:\r
  56 > Thanks for the thorough review.  My updated patches are on the\r
  57 > eager-metadata-v4 branch (also, for-review/eager-metadata-v4) at\r
  58 >   http://awakening.csail.mit.edu/git/notmuch.git\r
  59 \r
  60 Great. I failed at my commitment to quickly apply the updated series. I\r
  61 want to do it now, but I'm failing to be able to fetch the code:\r
  62 \r
  63         $ git fetch amdragon\r
  64         error: RPC failed; result=3D22, HTTP code =3D 417\r
  65         fatal: The remote end hung up unexpectedly\r
  66 \r
  67 (This is using the URL above.)\r
  68 \r
  69 > This doesn't cause any problems in the one weird case that still needs\r
  70 > the reference to the list.  After the _notmuch_tags_create, the caller\r
  71 > simply adds that reference.\r
  72 \r
  73 This, (and the other comments you made about the cleanup), all sound\r
  74 great to me. So I'm looking forward to seeing the code.\r
  75 \r
  76 > Heh, fixed.  I suppose I hadn't been thinking about it, since none of\r
  77 > the source in lib/ has other authors listed.\r
  78 >=20\r
  79 > But it raises an interesting question.  When is it kosher to add\r
  80 > oneself to a file's author list?  In this case I wrote about half of\r
  81 > the code in that admittedly short file, so that makes sense  Changing\r
  82 > a few lines presumably isn't enough.  Adding a few functions?\r
  83 \r
  84 My intention is simply to leave this up to the author of the code in\r
  85 question. If you feel your name belongs on the author list, then please\r
  86 put it there.\r
  87 \r
  88 Similarly, if you think your code is sufficient to deserve copyright\r
  89 protection, (so probably anything beyond simple typo fixes or otherwise\r
  90 tiny changes), then *please* add your name (or your employer's name as\r
  91 appropriate) to that list as well.\r
  92 \r
  93 A quick check with grep shows that I've definitely not been asking for\r
  94 this attribution as much as I should have. Hopefully we can all get in a\r
  95 better habit going forward.\r
  96 \r
  97 =2DCarl\r
  98 \r
  99 =2D-=20\r
 100 carl.d.worth@intel.com\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 iD8DBQFNtd706JDdNq8qSWgRAnPgAJ4hW0yQFMjdq5cRswEfSgsj2Yo4bQCfXrt4\r
 109 d+7R347ReGKfcec/iIJbujQ=\r
 110 =9nKU\r
 111 -----END PGP SIGNATURE-----\r
 112 --=-=-=--\r