Red Hat Bugzilla – Bug 64182
Leading newlines compound when reusing log messages
Last modified: 2007-04-18 12:42:21 EDT
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0rc1) Gecko/20020417
Description of problem:
The leading newline that now appears when editing log messages builds up if one
"reuses" the log message for successive commits.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
If you commit changes in multiple directories, and want to use the same
description for them all, you type in the first log and then for each new log,
you simply exit and accept that. This used to be fine but the newlines pile up
making tons of whitespace in the logs or forcing people to delete the excess
Actual Results: Accumulates newlines.
Expected Results: Should leave message intact.
I would suggest either reverting to the old behaviour (not sure what we gained
with this newline), or checking to see whether the log is being reused before
The following is an excerpt from the changelog of the cvs RPM that shows when
the change was introduced.
* Tue Jul 31 2001 Bernhard Rosenkraenzer <email@example.com> 1.11.1p1-3
- Fix up initial cvs login (#47457)
- Bring back the leading newline at the beginning of commit messages
"a" is one key less than "O". ;)
- Fix build in the current build system
Created attachment 85104 [details]
Patch against cvs-1.11.1p1-7
The leading newline inserted at the beginning of a log message was deprecated by the
CVS team in 1.11.1. The Red Hat patch cvs-1.11.1p1-cvspass.patch applied to cvs-
1.11.1p1-3 incorrectly addressed this problem by prepending a newline to the first line of
the standard log message template. The results is that a newline is added ahead of the
standard template each time a log message is reused as part of a recursive commit.
The attached patch cvs-1.11.1p1-logmsg.patch remedies this by backing out the change
to logmsg.c performed in cvs-1.11.1p1-cvspass.patch and correctly applies the leading
newline (only when no log message already exists).