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

[Clemens Lang]

On Mon, Feb 04, 2013 at 02:51:44PM -0600, Ryan Schmidt wrote:
> > pre-configure {
> > 	_check_require_active_variants
> > }
> 
> Shouldn't it be doing this in pre-build as well?

It's already doing this in pre-configure, shouldn't that be enough? One
could never run the build phase without running the configure phase and
one could never change variants between configure and build.

As for changing variants on a dependency between configure and build, we
might already have similar problems when a dependency doesn't change
variants, but it upgraded; e.g. configuring cyrus-sasl2, updating
kerberos5 and then building cyrus-sasl2 might also fail, and we don't
check for that situation either.

-- 
Clemens Lang