From owner-freebsd-hackers@FreeBSD.ORG Fri Jun 11 15:39:54 2004 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 4E3A916A4CE for ; Fri, 11 Jun 2004 15:39:54 +0000 (GMT) Received: from xorpc.icir.org (xorpc.icir.org [192.150.187.68]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4081143D49 for ; Fri, 11 Jun 2004 15:39:54 +0000 (GMT) (envelope-from rizzo@icir.org) Received: from xorpc.icir.org (localhost [127.0.0.1]) by xorpc.icir.org (8.12.9p1/8.12.8) with ESMTP id i5BFdRgd004410; Fri, 11 Jun 2004 08:39:27 -0700 (PDT) (envelope-from rizzo@xorpc.icir.org) Received: (from rizzo@localhost) by xorpc.icir.org (8.12.9p1/8.12.3/Submit) id i5BFdR5d004409; Fri, 11 Jun 2004 08:39:27 -0700 (PDT) (envelope-from rizzo) Date: Fri, 11 Jun 2004 08:39:27 -0700 From: Luigi Rizzo To: Jens Schweikhardt Message-ID: <20040611083927.B2580@xorpc.icir.org> References: <20040611025307.GG17367@teleri.net> <20040611140740.GA3355@schweikhardt.net> <20040611072615.A96657@xorpc.icir.org> <20040611152019.GA5028@schweikhardt.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20040611152019.GA5028@schweikhardt.net>; from schweikh@schweikhardt.net on Fri, Jun 11, 2004 at 05:20:19PM +0200 cc: Chip Norkus cc: hackers@freebsd.org Subject: Re: indent bugfix / added features X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Jun 2004 15:39:54 -0000 On Fri, Jun 11, 2004 at 05:20:19PM +0200, Jens Schweikhardt wrote: ... > Sigh. A request for a little bit of QA and an emoticon as well and > I'm criticised. I remember when I was not yet a committer that the > better I could demonstrate that the code has no ill-effect the more > chances some committer would bring it in the tree. yeah but that's not a reason to give the same 'treatment' to other people. In the end, if you don't feel like taking the risk, you don't commit the patch and nobody will blame you. > I've looked at too many "innocous" patches that performed unexpectedly > when run, to not let myself get away with this. Of course this may be i am not trying to discuss general principles, but only this specific case. This is a very small patch and you can easily check it while you type/paste it in (if nothing else just for curiosity on what was the problem and how was it fixed). In the end, if you don't feel like taking the risk, you don't commit the patch; nobody will blame you. > No hard feelings, Luigi :-) nor on my side! I was just trying to make a point that we should not try to scare or annoy people who are so kind to contribute patches just because we don't have time to scrutinize them (which in the end is our responsibility, not theirs). I wouldn't have said a word if you had some actual comments/criticism on the contributed code. Even if they were only style issues (which we shouldn't even bother to criticise in these cases, as they can be trivially fixed at commit time). But asking for "more input or i won't even look at your code" (at least, that was the sense one could perceive) was a bit too much... cheers luigi