while the command "cd /usr/ports/x11-themes/slim-themes; make -V PORT_OPTIONS" might led one to believe the magic used to generate the options is valid, unfortunately it is not. The PORTS_OPTIONS cannot be manipulated after bsd.ports.pre.mk is included; any change after that point isn't seen. the prove is this: make -V SELECTED_OPTIONS (blank) make -V UNSELECTED_OPTIONS ALL_THEMES If OPTIONS_DEFINE were set legally, they would be distributed across SELECTED_OPTIONS and UNSELECTED_OPTIONS. Unfortunately another method needs to be found with the last result as just entering the options by hand. with the exception of another couple of ports and slim-themes, all of ports in the tree have valid SELECTED_OPTIONS/UNSELECTED_OPTIONS.
correction: it's supposed to be DESELECTED_OPTIONS which is actually showing the expected values. There was another port similar to this one, mail/thunderbird-dictionaries, which I just fixed. So this is last port I'm aware of with invalid option settings.
by the way, the WWW site is gone (berlios) so that also needs to be replaced or removed ...
I think I can fix this without too many changes. Let me test my idea and if it passes the build tests, I'll post the patch.
Created attachment 173303 [details] Fix slim-themes options Okay, this works for me. I did remove the "ALL_THEMES" options. It doesn't buy anything (all options are "ON" by default anyway) and it can actually be confusing when ALL_THEMES is set but an individual theme is unchecked (it would have still installed). Moreover, using the MULTI option enforces the "at least one selected" requirement so it's also less complex. On your approval, I can commit it.
A commit references this bug: Author: marino Date: Thu Aug 18 06:02:13 UTC 2016 New revision: 420386 URL: https://svnweb.freebsd.org/changeset/ports/420386 Log: x11-themes/slim-themes: fix invalid option definitions Defining PORT_OPTIONS directly, especially after makefile fragment inclusions, is not valid and results in missing options values (yet masked when they are evaluated later after they are needed). Rework the automagic to function as intended with the exception of removing the redundant and potentially misleading "ALL_THEMES" option. All the themes are already set on by default and to change that required the extra step of unchecking ALL_THEMES (without doing so still resulted in all being selected even when options are unchecked which is confusing). Finally, use the option framework to enforce one choice with MULTI option rather than roll-your-own logic. PR: 211569 Approved by: maintainer timeout Discovered by: Synth sanity check failure Changes: head/x11-themes/slim-themes/Makefile