1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
|
$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
|
