Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 60563 - xmllint fixes for repoman did not make it into pre18
Summary: xmllint fixes for repoman did not make it into pre18
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Repoman (show other bugs)
Hardware: All All
: High normal
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2004-08-16 09:07 UTC by solar (RETIRED)
Modified: 2004-08-24 19:14 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description solar (RETIRED) gentoo-dev 2004-08-16 09:07:46 UTC 
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))
Comment 1 Brian Harring (RETIRED) gentoo-dev 2004-08-24 19:14:36 UTC 
Went out in pre20