On 2009-06-21 01:49, t...@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
_______________________________________________
macports-dev mailing list
macports-dev@lists.macosforge.org
http://lists.macosforge.org/mailman/listinfo.cgi/macports-dev

Reply via email to