svn commit: r334593 - in head/dns: bind96 bind96/files bind98 bind98/files bind99 bind99/files

[Eygene Ryabinkin]

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.
-- 
Eygene Ryabinkin                                        ,,,^..^,,,
[ Life's unfair - but root password helps!           | codelabs.ru ]
[ 82FE 06BC D497 C0DE 49EC  4FF0 16AF 9EAE 8152 ECFB | freebsd.org ]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20131126/27f10ee8/attachment.sig>