PortIndex2MySQL run failure on Sunday 2016-03-06 at 07:00:01

[Ryan Schmidt]

> On Mar 6, 2016, at 12:15 PM, Mojca Miklavec <mojca at macports.org> wrote:
> 
> On 6 March 2016 at 18:44, Ryan Schmidt wrote:
>> Begin forwarded message:
>> 
>> From: macports-mgr at lists.macosforge.org
>> Subject: PortIndex2MySQL run failure on Sunday 2016-03-06 at 07:00:01
>> Date: March 6, 2016 at 9:00:18 AM CST
>> To: admin at macosforge.org
>> 
>> 
>> Synchronizing local ports tree from
>> rsync://rsync.macports.org/release/tarballs/ports.tar
>> receiving file list ... done
>> ports.tar
>> 
>> sent 55810 bytes  received 97 bytes  37271.33 bytes/sec
>> total size is 63344640  speedup is 1133.04
>> receiving file list ... done
>> ports.tar.rmd160
>> 
>> sent 54 bytes  received 101 bytes  103.33 bytes/sec
>> total size is 512  speedup is 3.30
>> Creating port index in
>> /opt/local/var/macports/sources/rsync.macports.org/release/tarballs/ports
>> Failed to parse file games/angband/Portfile: Error evaluating variants
>> Warning: failed to open old entry for graphics/wxWidgets-3.0, making a new
>> one
>> Adding port graphics/wxWidgets-3.0
>> Adding subport wxWidgets-3.0-libcxx
>> Adding subport wxPython-3.0
>> Failed to parse file graphics/wxWidgets-3.0/Portfile with subport
>> 'wxgtk-3.0': Registry error: gtk3 not registered as installed & active.
>> 
>> Total number of ports parsed: 5
>> Ports successfully parsed: 3
>> Ports failed: 2
>> Up-to-date ports skipped: 19240
> 
> Whooops, I'm sorry, I didn't think about the possibility of this scenariot.
> 
> Are you able to help me out to come up with a proper patch or should
> we simply replace
> 
>        if {[active_variants gtk3 quartz ""]} {
>            default_variants-append +quartz
>        } else {
>            default_variants-append +x11
>        }
> 
> by
> 
>        default_variants-append +x11
> 
> ?
> 
> Feel free to do the commit if you want. We would probably need to add
> an additional "if gtk is active" before "if {[active_variants gtk3
> quartz ""]}", right?

I haven't looked at your code but the usual thing to write would be:

       if {[!variant_isset quartz]} {
           default_variants-append +x11
       }

https://trac.macports.org/wiki/PortfileRecipes#default_variants