Summary: | x11-misc/xdg-utils-1.0.2 doesn't need app-text/xmlto | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Arfrever Frehtes Taifersar Arahesis (RETIRED) <arfrever> |
Component: | New packages | Assignee: | Freedesktop bugs <freedesktop-bugs> |
Status: | VERIFIED FIXED | ||
Severity: | trivial | CC: | martin, tanderson |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Arfrever Frehtes Taifersar Arahesis (RETIRED)
2007-08-20 17:45:23 UTC
Weird, I wonder why it bothers to check for it then.. Anyway, fixed. if it checks for it, it needs to be in DEPENDS.. It most likely does need it for documentation which is configurable via ./configure. That check is broken. I have no /usr/bin/xmlto, but the output of configure is: checking whether make sets $(MAKE)... yes checking for a BSD-compatible install... /usr/bin/install -c checking for xmlto... /usr/bin/xmlto configure: creating ./config.status config.status: creating Makefile config.status: creating scripts/Makefile config.status: creating tests/Makefile Yeah I just looked into it, the check is flawed since it just defaults a value when it can't find it to continue on. This is technically wrong. Also the files that would be used by xmlto aren't even shipped in the tarball since the man pages (which is what xmlto is used for) are pre-generated. May I ask why the xmlto dependency was re-added in xdg-utils-1.0.2-r3? This is quite inconvenient since xdg-utils is currently the only "basic" package depending unconditionally on xmlto which in turn has a huge bulk of dependencies (docbook-xml-dts, docbook-xsl-stylesheets, sgml-common, build-docbook-catalog, automake-1.5*). So far, I never had any problems without xmlto, and in accordance with comment #4 there is no obvious reason why xmlto should be needed at all for xdg-utils (especially without the doc useflag). 31 May 2009; Thomas Anderson <gentoofan23@gentoo.org> xdg-utils-1.0.2-r3.ebuild: Add missing dependency on app-text/xmlto. @thomas: please comment. I've fixed that. Sorry for the added dep, it looked to me like it really was needed. Thanks for the dependency cleanup. |