Created attachment 210500 [details]
Add option for DOCS (disabled by default)
I see little reason for this, considering it's only 2 files that are affected. However, they should be installed by default (the DOCS option should be enabled), this way the package isn't changed.
Created attachment 210501 [details]
The patch isn't complete, %%PORTDOCS%%%%DOCSDIR%%/ice.xml is still installed into the stage dir even when DOCS is off.
You don't need to set DOCS in OPTIONS_DEFAULT, it's added automatically by the framework.
Created attachment 210510 [details]
OPTIONS_DEFAULT removed, %%PORTDOCS%%%%DOCSDIR%%/ice.xml is not installed if DOCS is off (build via poudriere 12.1/STABLE amd64).
thanks for DOCS is on by default via framework.
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/111247407
This is very little benefit for a lot of complexity. It is just two files in the package, text files that compresses quite well, but it means that on every update or test I need to test both the case with or without the DOCS option, doubling the number of tests needed.