diff options
author | Garrett Wollman <wollman@FreeBSD.org> | 1994-07-24 22:27:27 +0000 |
---|---|---|
committer | Garrett Wollman <wollman@FreeBSD.org> | 1994-07-24 22:27:27 +0000 |
commit | 3a5140872fdc234cb0048706fa5ca85e73bd82f6 (patch) | |
tree | 078804a00fe880b8a0ceea38bb17a5fbcb7179d7 /CVS-INFO | |
parent | 99fa41fb52d623f4f7584fead6b308ef4e75276a (diff) | |
download | src-test2-3a5140872fdc234cb0048706fa5ca85e73bd82f6.tar.gz src-test2-3a5140872fdc234cb0048706fa5ca85e73bd82f6.zip |
Notes
Diffstat (limited to 'CVS-INFO')
-rw-r--r-- | CVS-INFO | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/CVS-INFO b/CVS-INFO new file mode 100644 index 000000000000..7d416670f31e --- /dev/null +++ b/CVS-INFO @@ -0,0 +1,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 + |