From: Eli Zaretskii <eliz@gnu.org>
Subject: CVS commits and logs
Date: Sat, 04 Nov 2006 13:40:21 +0200 [thread overview]
Message-ID: <uirhv4eze.fsf@gnu.org> (raw)
It looks like some people are not aware of the guidelines about CVS
commits and commit logs, or maybe those guidelines are ambiguous or
unclear, because the style of the log text differs considerably from
one maintainer to another, and hints on commit procedure that is not
according to what I know. See the few examples below.
I think Richard should decide what is the canonical form and what are
the guidelines that everybody should follow, and we should then
describe those guidelines in some file in admin/. I describe here
the rules I know about, as a starting point:
. CVS log entries should be simply the ChangeLog entries with the
file name and the leading TABs stripped. It is okay to reformat
and refill the text to make a better use of the line real estate
after the leading TABs were removed, but otherwise the text should
remain intact (but see below for an exception).
. Each file should be committed separately, even if the changes are
related, and the CVS log entry should be for the changes in that
file only. In particular, the modified files and the ChangeLog
file with the appropriate log entry should be committed separately
(thus the CVS log entries for ChangeLog files should never include
log entries for the modified files). Exception: it is okay to
commit several changes to a single ChangeLog file in one "cvs ci"
command if those changes are related to the same feature/bugfix.
. Since the previous rule separates the text for similar or related
changes in different files, entries that say "Ditto." or otherwise
refer to text for other files' entries should be rewritten to be
self-contained in the CVS log. This is an exception from the
first rule, which says that the ChangeLog text should generally
remain intact.
. The ChangeLog files should be committed with an empty log message
(unless this is a real change in the ChangeLog file itself, not an
addition of log entries).
Here are some random examples of the various styles we currently have
in CVS log (the entries are indented 2 spaces to the right):
lisp/ChangeLog:
revision 1.10244
date: 2006-11-03 13:54:32 +0000; author: ramprasadb; state: Exp; lines: +4 -0
*** empty log message ***
----------------------------
revision 1.10243
date: 2006-11-02 23:46:14 +0000; author: monnier; state: Exp; lines: +8 -0
(server-auth-key): Remove. Replace by a process-property.
(server-start): Don't remove the file of the previous process, but
instead clear out the place for the new file.
(server-start): Set the :auth-key property.
(server-process-filter): Use the :auth-key property.
----------------------------
revision 1.10232
date: 2006-10-30 21:34:15 +0000; author: cyd; state: Exp; lines: +8 -0
* tutorial.el: Move defvars to avoid bytecomp warnings.
(tutorial--find-changed-keys): Check if viper-current-state is
bound before using it.
(help-with-tutorial): Check if viper-tutorial is defined before
using it.
----------------------------
revision 1.10200
date: 2006-10-18 14:05:16 +0000; author: jhd; state: Exp; lines: +6 -0
* term/x-win.el (res-geometry): Don't set geometry from Xresources
to default-frame-alist if default-frame-alist already contains
widht/height.
----------------------------
revision 1.10182
date: 2006-10-11 06:47:35 +0000; author: monnier; state: Exp; lines: +359 -0
Merge from upstream, upto version 5.22.
After 5.0:
`cperl-add-tags-recurse-noxs-fullpath': new function (for -batch mode)
Fine-grained changelog:
`cperl-hook-after-change': New configuration variable
`cperl-vc-sccs-header': Likewise.
`cperl-vc-sccs-header': Likewise.
`cperl-vc-header-alist': Default via two preceding variables
`cperl-invalid-face': Remove double quoting under XEmacs
lisp/faces.el:
revision 1.354
date: 2006-09-14 23:54:44 +0000; author: rms; state: Exp; lines: +1 -1
(mode-line-faces): Group mode-line instead of modeline.
----------------------------
revision 1.351
date: 2006-06-28 23:29:26 +0000; author: nickrob; state: Exp; lines: +1 -1
* help-mode.el (help-xref-symbol-regexp): Add property as a keyword
for ignoring links.
* faces.el (read-face-name): Use it.
----------------------------
revision 1.344
date: 2005-11-20 15:24:27 +0000; author: cyd; state: Exp; lines: +5 -11
* faces.el: Revert 2005-11-17 change. :ignore-defface is now
handled automagically.
----------------------------
revision 1.321
date: 2005-06-22 01:22:40 +0000; author: miles; state: Exp; lines: +1 -1
Revision: miles@gnu.org--gnu-2005/emacs--cvs-trunk--0--patch-438
Rename vertical-divider face to vertical-border
2005-06-22 Miles Bader <miles@gnu.org>
* lisp/faces.el (vertical-border): Renamed from `vertical-divider'.
2005-06-22 Miles Bader <miles@gnu.org>
* man/display.texi (Faces): Change `vertical-divider' to `vertical-border'.
2005-06-22 Miles Bader <miles@gnu.org>
* src/xfaces.c (Qvertical_border): Renamed from `Qvertical_divider'.
(realize_basic_faces, syms_of_xfaces): Update references to it.
* src/dispextern.h (enum face_id): Rename `VERTICAL_DIVIDER_FACE_ID'
to `VERTICAL_BORDER_FACE_ID'.
* src/dispnew.c (build_frame_matrix_from_leaf_window): Update references.
----------------------------
next reply other threads:[~2006-11-04 11:40 UTC|newest]
Thread overview: 110+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-04 11:40 Eli Zaretskii [this message]
2006-11-04 12:29 ` CVS commits and logs Eli Zaretskii
2006-11-04 12:53 ` Juanma Barranquero
2006-11-04 13:23 ` Eli Zaretskii
2006-11-04 13:44 ` Juanma Barranquero
2006-11-04 13:57 ` Eli Zaretskii
2006-11-04 14:37 ` Juanma Barranquero
2006-11-04 16:06 ` Reiner Steib
2006-11-04 22:30 ` Juanma Barranquero
2006-11-05 19:22 ` Richard Stallman
2006-11-05 7:08 ` Richard Stallman
2006-11-05 21:22 ` Juanma Barranquero
2006-11-04 22:28 ` Jason Rumney
2006-11-05 6:13 ` Eli Zaretskii
2006-11-05 7:08 ` Richard Stallman
2006-11-05 23:15 ` Bill Wohler
2006-11-06 2:30 ` Miles Bader
2006-11-06 16:08 ` Richard Stallman
2006-11-06 19:38 ` Miles Bader
2006-11-07 16:47 ` Richard Stallman
2006-11-07 22:44 ` Miles Bader
2006-11-09 0:53 ` Richard Stallman
2006-11-09 4:47 ` Miles Bader
2006-11-09 6:01 ` Bill Wohler
2006-11-11 2:11 ` Richard Stallman
2006-11-11 3:24 ` Miles Bader
2006-11-11 10:03 ` Andreas Schwab
2006-11-11 12:51 ` Miles Bader
2006-11-11 17:00 ` Switching to Subversion (was: CVS commits and logs) Bill Wohler
2006-11-11 20:00 ` Switching to Subversion Romain Francoise
2006-11-12 1:20 ` Bill Wohler
2006-11-12 11:26 ` Romain Francoise
2006-11-12 21:12 ` Richard Stallman
2006-11-12 21:20 ` David Kastrup
2006-11-12 21:49 ` Juanma Barranquero
2006-11-12 22:45 ` Miles Bader
2006-11-12 23:17 ` Juanma Barranquero
2006-11-12 23:29 ` Miles Bader
2006-11-12 23:40 ` Juanma Barranquero
2006-11-12 23:59 ` Miles Bader
2006-11-13 8:02 ` stephen
2006-11-13 12:38 ` Bruce Stephens
2006-11-13 20:26 ` Juanma Barranquero
2006-11-12 21:39 ` Juanma Barranquero
2006-11-12 22:43 ` Sascha Wilde
2006-11-12 23:22 ` Miles Bader
2006-11-12 23:37 ` Juanma Barranquero
2006-11-12 23:54 ` Miles Bader
2006-11-12 23:58 ` Juanma Barranquero
2006-11-13 0:15 ` Miles Bader
2006-11-13 1:20 ` Juanma Barranquero
2006-11-13 16:26 ` Stefan Monnier
2006-11-13 16:53 ` Juanma Barranquero
2006-11-13 16:59 ` Juanma Barranquero
2006-11-13 20:13 ` Stefan Monnier
2006-11-14 4:50 ` Bill Wohler
2006-11-14 6:49 ` Stefan Monnier
2006-11-27 21:42 ` Jari Aalto
2006-11-28 4:52 ` Miles Bader
2006-11-28 12:06 ` Jari Aalto
2006-11-13 8:41 ` Sascha Wilde
2006-11-13 8:50 ` David Kastrup
2006-11-13 8:59 ` Juanma Barranquero
2006-11-13 9:11 ` Sascha Wilde
2006-11-13 9:26 ` David Kastrup
2006-11-13 9:53 ` dhruva
2006-11-13 10:55 ` Thomas Arendsen Hein
2006-11-28 11:29 ` Jari Aalto
2006-11-13 12:38 ` Sascha Wilde
2006-11-13 12:40 ` Juanma Barranquero
2006-11-14 2:27 ` Han-Wen Nienhuys
2006-11-13 12:55 ` David Kastrup
2006-11-13 14:49 ` Sascha Wilde
2006-11-13 14:57 ` Juanma Barranquero
2006-11-13 8:51 ` Miles Bader
2006-11-13 11:43 ` Thomas Arendsen Hein
2006-11-13 12:31 ` Juanma Barranquero
2006-11-13 13:16 ` Miles Bader
2006-11-13 13:35 ` Juanma Barranquero
2006-11-13 13:54 ` David Kastrup
2006-11-13 14:08 ` Juanma Barranquero
2006-11-14 3:53 ` Bill Wohler
2006-11-13 14:46 ` Sascha Wilde
2006-11-13 22:31 ` Miles Bader
2006-11-13 22:44 ` Juanma Barranquero
2006-11-14 6:38 ` Stefan Monnier
2006-11-13 11:20 ` Thomas Arendsen Hein
2006-11-13 13:11 ` Miles Bader
2006-11-13 14:51 ` Sascha Wilde
2006-11-13 16:55 ` Thomas Arendsen Hein
2006-11-13 22:37 ` Miles Bader
2006-11-13 20:15 ` Richard Stallman
2006-11-14 2:49 ` Miles Bader
2006-11-13 8:47 ` joakim
2006-11-13 8:56 ` Juanma Barranquero
2006-11-13 9:05 ` joakim
2006-11-14 4:55 ` Bill Wohler
2006-11-13 20:15 ` Richard Stallman
2006-11-14 1:53 ` Juliusz Chroboczek
2006-11-14 2:10 ` Masatake YAMATO
2006-11-14 3:05 ` Eric Hanchrow
2006-11-14 3:23 ` Miles Bader
2006-11-12 5:14 ` CVS commits and logs Richard Stallman
2006-11-12 9:45 ` Miles Bader
2006-11-13 9:44 ` Richard Stallman
2006-11-07 18:07 ` Reiner Steib
2006-11-07 20:33 ` Eli Zaretskii
2006-11-08 1:50 ` Miles Bader
2006-11-11 9:43 ` Reiner Steib
2006-11-06 4:21 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=uirhv4eze.fsf@gnu.org \
--to=eliz@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).