Bug 64182 - Leading newlines compound when reusing log messages
Leading newlines compound when reusing log messages
Product: Red Hat Linux
Classification: Retired
Component: cvs (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Martin Stransky
Ben Levenson
Depends On:
  Show dependency treegraph
Reported: 2002-04-28 00:36 EDT by Need Real Name
Modified: 2007-04-18 12:42 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2005-03-09 06:35:03 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Patch against cvs-1.11.1p1-7 (1.12 KB, patch)
2002-11-14 14:00 EST, Douglas K. Fischer
no flags Details | Diff

  None (edit)
Description Need Real Name 2002-04-28 00:36:33 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):

How reproducible:

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.

Additional info:

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
including it.

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 <bero@redhat.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
Comment 1 Douglas K. Fischer 2002-11-14 14:00:13 EST
Created attachment 85104 [details]
Patch against cvs-1.11.1p1-7
Comment 2 Douglas K. Fischer 2002-11-14 14:06:54 EST
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).

Note You need to log in before you can comment on or make changes to this bug.