Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 367759 - gnome-extra/gdesklets-core: Prepare to use xml.use_pyxml()
Summary: gnome-extra/gdesklets-core: Prepare to use xml.use_pyxml()
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Joe Sapp (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: xml.use_pyxml
  Show dependency tree
 
Reported: 2011-05-17 17:22 UTC by Arfrever Frehtes Taifersar Arahesis (RETIRED)
Modified: 2011-10-08 17:41 UTC (History)
1 user (show)

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 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2011-05-17 17:22:06 UTC 
This package specifies dependency on dev-python/pyxml.

Please test this package with dev-lang/python-2.7.1-r2:2.7 or >=dev-lang/python-2.7.2_pre20110515:2.7, which require explicit calling of xml.use_pyxml() before using PyXML.

If this package actually needs PyXML, then modify all modules/scripts of this package, which import xml module. The following code should be added before first import of anything from xml module:

import xml
if hasattr(xml, "use_pyxml"):
    xml.use_pyxml()

This code works with previous versions of Python, so no changes in dependencies are needed.
Comment 1 Joe Sapp (RETIRED) gentoo-dev 2011-06-26 18:50:15 UTC 
Is this a gentoo or an upstream requirement?
Comment 2 Joe Sapp (RETIRED) gentoo-dev 2011-06-26 18:57:33 UTC 
Nevermind, I just saw the thread on -dev.  It would be nice to have a link to the thread in the tracker bug.
Comment 3 Joe Sapp (RETIRED) gentoo-dev 2011-06-27 02:23:34 UTC 
I think the dep for pyxml can be dropped, but I will investigate some more.  It looks like gdesklets only uses the sax module, which appears to be in the python distribution.
Comment 4 Joe Sapp (RETIRED) gentoo-dev 2011-10-08 17:41:19 UTC 
Fixed in CVS.  Turns out they don't need it with any python version in portage.