[112517] trunk/dports/tex/pTeX

[Takanori Yamamoto]

I forgot about the limitation. Thanks for letting me know.

On 2013/10/25, at 21:47, Ryan Schmidt wrote:

> 
> On Oct 25, 2013, at 01:43, takanori at macports.org wrote:
> 
>> +variant hiragino conflicts yu description {Use Hiragino fonts for typesetting} {}
>> +variant yu conflicts hiragino description {Use Yu fonts for typesetting (experimental; not recommended)} {
>> +    if {${os.platform} == "darwin" && ${os.major} >= 13} {
>> +        # nop
>> +    } else {
>> +        return -code error "variant +yu is supported on Mavericks or later only"
>> +    }
>> 
>> }
> 
> You should only “return -code error” inside a phase (e.g. enclose the “return -code error” in a “pre-fetch” block). Otherwise:
> 
> $ port info pTeX +yu
> Error: pTeX: Error executing yu: variant +yu is supported on Mavericks or later only
> Error: Unable to open port: Error evaluating variants
> 
> Imagine what would happen if a user had “+yu” in their variants.conf: operations like “sudo port clean all” would fail with this error at pTeX.
>