45 lines
1.8 KiB
Plaintext
45 lines
1.8 KiB
Plaintext
|
$FreeBSD$
|
||
|
|
|
||
|
|
This file really doesn't belong here, but I wanted to elicit comment
|
||
|
|
without starting a discussion as to where it really does belong.
|
||
|
|
Anyway, here goes...
|
||
|
|
|
||
|
|
When the new CVS tree was set up back in May, I wanted to ensure that
|
||
|
|
everybody at least gave some thought to asking other people to look at
|
||
|
|
code before it is committed. Since we have had a number of rancorous
|
||
|
|
discussions in the past about both committing code prematurely, and
|
||
|
|
committing code that was written by someone else, I decided to use the
|
||
|
|
`template' function of CVS to give people a chance to think about it.
|
||
|
|
|
||
|
|
The template looks like this:
|
||
|
|
|
||
|
|
Reviewed by:
|
||
|
|
Submitted by:
|
||
|
|
CVS: ----------------------------------------------------------------------
|
||
|
|
CVS: Delete the ``Reviewed by'' line if this is somebody else's work
|
||
|
|
CVS: (you are the reviewer).
|
||
|
|
CVS: Delete the ``Submitted by'' line if this is NOT somebody else's
|
||
|
|
CVS: work (you are the author).
|
||
|
|
|
||
|
|
If you are making substantial or possibly controversial changes to the
|
||
|
|
code, please talk to someone else who either has commit authority, or
|
||
|
|
who is recognized as an expert in the BSD community. This latter
|
||
|
|
category includes the author of the program or subsystem you're
|
||
|
|
modifying, like Rick Macklem for the NFS code, or Keith Bostic for
|
||
|
|
`vi'.
|
||
|
|
|
||
|
|
If the changes are insubstantial and noncontroversial (e.g., fixing
|
||
|
|
spelling errors, or most Makefile changes) there is no need for
|
||
|
|
review. Please delete the ``Reviewed by:'' line, though, so that the
|
||
|
|
logs look cleaner.
|
||
|
|
|
||
|
|
If you are committing code or modifications which were originally done
|
||
|
|
by some other person or group, them please please please make sure to
|
||
|
|
credit them in the ``Submitted by'' line. In this case, you are
|
||
|
|
considered to be the reviewer. If it is your own code, or that of
|
||
|
|
another team member, then you should delete the ``Submitted by'' line,
|
||
|
|
so that the logs look cleaner.
|
||
|
|
|
||
|
|
-GAWollman
|
||
|
|
|