From owner-cvs-all Tue Nov 27 14:26:46 2001 Delivered-To: cvs-all@freebsd.org Received: from mailgate.originative.co.uk (mailgate.originative.co.uk [62.232.68.68]) by hub.freebsd.org (Postfix) with ESMTP id B7FAD37B623; Tue, 27 Nov 2001 14:26:28 -0800 (PST) Received: from lobster.originative.co.uk (lobster [62.232.68.81]) by mailgate.originative.co.uk (Postfix) with ESMTP id 2B9541D169; Tue, 27 Nov 2001 22:26:26 +0000 (GMT) Date: Tue, 27 Nov 2001 22:26:26 -0000 From: Paul Richards To: Robert Watson , Mike Barcroft Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: Committer's guide policy on commit message contents Message-ID: <383590000.1006899985@lobster.originative.co.uk> In-Reply-To: References: X-Mailer: Mulberry/2.1.1 (Linux/x86) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG --On Tuesday, November 27, 2001 16:37:49 -0500 Robert Watson wrote: > > On Tue, 27 Nov 2001, Mike Barcroft wrote: > >> From the Committers Guide: "Do not waste space in the commit messages >> explaining what you did. That is what cvs diff is for. Instead, tell >> us why you did it." > > Ignoring, for a moment, any context -- I think this is actually not a good > policy. In the event that there are complex or large changes being made, > a brief summary of the changes, along with the rationale, is appropriate > in the commit message. At least, that's my feeling :-). Is this > something that would be worth changing? I've always interpreted that to mean, don't say things like "put an uderscore in front of variables", since that's self evident from the diff, instead explain why you saw the need to add the underscores and what problems have been solved in doing so. I think we agree on the general intent but the wording perhaps isn't as good as it needs to be. Paul Richards FreeBSD Services Ltd http://www.freebsd-services.com To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message