Bug 243158 - x11/libICE: add DOCS option
Summary: x11/libICE: add DOCS option
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-x11 mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-01-07 13:12 UTC by Andrew Romanenko
Modified: 2020-01-07 18:41 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (x11)


Attachments
libICE.patch (865 bytes, patch)
2020-01-07 13:12 UTC, Andrew Romanenko
no flags Details | Diff
libICE_v0.patch (780 bytes, patch)
2020-01-07 13:40 UTC, Andrew Romanenko
no flags Details | Diff
libICE_v1.patch (758 bytes, patch)
2020-01-07 18:41 UTC, Andrew Romanenko
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Romanenko 2020-01-07 13:12:20 UTC
Created attachment 210500 [details]
libICE.patch

Add option for DOCS (disabled by default)
Comment 1 Niclas Zeising freebsd_committer 2020-01-07 13:15:54 UTC
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.
Comment 2 Andrew Romanenko 2020-01-07 13:40:09 UTC
Created attachment 210501 [details]
libICE_v0.patch

ok
Comment 3 Niclas Zeising freebsd_committer 2020-01-07 14:22:06 UTC
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.
Comment 4 Andrew Romanenko 2020-01-07 18:41:32 UTC
Created attachment 210510 [details]
libICE_v1.patch

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.