On Jun 7, 2013, at 09:52, [email protected] wrote:

> Revision: 106768
>          https://trac.macports.org/changeset/106768
> Author:   [email protected]
> Date:     2013-06-07 07:52:02 -0700 (Fri, 07 Jun 2013)
> Log Message:
> -----------
> active_variants: don't print depspec in user-visible error message

Thanks; that had been bugging me.


> 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-06-07 13:58:41 UTC (rev 106767)
> +++ trunk/dports/_resources/port1.0/group/active_variants-1.1.tcl     
> 2013-06-07 14:52:02 UTC (rev 106768)
> @@ -179,10 +179,11 @@
>       global _require_active_variants_list
>       foreach _require_active_variant $_require_active_variants_list {
>               set name [lindex $_require_active_variant 0]

Could we also rewrite this so that this variable is not named "name"? "name" is 
already a well-known option in portfile syntax; it is confusing to overload its 
meaning.

_______________________________________________
macports-dev mailing list
[email protected]
https://lists.macosforge.org/mailman/listinfo/macports-dev

Reply via email to