Don't print messages inside platform directives

[Ryan Schmidt]

On May 25, 2007, at 17:14, Ryan Schmidt wrote:

> On May 25, 2007, at 16:49, N_Ox wrote:
>
>> Le 25 mai 07 à 23:44, Ryan Schmidt a écrit :
>>
>>> platform darwin 8 {}
>>> pre-fetch {
>>> 	if { [variant_isset darwin_8] } {
>>> 		ui_error "==== ATTENTION!!!! ===="
>>> 		ui_error "Sockstat is not working under darwin 8 nor tiger.  
>>> Please use"
>>> 		ui_error "lsof -i6/lsof -i4/lsof -U instead."
>>> 		ui_error "==== ATTENTION!!!! ===="
>>> 		ui_error "Please sudo port uninstall sockstat"
>>> 	}
>>> }
>>
>> there's just something bothering me with platform/variant  
>> directives i've seen here and there.
>> Why do you use a dummy platform and use variant_isset in the stage  
>> directive?
>>
>> From what i've seen (and coded, of course), every stage in every  
>> activated variant/platform is executed.
>
> If you don't declare "platform darwin 8" to be *something*, even an  
> empty "{}", then "[variant_isset _darwin_8]" will be false and the  
> message will not be printed. Try it out. I just did.

Or perhaps it would be better and clearer to use the syntax shown in  
the ipcs portfile:

platform darwin 8 {
	pre-fetch {
		ui_msg "\n
			This port is broken on OS X 10.4, but not necessary on 10.4 either  
because
			beginning with 10.4, Apple includes the ipcs and ipcrm utilities  
as standard.\n"
		exit 1
	}
}