From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [Emacs-diffs] master 5515625: ; ChangeLog.2 fixes Date: Sun, 25 Oct 2015 21:21:02 +0200 Message-ID: <83fv0ydaap.fsf@gnu.org> References: <20151025140139.5262.2147@vcs.savannah.gnu.org> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1445800895 22044 80.91.229.3 (25 Oct 2015 19:21:35 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 25 Oct 2015 19:21:35 +0000 (UTC) Cc: bruce.connor.am@gmail.com, emacs-devel@gnu.org To: Juanma Barranquero Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sun Oct 25 20:21:20 2015 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1ZqQr0-0005yo-PC for ged-emacs-devel@m.gmane.org; Sun, 25 Oct 2015 20:21:19 +0100 Original-Received: from localhost ([::1]:49240 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqQqz-0005Fg-Uz for ged-emacs-devel@m.gmane.org; Sun, 25 Oct 2015 15:21:17 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:51305) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqQqi-0005FL-Pd for emacs-devel@gnu.org; Sun, 25 Oct 2015 15:21:02 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZqQqd-0005Rv-S9 for emacs-devel@gnu.org; Sun, 25 Oct 2015 15:21:00 -0400 Original-Received: from mtaout20.012.net.il ([80.179.55.166]:65455) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqQqd-0005Rg-K0 for emacs-devel@gnu.org; Sun, 25 Oct 2015 15:20:55 -0400 Original-Received: from conversion-daemon.a-mtaout20.012.net.il by a-mtaout20.012.net.il (HyperSendmail v2007.08) id <0NWS00200ITE4100@a-mtaout20.012.net.il> for emacs-devel@gnu.org; Sun, 25 Oct 2015 21:20:52 +0200 (IST) Original-Received: from HOME-C4E4A596F7 ([84.94.185.246]) by a-mtaout20.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0NWS002FDJ2S1K60@a-mtaout20.012.net.il>; Sun, 25 Oct 2015 21:20:52 +0200 (IST) In-reply-to: X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: Solaris 10 X-Received-From: 80.179.55.166 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:192615 Archived-At: > From: Juanma Barranquero > Date: Sun, 25 Oct 2015 16:47:15 +0100 > Cc: emacs-devel > > The CONTRIBUTE file talks about the commit log messages. We generate the > ChangeLog from the commit messages, but it is its own thing, and I think that > making it more readable, removing redundancies, etc. is good. They mostly have > different targets; we developers are now much more likely to use git log or > gitk than go fishing into ChangeLogs, while people who downloads a source > tarball only has the ChangeLogs. That's why I spend time cleaning them. Thank you for your work. > As for that particular example, I've mostly respected cases of "one line > description non-ending in period", but when the one-line description is of the > type > > * file (blabla): some change" > > I usually change it to > > * dir/file (blabla): Some change. > > for coherence with other entries that do not happen to be the one-line > description of a commit. That's correct. Personally, I think the commit messages themselves could have the period in this special case, but that's just MO. > I also remove (here and in a patch I'm working on) the one-line description in > cases like: > > Do XXX in function YYY of file ZZZ > > * ZZZ: Do XXX in function YYY. > > or > > Do XXX in function YYY of file ZZZ > > * ZZZ(YYY): Do XXX. > > but I've left one-line descriptions untouched when they add information or are > formulated differently. This is borderline, and you might as well leave those untouched. > That said, I do these changes that I feel make ChangeLog better, but anyone is > of course entitled to do the same or revert my changes. Or, if there's some > consensus that some of these changes are wrong, I'll adapt to whatever is > preferred. These are issues of style, so there are no hard rules. Back when we maintained ChangeLog files by hand the style was not really uniform, either. Some more comments to your changes: 2015-10-24 Artur Malabarba - * lisp/character-fold.el: Many improvements - + * lisp/character-fold.el: Many improvements. (character-fold-search-forward, character-fold-search-backward): - New command + New command. (character-fold-to-regexp): Remove lax-whitespace hack. (character-fold-search): Remove variable. Only isearch and query-replace use char-folding, and they both have their own The "many improvements" part could simply go away, it doesn't add any useful information. Also, Artur, I think having part of a ChangeLog style entries in the header line and the rest in the body is not a very good idea. Here's one more example: 2015-10-24 Artur Malabarba - * lisp/isearch.el (search-default-regexp-mode): New variable - + * lisp/isearch.el (search-default-regexp-mode): New variable. (isearch-mode): Use it. I think the commit message should have been as corrected to begin with, and a header line should say something else, without being formatted as an entry. * nt/icons/emacs.ico: - * nextstep/Cocoa/Emacs.base/Contents/Resources/Emacs.icns: Use the new - icons. + * nextstep/Cocoa/Emacs.base/Contents/Resources/Emacs.icns: + Use the new icons. This is okay, but when I see such changes, I always ask myself whether it's worth the trouble. Your call. 2015-10-20 Dmitry Gutov - Don't declare vc-exec-after anymore - * lisp/vc/vc-svn.el: * lisp/vc/vc-mtn.el: * lisp/vc/vc-hg.el: * lisp/vc/vc-cvs.el: * lisp/vc/vc-git.el: - * lisp/vc/vc-bzr.el: Don't declare vc-exec-after anymore. Its - usages have been replaced with vc-run-delayed. + * lisp/vc/vc-bzr.el: Don't declare vc-exec-after anymore. + Its usages have been replaced with vc-run-delayed. Not sure why the header line was deleted, it looks OK to me. Thanks again for working on this.