From owner-freebsd-chat Mon Nov 25 12:28:50 2002 Delivered-To: freebsd-chat@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6418E37B401; Mon, 25 Nov 2002 12:28:49 -0800 (PST) Received: from rwcrmhc51.attbi.com (rwcrmhc51.attbi.com [204.127.198.38]) by mx1.FreeBSD.org (Postfix) with ESMTP id 91D7D43EAA; Mon, 25 Nov 2002 12:28:45 -0800 (PST) (envelope-from swear@attbi.com) Received: from localhost.localdomain ([12.242.158.67]) by rwcrmhc51.attbi.com (rwcrmhc51) with ESMTP id <2002112520284005100rrq4qe>; Mon, 25 Nov 2002 20:28:40 +0000 Received: from localhost.localdomain (localhost [127.0.0.1]) by localhost.localdomain (8.12.6/8.12.5) with ESMTP id gAPKT3d8079614; Mon, 25 Nov 2002 12:29:03 -0800 (PST) (envelope-from swear@attbi.com) Received: (from jojo@localhost) by localhost.localdomain (8.12.6/8.12.5/Submit) id gAPKSqvH079611; Mon, 25 Nov 2002 12:28:52 -0800 (PST) (envelope-from swear@attbi.com) X-Authentication-Warning: localhost.localdomain: jojo set sender to swear@attbi.com using -f To: freebsd-chat@freebsd.org Cc: Giorgos Keramidas Subject: Re: Style(9) question References: <20021124211557.GB1145@gothmog.gr> From: swear@attbi.com (Gary W. Swearingen) Date: 25 Nov 2002 12:28:52 -0800 In-Reply-To: <20021124211557.GB1145@gothmog.gr> Message-ID: <4dbs4d8kcb.s4d@localhost.localdomain> Lines: 24 User-Agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Cuyahoga Valley) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: owner-freebsd-chat@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org Giorgos Keramidas writes: > I always try to be as gentle as I can, when I need to make changes to > submitted patches, and I can only hope that it doesn't make anyone too > upset, leading them to stop working with FreeBSD. People should accept, without getting too upset, committers' authority to make such changes and committers should accept, without getting too upset, patchers' not-too-upset commments about the change. Giorgos' suggested practice of briefing patchers about what was changed and why (and not just saying "committed with changes", forcing patchers to dig through freebsd.org for diffs) goes a long way towards smoothing over the natural offense to patchers's feelings than tend to occur with any change. It tends to head off the suspicion that the committer is just riding rough-shod over the patcher and slipping in, without notice, changes that the patcher is likely to not want made, Even better (for the patcher, and debatably for FreeBSD, but not for the committer) is giving the patcher a chance to comment on a proposed change before committing the change. But people should tolerate a lot of maybe-too-fast change commits so as to keep things moving and not over-work the committer. The patcher can always do another PR, if he thinks it's important. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-chat" in the body of the message