On Jan 21, 2006, at 2:29 PM, Ruediger Pluem wrote:
Ok. Then I had a different understanding from my osmosis :-).
Any other comments on this?
I have no problem adopting the above rules for future CHANGE entries.
Jim is correct.
It is easy to forget now because Subversion doesn't have the
rcstemplate feature, but commits should still have:
PR:
Obtained from:
Submitted by:
Reviewed by:
in the log when they are applicable.
CVS:
----------------------------------------------------------------------
CVS: PR:
CVS: If this change addresses a PR in the problem report tracking
CVS: database, then enter the PR number(s) here.
CVS: Obtained from:
CVS: If this change has been taken from another system, such as NCSA,
CVS: then name the system in this line, otherwise delete it.
CVS: Submitted by:
CVS: If this code has been contributed to Apache by someone else;
i.e.,
CVS: they sent us a patch or a new module, then include their name/
email
CVS: address here. If this is your work then delete this line.
CVS: Reviewed by:
CVS: If we are doing pre-commit code reviews and someone else has
CVS: reviewed your changes, include their name(s) here.
CVS: If you have not had it reviewed then delete this line.
....Roy
