From owner-svn-ports-all@FreeBSD.ORG Tue Nov 26 15:48:21 2013 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 00AD6740; Tue, 26 Nov 2013 15:48:20 +0000 (UTC) Received: from 0.mx.codelabs.ru (0.mx.codelabs.ru [144.206.233.71]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 8DB8F2764; Tue, 26 Nov 2013 15:48:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=codelabs.ru; s=three; h=Sender:In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date; bh=K1c6m1T/98hRAuy7O+tLZY6MjZz5ipipHNEFR+4vMZk=; b=qz74phGC+IBvMspemCEBBB0a/CeqQJVhhkCDW9AxdDf0OP5ca4ZCiXXrS+oOMFf/m//M6PGCX3lncXCyw4ztHpI7pfDSonKq1eo8eXwE0BqCPCQoaTyLmoD4irBzFYVVhwMMYi96u9y6JguVcK47nnS0thDnrYd6Qc/PB9r+ckG3DyPWTYK/2rLWzj5/HFnzh+Eaqo9VCpG78O51S49gS9+2o0uD0B7fnXn/oxOAHeiSI7/XUrJi38VCjDh0KyXosVw6ZD7R2r3IKJd+R8L0KCMLz+QiALphPapPDGl0wcfToOdUTHpDyIWaFo6wCIu19nGC4QNP/gxYFLY3FqGQ9QNu81V0nzoJNGIE5S4NpG6TOYP7SYh2yTmjCFAg7dV1TbapYUB+xRH+Wy7FZB0lhHMy7gjyHaH15Y7UvaxDLOfZP8kHK2LF3umekFb9fgp7o5WeCg5WHBGfcVaj9uBfyaSRW1zLurVAj2qrWhFJYfkO5XK52/njtlgKZnaSdxMLoC4McYku+tZTy7z6R0WVgn2+n5LBEPhwynbeytBjJTocDMf4GI4YoGcY3maKwHF7xU/Rben1rqyNF0dYr1qbFSHRs8ySSWiPMfXuLgvHsgYBIA/qUg1tH6g3Ce9lbh2ed2JnEfDw/b2q6pYgcck9+OQRAxRe99T6fvunMFM/IE0=; Received: from void.codelabs.ru (void.codelabs.ru [144.206.233.66]) by 0.mx.codelabs.ru with esmtpsa (TLSv1:DHE-RSA-AES256-SHA:256) id 1VlKs5-000Mll-A4; Tue, 26 Nov 2013 19:48:17 +0400 Date: Tue, 26 Nov 2013 19:48:14 +0400 From: Eygene Ryabinkin To: marino@freebsd.org Subject: Re: svn commit: r334593 - in head/dns: bind96 bind96/files bind98 bind98/files bind99 bind99/files Message-ID: 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> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="J/zg8ciPNcraoWb6" Content-Disposition: inline In-Reply-To: <529451E7.4010801@marino.st> Sender: rea@codelabs.ru 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-all@freebsd.org X-Mailman-Version: 2.1.16 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: Tue, 26 Nov 2013 15:48:21 -0000 --J/zg8ciPNcraoWb6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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. > >> =20 > >> .if ${PORT_OPTIONS:MREPLACE_BASE} > >> -.if ${OSVERSION} >=3D 1000500 > >> +.if ${OPSYS} =3D=3D FreeBSD && ${OSVERSION} >=3D 1000500 > >=20 > > Given that OSVERSION is used extensively in the tree, I'm worrying that > > adding these extra "${OPSYS} =3D=3D 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 =3D=3D FreeBSD instead of patching every port? >=20 > 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. >=20 > 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. --=20 Eygene Ryabinkin ,,,^..^,,, [ Life's unfair - but root password helps! | codelabs.ru ] [ 82FE 06BC D497 C0DE 49EC 4FF0 16AF 9EAE 8152 ECFB | freebsd.org ] --J/zg8ciPNcraoWb6 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (FreeBSD) iF4EABEIAAYFAlKUwr4ACgkQFq+eroFS7PsllwD/c+huJ6Dk1mq5ZOr9PrMlud6s UUIAonMnCG+NQ17y9IQA/j7Y3gcvSB987dUxe2N5KFPV80nRbWJ0n7ueeJocla1a =opUh -----END PGP SIGNATURE----- --J/zg8ciPNcraoWb6--