Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 367735

Summary: app-emulation/xen-tools: Prepare to use xml.use_pyxml()
Product: Gentoo Linux Reporter: Arfrever Frehtes Taifersar Arahesis (RETIRED) <arfrever>
Component: Current packagesAssignee: Gentoo Xen Devs <xen>
Status: RESOLVED FIXED    
Severity: normal CC: idella4, sven.koehler
Priority: High    
Version: unspecified   
Hardware: All   
OS: All   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 367729    
Attachments: xen-tools-4.1.2-pyxml.patch

Description Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2011-05-17 17:08:00 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 Ian Delaney (RETIRED) gentoo-dev 2011-09-09 11:05:47 UTC
don't know about editing python packages, but xen & xen-tools compiled properly against this python version
Comment 2 Arfrever Frehtes Taifersar Arahesis 2011-10-14 20:00:06 UTC
You should test run-time behavior after uninstallation of dev-python/pyxml.
Comment 3 Markos Chandras (RETIRED) gentoo-dev 2011-10-23 16:24:21 UTC
Reopening since they are not fixed yet and you fail to understand what this bug is all about
Comment 4 Ian Delaney (RETIRED) gentoo-dev 2011-10-30 07:41:01 UTC
ok,

I did unmerge pyxml, and xm still created a vm.  This appears to demo that pyxml dependency is a state in versions now well surpassed.  The whole python based xend portion of xen is on its way to being depracated and replaced with xl and its associated libraries.  The invalid status last reversed looks to hold a 99 % chance of being a correct call.

If this package actually needs PyXML,

does seem to be the  central issue.  Due to everyone being busy and providing less than comprehensive or convincing  info to qeuries put, allow me the time it takes to finally learn python so I can confirm it myself.
Comment 5 Arfrever Frehtes Taifersar Arahesis 2011-10-30 21:51:19 UTC
app-emulation/xen-tools should also depend on dev-python/pypam.
Comment 6 Arfrever Frehtes Taifersar Arahesis 2011-10-30 21:52:06 UTC
Created attachment 291309 [details, diff]
xen-tools-4.1.2-pyxml.patch
Comment 7 Ian Delaney (RETIRED) gentoo-dev 2011-11-01 14:55:33 UTC
Committed to the tree
Comment 8 Ian Delaney (RETIRED) gentoo-dev 2011-11-02 12:19:31 UTC
premature; ooopsie
Comment 9 Ian Delaney (RETIRED) gentoo-dev 2011-11-07 19:15:17 UTC
now fixed in the tree
Comment 10 Arfrever Frehtes Taifersar Arahesis 2011-11-07 23:19:25 UTC
Dependency on dev-python/pyxml should be restored.
Wrong patch has been committed.
Comment 11 Sven 2011-11-09 23:05:33 UTC
(In reply to comment #10)
> Wrong patch has been committed.

I have opened bug 390035 because of that
Comment 12 Ian Delaney (RETIRED) gentoo-dev 2011-11-10 02:25:10 UTC
dev notified and will replace
Comment 13 Ian Delaney (RETIRED) gentoo-dev 2011-11-10 11:30:53 UTC
*** Bug 390035 has been marked as a duplicate of this bug. ***
Comment 14 Tony Vroon (RETIRED) gentoo-dev 2011-11-10 12:12:52 UTC
Reopening due to incorrect patch having been committed.
Comment 15 Ian Delaney (RETIRED) gentoo-dev 2011-11-10 18:45:20 UTC
now in the tree