93/da8325251e1e9998e656f24bf20a59658ea166

   1 Return-Path: <tomi.ollila@iki.fi>\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 arlo.cworth.org (Postfix) with ESMTP id E47626DE14D6\r
   6  for <notmuch@notmuchmail.org>; Tue, 27 Oct 2015 15:55:34 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.746\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.746 tagged_above=-999 required=5 tests=[AWL=0.094, \r
  12  SPF_NEUTRAL=0.652] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id owF80TKUMsci for <notmuch@notmuchmail.org>;\r
  16  Tue, 27 Oct 2015 15:55:32 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id 095656DE133A\r
  19  for <notmuch@notmuchmail.org>; Tue, 27 Oct 2015 15:55:31 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21  by guru.guru-group.fi (Postfix) with ESMTP id 67127100046;\r
  22  Wed, 28 Oct 2015 00:55:33 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Carl Worth <cworth@cworth.org>, David Bremner <david@tethera.net>,\r
  25  notmuch@notmuchmail.org\r
  26 Subject: Re: elisp + company completion patches, v7\r
  27 In-Reply-To: <871tche9vr.fsf@wondoo.home.cworth.org>\r
  28 References: <1445786638-13763-1-git-send-email-david@tethera.net>\r
  29  <m2lhaqu23j.fsf@guru.guru-group.fi> <871tche9vr.fsf@wondoo.home.cworth.org>\r
  30 User-Agent: Notmuch/0.21~rc3+7~g28030b0 (http://notmuchmail.org) Emacs/24.3.1\r
  31  (x86_64-unknown-linux-gnu)\r
  32 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  33  $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  34  !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  35 Date: Wed, 28 Oct 2015 00:55:33 +0200\r
  36 Message-ID: <m2k2q7exay.fsf@guru.guru-group.fi>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.20\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: <https://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: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Tue, 27 Oct 2015 22:55:35 -0000\r
  52 \r
  53 On Mon, Oct 26 2015, Carl Worth <cworth@cworth.org> wrote:\r
  54 \r
  55 > On Sun, Oct 25 2015, Tomi Ollila wrote:\r
  56 >> I, if this does not sound nitpicking, would amend those 'This patch' and\r
  57 >> 'With this patch' texts in commit messages away by some rewording -- those\r
  58 >> pose as a bad example for someone reading the code -- and diminish the\r
  59 >> effect when I complain that again in future patched ;)\r
  60 >\r
  61 > What's the motivation here?\r
  62 >\r
  63 > It's often the case that a good commit message needs to describe the way\r
  64 > things were working prior to a commit, (describing the bug or missing\r
  65 > feature or what have you), and also needs to describe the change that is\r
  66 > affected by the commit itself.\r
  67 >\r
  68 > So I actually like seeing commit messages that say things like:\r
  69 >\r
  70 >     Prior to this commit... (some bug existed...)\r
  71 >\r
  72 >     In this commit... (some code is reworked to fix the bug...)\r
  73 >\r
  74 > But maybe I've missed the point of your message.\r
  75 >\r
  76 >\r
  77 > -Carl\r
  78 \r
  79 Probably the missing point is the fine distinction between \r
  80 'this patch' and 'this commit'\r
  81 \r
  82 While we do sets of changes and then that leads to commit series the final\r
  83 step of running 'git format-patch' somehow leads people writing commit\r
  84 message to add words 'this patch' to the message...\r
  85 ... or (what just come into my mind) they just think these changes as\r
  86 patches (is that linux-kernel influence ?)\r
  87 \r
  88 To me the commit history does not look like series of patches (but series\r
  89 of commits or changes -- and changes are usually shown as "diffs") --\r
  90 patching gives a connotation of fixing something (with glue or gum ;)\r
  91 \r
  92 That is how I see it. That doesn't mean I am right there (and in this mailing\r
  93 list I've been wrong in statistically significant amount (*) of times).\r
  94 \r
  95 Un{less,til} decided otherwise I'll continue with delicate efforts to\r
  96 keep this style issue in the minds of developers; despite (**) informing\r
  97 me that will be doomed...\r
  98 \r
  99 Tomi\r
 100 \r
 101 (*) https://en.wikipedia.org/wiki/Statistical_significance\r
 102 (**) https://en.wikipedia.org/wiki/Wiio%27s_laws\r