[102552] trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl

[Ryan Schmidt]

On Feb 4, 2013, at 14:39, cal at macports.org wrote:

> Revision: 102552
>          https://trac.macports.org/changeset/102552
> Author:   cal at macports.org
> Date:     2013-02-04 12:39:00 -0800 (Mon, 04 Feb 2013)
> Log Message:
> -----------
> active_variants 1.1: enforce require_active_variants constrations before archivefetch
> 
> Previously, the constraints were only checked in pre-configure, which was,
> however, never run when installing from a binary package.
> Closes #37936.
> 
> Modified Paths:
> --------------
>    trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl
> 
> Modified: trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl
> ===================================================================
> --- trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl	2013-02-04 20:05:55 UTC (rev 102551)
> +++ trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl	2013-02-04 20:39:00 UTC (rev 102552)
> @@ -188,3 +188,10 @@
> pre-configure {
> 	_check_require_active_variants
> }
> +
> +# register pre-archivefetch handler that checks for all requested variants
> +# this is required when downloading binary archives for a package, because
> +# pre-configure is never run for those
> +pre-archivefetch {
> +	_check_require_active_variants
> +}

Shouldn't it be doing this in pre-build as well?