Summary: | dev-libs/openssl FEATURES misuse (noman) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Thilo Bangert (RETIRED) (RETIRED) <bangert> |
Component: | New packages | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED WONTFIX | ||
Severity: | QA | CC: | didier-bugzillagentoo, ferringb |
Priority: | High | Keywords: | QAcanfix |
Version: | 1.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 174335 | ||
Attachments: |
Replace FEATURES/noman by doc USE flag for dev-libs/openssl-0.9.8l-r2 ebuild
Remove test for noman in FEATURES |
Description
Thilo Bangert (RETIRED) (RETIRED)
2009-07-22 19:13:25 UTC
Created attachment 218851 [details, diff]
Replace FEATURES/noman by doc USE flag for dev-libs/openssl-0.9.8l-r2 ebuild
Here is a patch against =dev-libs/openssl-0.9.8l-r2 to replace the test for noman in FEATURES by a doc USE flag.
I chose the doc USE flags because it is described as "Adds extra documentation (API, Javadoc, etc)" but it could easily changed if needed. The doc USE flag is on by default to preserve backward compatibility for most people.
To emerge =dev-libs/openssl-0.9.8l-r2 ebuild with man pages:
# emerge -av1 '=dev-libs/openssl-0.9.8l-r2'
To emerge =dev-libs/openssl-0.9.8l-r2 ebuild without man pages:
# USE=-doc emerge -av1 '=dev-libs/openssl-0.9.8l-r2'
Similar patch could be created for =dev-libs/openssl-1.0.0_beta5 but I wait for feedback on the patch for =dev-libs/openssl-0.9.8l-r2 to do the job.
Any comments ?
Comment on attachment 218851 [details, diff]
Replace FEATURES/noman by doc USE flag for dev-libs/openssl-0.9.8l-r2 ebuild
USE=doc does not cover man pages
(In reply to comment #2) > (From update of attachment 218851 [details, diff]) > USE=doc does not cover man pages > Is there another USE flag I can use instead of doc ? Is USE=man OK ? The ebuild should always install man pages. A user can use the following setting in /etc/make.conf to not install man pages: INSTALL_MASK="/usr/share/man" (In reply to comment #4) > The ebuild should always install man pages. > A user can use the following setting in /etc/make.conf to not install man > pages: > > INSTALL_MASK="/usr/share/man" > I see you point. I'm going to attach a patch for the =dev-libs/openssl-0.9.8n ebuild to simply remove the test for noman in FEATURES. Created attachment 225465 [details, diff]
Remove test for noman in FEATURES
The FEATURES usage here is fine; the pkg doesn't rely on FEATURES awareness, it optionally flips off manpage generation from even occuring via it. |