[52668] trunk/dports/security/cyrus-sasl2/Portfile

[Rainer Müller]

On 2009-06-21 01:49, toby at macports.org wrote:
> --- trunk/dports/security/cyrus-sasl2/Portfile	2009-06-20 23:35:06 UTC (rev 52667)
> +++ trunk/dports/security/cyrus-sasl2/Portfile	2009-06-20 23:49:38 UTC (rev 52668)
> @@ -84,16 +84,14 @@
>      file copy ${worksrcpath}/doc ${destroot}${prefix}/share/doc/${name}
>  }
>  
> -platform darwin {}
> -platform freebsd {}
>  variant kerberos description "Enable Kerberos support (default)" {
>      configure.args-append   --enable-gssapi \
>                              --enable-keep-db-open
>  
> -	if {[variant_isset darwin]} {
> +	if {${os.platform} == "darwin"} {
>  		configure.args-append --with-gss_impl=mit
>  	}
> -	if {[variant_isset freebsd]} {
> +	if {${os.platform} == "freebsd"} {
>  		configure.args-append --with-gss_impl=heimdal
>  	}
>  }

I think we discussed this before long time ago and agreed to prefer
using platform variants in such cases. Otherwise the installed version
does not indicate at all that parts of it are platform specific. With
using platform variants, this would have +darwin/+freebsd.

At the moment this is not important for most users, but if someone wants
to distribute archives or binaries it will.

Rainer