2b/6bc2537f93ca44a5c9a3c7f73f7786ed51f345

   1 Return-Path: <flo@chaoflow.net>\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 C6D39429E22\r
   6         for <notmuch@notmuchmail.org>; Fri,  6 May 2011 16:30:13 -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 PQpyuCChRDzq for <notmuch@notmuchmail.org>;\r
  16         Fri,  6 May 2011 16:30:13 -0700 (PDT)\r
  17 Received: from tesla.chaoflow.net (tesla.chaoflow.net [188.40.54.22])\r
  18         (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id BEE44431FB5\r
  21         for <notmuch@notmuchmail.org>; Fri,  6 May 2011 16:30:12 -0700 (PDT)\r
  22 Received: from eve.chaoflow.net (ppp-93-104-187-164.dynamic.mnet-online.de\r
  23         [93.104.187.164])\r
  24         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  25         (No client certificate requested)\r
  26         (Authenticated sender: flo@chaoflow.net)\r
  27         by tesla.chaoflow.net (Postfix) with ESMTPSA id 4E00A45F5;\r
  28         Sat,  7 May 2011 01:30:11 +0200 (CEST)\r
  29 Received: by eve.chaoflow.net (Postfix, from userid 1000) id E9D7E2AC;\r
  30         Fri, 6 May 2011 23:30:10 +0000 (Local time zone must be set--see zic\r
  31         manual page)\r
  32 From: Florian Friesdorf <flo@chaoflow.net>\r
  33 To: Carl Worth <cworth@cworth.org>, notmuch@notmuchmail.org\r
  34 Subject: patch submission guide (was Re: [PATCH] create and set temporary home\r
  35         directory)\r
  36 In-Reply-To: <87pqnvk08h.fsf@yoom.home.cworth.org>\r
  37 References: <877har5uek.fsf@eve.chaoflow.net>\r
  38         <87pqnvk08h.fsf@yoom.home.cworth.org>\r
  39 User-Agent: Notmuch/0.6 (http://notmuchmail.org) Emacs/23.3.1\r
  40         (x86_64-unknown-linux-gnu)\r
  41 Date: Sat, 07 May 2011 01:30:10 +0200\r
  42 Message-ID: <87k4e376gt.fsf@eve.chaoflow.net>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: multipart/signed; boundary="=-=-=";\r
  45         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  46 X-Virus-Scanned: clamav-milter 0.96.5 at tesla\r
  47 X-Virus-Status: Clean\r
  48 X-BeenThere: notmuch@notmuchmail.org\r
  49 X-Mailman-Version: 2.1.13\r
  50 Precedence: list\r
  51 List-Id: "Use and development of the notmuch mail system."\r
  52         <notmuch.notmuchmail.org>\r
  53 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  55 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  56 List-Post: <mailto:notmuch@notmuchmail.org>\r
  57 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  58 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  59         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  60 X-List-Received-Date: Fri, 06 May 2011 23:30:14 -0000\r
  61 \r
  62 --=-=-=\r
  63 Content-Transfer-Encoding: quoted-printable\r
  64 \r
  65 \r
  66 Hi Carl,\r
  67 \r
  68 On Fri, 06 May 2011 14:06:22 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  69 > On Mon, 18 Apr 2011 19:41:39 +0200, Florian Friesdorf <flo@chaoflow.net> =\r
  70 wrote:\r
  71 > > My first patch send to the list, not sure whether done properly.\r
  72 >=20\r
  73 > Just fine, Florian. Thanks for the contribution!\r
  74 >=20\r
  75 > One small thing you might do differently in the future is to tweak the\r
  76 > email message to read exactly like a commit message. For example, a\r
  77 > sentence like the above "not sure whether done properly" is fine in an\r
  78 > email message, but doesn't make sense in the commit message.\r
  79 >=20\r
  80 > > I think the tests should not touch the build user's home directory. The\r
  81 > > patch creates a directory in the temporary test directory and sets home\r
  82 > > accordingly.\r
  83 >=20\r
  84 > Similarly, everything in a commit message is known to be your opinion,\r
  85 > so you should omit phrases like "I think". Instead, you should describe\r
  86 > what the commit actually does, and then describe why it does that.\r
  87 \r
  88 That text was intended as a comment (cover letter according to what I\r
  89 just learned) and the commit message itself was intended to be more to\r
  90 the point.\r
  91 \r
  92 > Finally, this little separator with three dashes:\r
  93 >=20\r
  94 > > ---\r
  95 > >  test/test-lib.sh |    7 +++++++\r
  96 > >  1 files changed, 7 insertions(+), 0 deletions(-)\r
  97 >=20\r
  98 > Is what lets git know where your commit message ends. Anything you\r
  99 > include after that (And before the patch itself) will be ignored by\r
 100 > git. So that's the perfect place to put a sentence like "My first\r
 101 > patch---not sure whether done properly".\r
 102 \r
 103 I now sent a patch with a cover letter using only git (search\r
 104 =2D-format=3Dsanitized_text):\r
 105 \r
 106 git format-patch --cover-letter <COMMIT>^\r
 107 git send-email 000*\r
 108 \r
 109 If everything is correct, I would conclude this into a short howto on\r
 110 notmuchmail.org. So please, if there is still room for improvement, let\r
 111 me know.\r
 112 \r
 113 Do we prefer sendemail.chainreplyto or the new shallow format for patch\r
 114 series [1]?\r
 115 \r
 116 Should people who have a public git repository use it to publish their\r
 117 patches (in addition to sending them here / instead of)?\r
 118 \r
 119 I have seen (amdragon I think) the concept of\r
 120 for-review/... branches. Would that be a best practice?\r
 121 \r
 122 > I applied this patch last week, and would have pushed it, except that\r
 123 > just after applying it, I also tried cleaning up some of this part of\r
 124 > the code. And in the process it seems I managed to get the test suite to\r
 125 > run "rm -rf ${HOME}" with my actual home directory (oops!).\r
 126 \r
 127 I got a bit afraid, shouldnt my patch prevent that? I cannot imagine it\r
 128 caused it.\r
 129 \r
 130 [1] https://felipec.wordpress.com/2009/10/25/git-send-email-tricks/\r
 131 \r
 132 regards\r
 133 florian\r
 134 =2D-=20\r
 135 Florian Friesdorf <flo@chaoflow.net>\r
 136   GPG FPR: 7A13 5EEE 1421 9FC2 108D  BAAF 38F8 99A3 0C45 F083\r
 137 Jabber/XMPP: flo@chaoflow.net\r
 138 IRC: chaoflow on freenode,ircnet,blafasel,OFTC\r
 139 \r
 140 --=-=-=\r
 141 Content-Type: application/pgp-signature\r
 142 \r
 143 -----BEGIN PGP SIGNATURE-----\r
 144 Version: GnuPG v2.0.17 (GNU/Linux)\r
 145 \r
 146 iQIcBAEBAgAGBQJNxISCAAoJEDj4maMMRfCDMugP/0Dvv7/s1iT6zmDe3tRrdiXF\r
 147 8lpRyeoUYHqzzqV75nSFEMFtNZRvutn5CShe+1k9jWN5UHHJijVxwtcTLU2Q1kgO\r
 148 aEXMnE6u+HsO4Cm5pvz3v+LVOwK0LE5wc/WOtVh8k3GWEqRSbTDKkuDaSZNiSSHZ\r
 149 D0OL/fAoEgFrcv810zIqu3IFAFS4rGRIT7kVICtJRI7w4VOs34c7q/9R70J+vKxZ\r
 150 YHpCsbzNk3r5pwD7jjYpVseij1di2MlF9ux7eF9GqDlH8SpAPQcHugn07/oAIP/Q\r
 151 ny+Aml4kgyjT/8Aj/ZssOOcaP2Ut5XAKUT4DN/JU6Ddh+b1L3suGlswobgFGkA4T\r
 152 v7dZcKG3DwcCv9i//9v9rhnxfqJ0tzAh1Oo3ok4V7wifxLujkICrR1ZdbzNkIMwA\r
 153 pwDu+ZKGhVDNY9I2h1vsMmwTso483FSIR5ySPFJ1mNI92tjAwayW2m9l8Il0/G0s\r
 154 4EwAbYJNc++CV8KgRi5+HQ9oQosI6BRI3Ng+Gy7WRMg2uPCRCp3figKHbSUVmeWA\r
 155 x1CsFdIlGtDH9kGIuohIg1MkM+AdQUySIhLD7P2hBV6FPwbf3MVIt0OiDu1V2VCm\r
 156 7TAWDf+j5fHgZ3yyfGAuWQFFTUV6QakEt92krSDTn+CC7/lax1f9JbN9UpAGFJmC\r
 157 OaO4GhQIDM4PseGEtA6r\r
 158 =Lb4y\r
 159 -----END PGP SIGNATURE-----\r
 160 --=-=-=--\r