From owner-svn-src-head@freebsd.org Thu Dec 1 18:32:22 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 7DCDAC6134B for ; Thu, 1 Dec 2016 18:32:22 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from outbound1a.eu.mailhop.org (outbound1a.eu.mailhop.org [52.58.109.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 18FE61FC1 for ; Thu, 1 Dec 2016 18:32:21 +0000 (UTC) (envelope-from ian@freebsd.org) X-MHO-User: 7ca46bdd-b7f4-11e6-9673-39b5816e8152 X-Report-Abuse-To: https://support.duocircle.com/support/solutions/articles/5000540958-duocircle-standard-smtp-abuse-information X-Originating-IP: 73.78.92.27 X-Mail-Handler: DuoCircle Outbound SMTP Received: from ilsoft.org (unknown [73.78.92.27]) by outbound1.eu.mailhop.org (Halon) with ESMTPSA id 7ca46bdd-b7f4-11e6-9673-39b5816e8152; Thu, 01 Dec 2016 18:32:18 +0000 (UTC) Received: from rev (rev [172.22.42.240]) by ilsoft.org (8.15.2/8.15.2) with ESMTP id uB1IWCdu004011; Thu, 1 Dec 2016 11:32:12 -0700 (MST) (envelope-from ian@freebsd.org) Message-ID: <1480617132.1889.196.camel@freebsd.org> Subject: Re: svn commit: r309344 - head/lib/libutil From: Ian Lepore To: "Ngie Cooper (yaneurabeya)" , Mark Linimon Cc: "Conrad E. Meyer" , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Date: Thu, 01 Dec 2016 11:32:12 -0700 In-Reply-To: <5B2D372D-52AB-42FB-B0AA-942A75466602@gmail.com> References: <201612010221.uB12Lb6x067202@repo.freebsd.org> <4DFE017D-90FD-44F4-8FE7-7EE53D499F7B@gmail.com> <1480609927.1889.189.camel@freebsd.org> <20161201170116.GA20872@lonesome.com> <5B2D372D-52AB-42FB-B0AA-942A75466602@gmail.com> Content-Type: text/plain; charset="iso-8859-7" X-Mailer: Evolution 3.18.5.1 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 01 Dec 2016 18:32:22 -0000 On Thu, 2016-12-01 at 09:33 -0800, Ngie Cooper (yaneurabeya) wrote: > > > > On Dec 1, 2016, at 9:01 AM, Mark Linimon > > wrote: > > > > On Thu, Dec 01, 2016 at 09:32:07AM -0700, Ian Lepore wrote: > > > > > > On Wed, 2016-11-30 at 19:07 -0800, Ngie Cooper wrote: > > > > > > > > Really?? I wish you hadn't added me to this list.. I don't in > > > > any way > > > > endorse the way that you went about dealing with this > > > > disagreement. > > > Really?  Well then feel free to ADD my name to the list of people > > > who > > > found that comment, and the subsequent non-response to the > > > complaints > > > about it, completely inappropriate.   > > I expressed an opinion on an IRC channel that I found the comment > > offensive. > > > > But I would rather my permission had been asked ahead of time > > before > > having that comment attached to a public commit log.  I think this > > is > > part of the point that ngie is trying to make. > > > > I'm well-known for my wish that core took a more active, public, > > stance > > on such matters, but I think here that two wrongs may not have made > > a > > right. > Yes. > > And more importantly, the new comment doesnąt tell me *why* I > shouldnąt use FD_CLOEXEC. The owness is still on the reader to figure > out why using `fcntl(fd, F_SETFD, FD_CLOEXEC)` is wrong. > > All the new commit does is makes the commit that des@ made more > palatable and benign to readers. And how is that a bad thing, compared to the prior situation?  Insult has been removed and information has been added.  You complain that the information that was added was incomplete.  Well, there is one person who could fix that, and he has apparently chosen not to.  So perhaps your ongoing complaint should be directed accordingly. -- Ian