CVS-INFO

   1 $FreeBSD$
   2
   3 This file really doesn't belong here, but I wanted to elicit comment
   4 without starting a discussion as to where it really does belong.
   5 Anyway, here goes...
   6
   7 When the new CVS tree was set up back in May, I wanted to ensure that
   8 everybody at least gave some thought to asking other people to look at
   9 code before it is committed.  Since we have had a number of rancorous
  10 discussions in the past about both committing code prematurely, and
  11 committing code that was written by someone else, I decided to use the
  12 `template' function of CVS to give people a chance to think about it.
  13
  14 The template looks like this:
  15
  16 Reviewed by:
  17 Submitted by:
  18 CVS: ----------------------------------------------------------------------
  19 CVS: Delete the ``Reviewed by'' line if this is somebody else's work
  20 CVS: (you are the reviewer).
  21 CVS: Delete the ``Submitted by'' line if this is NOT somebody else's
  22 CVS: work (you are the author).
  23
  24 If you are making substantial or possibly controversial changes to the
  25 code, please talk to someone else who either has commit authority, or
  26 who is recognized as an expert in the BSD community.  This latter
  27 category includes the author of the program or subsystem you're
  28 modifying, like Rick Macklem for the NFS code, or Keith Bostic for
  29 `vi'.
  30
  31 If the changes are insubstantial and noncontroversial (e.g., fixing
  32 spelling errors, or most Makefile changes) there is no need for
  33 review.  Please delete the ``Reviewed by:'' line, though, so that the
  34 logs look cleaner.
  35
  36 If you are committing code or modifications which were originally done
  37 by some other person or group, them please please please make sure to
  38 credit them in the ``Submitted by'' line.  In this case, you are
  39 considered to be the reviewer.  If it is your own code, or that of
  40 another team member, then you should delete the ``Submitted by'' line,
  41 so that the logs look cleaner.
  42
  43 -GAWollman
  44