From owner-cvs-all Tue Nov 27 14:41:46 2001 Delivered-To: cvs-all@freebsd.org Received: from fledge.watson.org (fledge.watson.org [204.156.12.50]) by hub.freebsd.org (Postfix) with ESMTP id 71EE537B416; Tue, 27 Nov 2001 14:41:37 -0800 (PST) Received: from localhost (arr@localhost) by fledge.watson.org (8.11.6/8.11.5) with SMTP id fARMfPW21685; Tue, 27 Nov 2001 17:41:25 -0500 (EST) (envelope-from arr@FreeBSD.org) X-Authentication-Warning: fledge.watson.org: arr owned process doing -bs Date: Tue, 27 Nov 2001 17:41:24 -0500 (EST) From: "Andrew R. Reiter" X-Sender: arr@fledge.watson.org To: Mike Barcroft Cc: Robert Watson , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: Committer's guide policy on commit message contents In-Reply-To: <20011127165507.B12400@espresso.q9media.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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 Tue, 27 Nov 2001, Mike Barcroft wrote: : :I completely agree. The main intent of my message was to point out :the absurdity of following this rule. If commit messages did indeed :only answer the why, they would be very cryptic. : :Best regards, :Mike Barcroft Hrm. I always thought that you were supposed to say only why since you can see what was changed in the commit (Yes, you can diff versions in cvs.. it's not that bad :-P). I agree with Robert regarding the idea of including little summaries if and when large commits with multiple parts are done -- At this point, reviewing a diff would be unreasonable so hints as to what is going on would be beneficial from a time saving perspective _and_ from a "racking your brain to figure out what's going on" perspective. While one might say that including what you changed would always save you time, I agree that is the case, but I think that viewing source first hand is always better than just understanding what the persons log summarizes. just my opinion Cheers, Andrew -- Andrew R. Reiter arr@watson.org arr@FreeBSD.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message