Summary: | app-emulation/qemu-kvm: could you please add 'doc' USE? | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | kulti <kultihell> |
Component: | New packages | Assignee: | Gentoo QEMU Project <qemu+disabled> |
Status: | RESOLVED CANTFIX | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=435534 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
kulti
2012-04-29 08:15:29 UTC
Well we always include that depend and the dohtml call is always there. I've added USE=doc to call --enable-docs for the configure script in the qemu-kvm-1.1.0 ebuild Looks like partialy fix... I can't understandd, why app-text/texi2html dependencies still needed? I fix it in local overlay to doc? ( app-text/texi2html ) Was built successfully. (In reply to comment #2) > Looks like partialy fix... I can't understandd, why app-text/texi2html > dependencies still needed? > > I fix it in local overlay to > > doc? ( app-text/texi2html ) > > Was built successfully. Not really sure where you would have had to add that since we've already got: DEPEND="${RDEPEND} app-text/texi2html virtual/pkgconfig kernel_linux? ( >=sys-kernel/linux-headers-2.6.35 )" And its certainly not a run time depend, its used at build time. Yes, it's build dependencies, but I haven't texi2html in my system and not need it. So I cannot see any reason to stay hold it in build-dependency, when we can do it optional with doc USE-flag. @@ -99,7 +99,7 @@ RDEPEND=" xfs? ( sys-fs/xfsprogs )" DEPEND="${RDEPEND} - app-text/texi2html + doc? ( app-text/texi2html ) virtual/pkgconfig kernel_linux? ( >=sys-kernel/linux-headers-2.6.35 )" Still need to be fixed. Please review my patch for texi2html dependency: DEPEND="${RDEPEND} - app-text/texi2html + doc? ( app-text/texi2html ) virtual/pkgconfig kernel_linux? ( >=sys-kernel/linux-headers-2.6.35 )" Fixed in tree. Due to bug #435534, I'm going to have to revert this. This prevents man pages from being installed. Unfortunately there's only a big hammer for all docs (including man pages) and since we always want to install man pages this must always be on. You are more than welcome to submit a patch upstream which breaks out man pages from other doc generation. Reverted in qemu-1.4.2, due to #435534 |