summaryrefslogtreecommitdiff
path: root/CVS-INFO
diff options
context:
space:
mode:
authorGarrett Wollman <wollman@FreeBSD.org>1994-07-24 22:27:27 +0000
committerGarrett Wollman <wollman@FreeBSD.org>1994-07-24 22:27:27 +0000
commit3a5140872fdc234cb0048706fa5ca85e73bd82f6 (patch)
tree078804a00fe880b8a0ceea38bb17a5fbcb7179d7 /CVS-INFO
parent99fa41fb52d623f4f7584fead6b308ef4e75276a (diff)
downloadsrc-test2-3a5140872fdc234cb0048706fa5ca85e73bd82f6.tar.gz
src-test2-3a5140872fdc234cb0048706fa5ca85e73bd82f6.zip
Notes
Diffstat (limited to 'CVS-INFO')
-rw-r--r--CVS-INFO44
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
+