From owner-svn-ports-head@FreeBSD.ORG Tue Nov 26 16:08:14 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 0C89DEF3; Tue, 26 Nov 2013 16:08:14 +0000 (UTC) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id D5B732989; Tue, 26 Nov 2013 16:08:12 +0000 (UTC) Received: from [192.168.0.21] (unknown [130.255.26.50]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 3C1DE435A3; Tue, 26 Nov 2013 10:07:47 -0600 (CST) Message-ID: <5294C740.1080207@marino.st> Date: Tue, 26 Nov 2013 17:07:28 +0100 From: John Marino User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130509 Thunderbird/17.0.6 MIME-Version: 1.0 To: Eygene Ryabinkin Subject: Re: svn commit: r334593 - in head/dns: bind96 bind96/files bind98 bind98/files bind99 bind99/files References: <201311221341.rAMDfY3g002148@svn.freebsd.org> <01490d316b5dd7f01f1c843871542666@secure.marino.st> <2DEEB271-4795-4DF0-8F85-30316BA3070F@FreeBSD.org> <5291C8BD.3070109@marino.st> <20131126035339.GA43389@FreeBSD.org> <529451E7.4010801@marino.st> In-Reply-To: X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-ports-head@freebsd.org, Alexey Dokuchaev , owner-ports-committers@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org, Erwin Lansing X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.16 Precedence: list Reply-To: marino@freebsd.org List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Nov 2013 16:08:14 -0000 On 11/26/2013 16:48, Eygene Ryabinkin wrote: > Tue, Nov 26, 2013 at 08:46:47AM +0100, John Marino wrote: >> On 11/26/2013 04:53, Alexey Dokuchaev wrote: >>> On Sun, Nov 24, 2013 at 10:37:01AM +0100, John Marino wrote: >>>> I have attached a diff with the proposed changes covering all three >>>> ports. It's pretty straight-forward. >>>> >>>> .if ${PORT_OPTIONS:MREPLACE_BASE} >>>> -.if ${OSVERSION} >= 1000500 >>>> +.if ${OPSYS} == FreeBSD && ${OSVERSION} >= 1000500 >>> >>> Given that OSVERSION is used extensively in the tree, I'm worrying that >>> adding these extra "${OPSYS} == FreeBSD" checks will our makefiles less >>> readable. Wouldn't it make more sense to place this check somewhere in >>> single place, and define whatever OSVERSION that looks reasonable when >>> OPSYS == FreeBSD instead of patching every port? >> >> There is a "discussion" about this on the portmgr list. Well, it's not >> really a discussion. I started it, bapt answered it once, and the >> portmgr stopped talking about it. >> >> As far as "less readable", that really should not be a concern at all. >> It is simply invalid to use OSVERSION without OPSYS since it's OPSYS >> specific. > > What are the other OPSYS that FreeBSD ports collection should support? > > If there are any that are really needed, then OPSYS must be combined > with OSVERSION, but if there is no strong current demand for that, it > is probably not the right time to add complexity to the Makefiles. *Hundreds* of problems with ports have already been detected and fixed as a direct result of the creation in dports. In many cases, problems were fixed on F10 before the general users saw them. Based on that track record alone, does that not merit using OSVERSION properly to support another system that brings those benefits? And yes, it's possible that yet other platforms can use ports in the future. Right now each new instance of OSVERSION can break a port, so now is the right time to not only make developers aware of that but to reducing its use. I am also not agreeing that prefixing OSVERSION with "OPSYS == &&" is "complex". People seriously are not going to have trouble with the concept that OSVERSION is platform specific. John