dd/42c71d43b815dfef586e1a5b4d3715da2be013

   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 B8E74429E22\r
   6         for <notmuch@notmuchmail.org>; Fri,  6 May 2011 14:06:29 -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 BVFbgZTqrkzh; Fri,  6 May 2011 14:06:29 -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 1FC89431FB5;\r
  18         Fri,  6 May 2011 14:06:29 -0700 (PDT)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id B5C69254185; Fri,  6 May 2011 14:06:28 -0700 (PDT)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Florian Friesdorf <flo@chaoflow.net>, notmuch@notmuchmail.org\r
  23 Subject: Re: [PATCH] create and set temporary home directory\r
  24 In-Reply-To: <877har5uek.fsf@eve.chaoflow.net>\r
  25 References: <877har5uek.fsf@eve.chaoflow.net>\r
  26 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  27         (i486-pc-linux-gnu)\r
  28 Date: Fri, 06 May 2011 14:06:22 -0700\r
  29 Message-ID: <87pqnvk08h.fsf@yoom.home.cworth.org>\r
  30 MIME-Version: 1.0\r
  31 Content-Type: multipart/signed; boundary="=-=-=";\r
  32         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  33 X-BeenThere: notmuch@notmuchmail.org\r
  34 X-Mailman-Version: 2.1.13\r
  35 Precedence: list\r
  36 List-Id: "Use and development of the notmuch mail system."\r
  37         <notmuch.notmuchmail.org>\r
  38 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  40 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  41 List-Post: <mailto:notmuch@notmuchmail.org>\r
  42 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  43 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  45 X-List-Received-Date: Fri, 06 May 2011 21:06:29 -0000\r
  46 \r
  47 --=-=-=\r
  48 Content-Transfer-Encoding: quoted-printable\r
  49 \r
  50 On Mon, 18 Apr 2011 19:41:39 +0200, Florian Friesdorf <flo@chaoflow.net> wr=\r
  51 ote:\r
  52 > My first patch send to the list, not sure whether done properly.\r
  53 \r
  54 Just fine, Florian. Thanks for the contribution!\r
  55 \r
  56 One small thing you might do differently in the future is to tweak the\r
  57 email message to read exactly like a commit message. For example, a\r
  58 sentence like the above "not sure whether done properly" is fine in an\r
  59 email message, but doesn't make sense in the commit message.\r
  60 \r
  61 > I think the tests should not touch the build user's home directory. The\r
  62 > patch creates a directory in the temporary test directory and sets home\r
  63 > accordingly.\r
  64 \r
  65 Similarly, everything in a commit message is known to be your opinion,\r
  66 so you should omit phrases like "I think". Instead, you should describe\r
  67 what the commit actually does, and then describe why it does that.\r
  68 \r
  69 Finally, this little separator with three dashes:\r
  70 \r
  71 > ---\r
  72 >  test/test-lib.sh |    7 +++++++\r
  73 >  1 files changed, 7 insertions(+), 0 deletions(-)\r
  74 \r
  75 Is what lets git know where your commit message ends. Anything you\r
  76 include after that (And before the patch itself) will be ignored by\r
  77 git. So that's the perfect place to put a sentence like "My first\r
  78 patch---not sure whether done properly".\r
  79 \r
  80 So with all of the above points taken together, your email might have\r
  81 read something like what I've provided below.\r
  82 \r
  83 I applied this patch last week, and would have pushed it, except that\r
  84 just after applying it, I also tried cleaning up some of this part of\r
  85 the code. And in the process it seems I managed to get the test suite to\r
  86 run "rm -rf ${HOME}" with my actual home directory (oops!).\r
  87 \r
  88 Fortunately, I was able to recover from most of that with my dirvish\r
  89 backups. But then I was also on vacation for several days.\r
  90 \r
  91 Anyway, I'm excited to see the release candidate branch that jrollins is\r
  92 putting together now. Hopefully I'll find some good way to collaborate\r
  93 with the work he is doing.\r
  94 \r
  95 =2DCarl\r
  96 \r
  97 From: Florian Friesdorf <flo@chaoflow.net>, notmuch@notmuchmail.org\r
  98 Subject: [PATCH] Create and set temporary home directory\r
  99 To: notmuch@notmuchmail.org\r
 100 \r
 101 Create and set temporary home directory\r
 102 =20=20=20=20\r
 103 Within test/emacs two tests access the build users home\r
 104 directory. These tests fail in case of a non-existent home directory.\r
 105 =20=20=20=20\r
 106 Fix these tests to not touch the build user's home directory. Instead,\r
 107 create a directory in the temporary test directory and set HOME\r
 108 accordingly.\r
 109 \r
 110 =2D--\r
 111 \r
 112 This is my first patch sent to the list. I'm not sure whether I've done\r
 113 it properly.\r
 114 \r
 115  test/test-lib.sh |    7 +++++++\r
 116  1 files changed, 7 insertions(+), 0 deletions(-)\r
 117 ...\r
 118 \r
 119 --=-=-=\r
 120 Content-Type: application/pgp-signature\r
 121 \r
 122 -----BEGIN PGP SIGNATURE-----\r
 123 Version: GnuPG v1.4.11 (GNU/Linux)\r
 124 \r
 125 iEYEARECAAYFAk3EYs4ACgkQ6JDdNq8qSWio6ACgorXAURXCMMT1v5FsvmY0VayM\r
 126 N60An3/AKxe2JKgfbT/CVOXdHL/w7fMa\r
 127 =1Us3\r
 128 -----END PGP SIGNATURE-----\r
 129 --=-=-=--\r