From owner-freebsd-ports@freebsd.org Sun Jul 19 16:44:30 2015 Return-Path: Delivered-To: freebsd-ports@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 C91E39A5793 for ; Sun, 19 Jul 2015 16:44:30 +0000 (UTC) (envelope-from michelle@sorbs.net) Received: from mailman.ysv.freebsd.org (mailman.ysv.freebsd.org [IPv6:2001:1900:2254:206a::50:5]) by mx1.freebsd.org (Postfix) with ESMTP id ADA301C66 for ; Sun, 19 Jul 2015 16:44:30 +0000 (UTC) (envelope-from michelle@sorbs.net) Received: by mailman.ysv.freebsd.org (Postfix) id A17569A5792; Sun, 19 Jul 2015 16:44:30 +0000 (UTC) Delivered-To: ports@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 873B29A5791 for ; Sun, 19 Jul 2015 16:44:30 +0000 (UTC) (envelope-from michelle@sorbs.net) Received: from hades.sorbs.net (hades.sorbs.net [67.231.146.201]) by mx1.freebsd.org (Postfix) with ESMTP id E50261C65; Sun, 19 Jul 2015 16:44:29 +0000 (UTC) (envelope-from michelle@sorbs.net) MIME-version: 1.0 Content-transfer-encoding: 7BIT Content-type: text/plain; CHARSET=US-ASCII Received: from isux.com (firewall.isux.com [213.165.190.213]) by hades.sorbs.net (Oracle Communications Messaging Server 7.0.5.29.0 64bit (built Jul 9 2013)) with ESMTPSA id <0NRQ000EXURZB100@hades.sorbs.net>; Sun, 19 Jul 2015 09:50:24 -0700 (PDT) Message-id: <55ABD3EA.8010704@sorbs.net> Date: Sun, 19 Jul 2015 18:44:26 +0200 From: Michelle Sullivan User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.24) Gecko/20100301 SeaMonkey/1.1.19 To: Baptiste Daroussin Cc: "ports@freebsd.org" , Dimitry Andric Subject: Re: Self committing... allowed or not? References: <55AB91ED.3080908@sorbs.net> <9917125A-6342-4F62-B374-E4F456EDC015@FreeBSD.org> <55ABBFEC.60302@sorbs.net> <20150719154449.GD50618@ivaldir.etoilebsd.net> In-reply-to: <20150719154449.GD50618@ivaldir.etoilebsd.net> X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Jul 2015 16:44:31 -0000 Baptiste Daroussin wrote: > On Sun, Jul 19, 2015 at 05:19:08PM +0200, Michelle Sullivan wrote: > >> Dimitry Andric wrote: >> >>> On 19 Jul 2015, at 14:02, Michelle Sullivan wrote: >>> >>> >>>> please correct me if I'm wrong but isn't self committing (those with the >>>> commit bit committing their own patches without QA/review/adding >>>> patchfiles to the PR) against the rules?... or is it just a free-for-all >>>> now? >>>> >>>> >>> If they are the maintainer, it is OK by definition. Otherwise, approval >>> from either the maintainer or portmgr@ is needed. >>> >>> However, a number of people are on vacation, and they have notified >>> other developers that is OK to fix their ports while they are away. >>> Within reason, of course. :-) >>> >>> In any case, which specific ports are you worried about? >>> >>> -Dimitry >>> >>> >>> >> Here's the case and the three referenced commits: >> >> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=199265 >> >> And I know the top-level dependency will now break other things because >> of a minor detail that the committer did not take into account... That >> said I don't know if any other dependencies on it exist (so therefore it >> might not break anything else - however I am fairly sure it wasn't >> checked by the committer because of the speed and absoluteness of the >> change) because I don't need it/use it myself... but that is not the >> point. I was 'just lucky' to come across this change process as I was >> not looking for anything, just happened to be in the right place at the >> right time to see it, and considering the hoops use plebs (those without >> the commit bit) have to jump through I thought it was rather ironic that >> 3 separate ports were changed, no testing was recorded in the PR as we >> the plebs are required to do, no patches uploaded as we the plebs have >> to do and no review as we the plebs have to have... >> >> > do you appear to know the said ports were broken (segfault) at startup because > of various libssl mixup, they have been tested and fixed. if another issue > appears on those ports I will fix them. > I'm guessing you missed the '--use-ldap' in the top level dependency... I'm assuming you know there are issues with openldap and the use base vs use ports issue... particularly with dependent ports and incompatible options... your 'fix' quite possibly fixed one problem and caused another (not your fault as it happens - but an unintended consequence of an unchecked change... if you want to bring order and stability this is not the way to proceed. (That said neither is laborious change control and peer review, but some is needed and the rules should apply to everyone or there will be more chaos.)) -- Michelle Sullivan http://www.mhix.org/