| Summary: | xmllint fixes for repoman did not make it into pre18 | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | solar (RETIRED) <solar> |
| Component: | Repoman | Assignee: | Portage team <dev-portage> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | Keywords: | InVCS |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
Went out in pre20 |
The very thing that held pre18 on Aug 16 seems to be missing. As we can see the code still reads as st=getstatusoutput("xmllint --nonet '%s/metadata.xml' --dtdvalid '%s/metadata.dtd' 2>&1 > /dev/null" % (checkdir,portage.CACHE_PATH)) when it should be st=getstatusoutput("cat '%s/metadata.xml' | sed -e 's|http://www.gentoo.org/dtd/metadata.dtd|%s/metadata.dtd|' | xmllint --valid - 2>&1 > /dev/null" % (checkdir,portage.CACHE_PATH)) or st=getstatusoutput("grep -v 'metadata.dtd' '%s/metadata.xml' | xmllint --dtdvalid '%s/metadata.dtd' - 2>&1 > /dev/null" % (checkdir,portage.CACHE_PATH))