From owner-svn-ports-all@freebsd.org Fri Apr 1 09:11:22 2016 Return-Path: Delivered-To: svn-ports-all@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 E1A70AEB683; Fri, 1 Apr 2016 09:11:22 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id BE8E91D6E; Fri, 1 Apr 2016 09:11:22 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: by shepard.synsport.net (Postfix, from userid 80) id 6D45243CD5; Fri, 1 Apr 2016 04:11:20 -0500 (CDT) To: Thomas Zander Subject: Re: svn commit: r412296 - head/lang/rust MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Date: Fri, 01 Apr 2016 11:11:20 +0200 From: "John Marino (FreeBSD)" Cc: John Marino , Jan Beich , "ports-committers@FreeBSD.org" , svn-ports-all@freebsd.org, svn-ports-head@freebsd.org, thomas.e.zander@googlemail.com Reply-To: marino@freebsd.org Mail-Reply-To: marino@freebsd.org In-Reply-To: References: <201603312004.u2VK4n5n028013@repo.freebsd.org> <7485ef18e1261c87b17a9c23da01259a@secure.marino.st> Message-ID: <1d06dfb14f40d98439d2f1f759375ada@secure.marino.st> X-Sender: freebsd.contact@marino.st User-Agent: Roundcube Webmail/0.9.1 X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 01 Apr 2016 09:11:23 -0000 On 2016-04-01 10:56, Thomas Zander wrote: > I said this does not fix a problem with the port, but it does > introduce a new feature, and the maintainer should be in the loop > before it is committed. Because at the end of the day the maintainer > is responsible for supporting a feature and receives the blame if > something breaks. What new feature? It just fixes DF support. In fact, Rust has been better supported on DragonFly for a long time. There was a dragonfly-only port (contained in dports) created because the freebsd ports version was so far behind. The fixes from that port actually came back to FreeBSD. This is what your message is losing. The implication is that DF brings no benefit to ports but that's simply not the case. The price for these benefits are a couple of no-op lines in the makefiles, which is a great deal for FreeBSD. > Exactly this is my point: This commit does not a have a benefit to > FreeBSD. It adds *only* DF-specifics. Therefore my instincts tell me > this should be part of dports, and I would have appreciated this > discussion before the commit, not after the fact. That's all. And > since the fix-it-blanket is not clear enough in all cases (obviously > Jan and me came to different conclusions whether this one is a fix), > it would be good to have a word from portmgr whether something like > this is part of it or not. Because for sure there are/will be more PRs > with a similar intention. First, the point is wrong, even on this specific port, FreeBSD has definitely benefitted from DF participation. Secondly, if ports are split unnecessary, everyone is unaware of the impact of changes. It's not like the dports patches are visible. The split patches was for conflict, not DF-specifics. Thirdly, this version of rust built before, so restoring the build is under a standard blanket. The main directives are DF support is not supposed to increase obligations on maintainers, and I'm not allowed to "demand" a maintainer support DF. Jan's commit did not increase the obligation to the maintainer, he only fixed a port that previously built, which is definitely appreciated. John