Bug 842843

Summary: Remove PyXML dep, add lxml
Product: [Fedora] Fedora Reporter: Toshio Ernie Kuratomi <a.badger>
Component: xenAssignee: Michael Young <m.a.young>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: jforbes, kraxel, m.a.young, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: xen-4.1.2-24.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-05 22:50:04 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 843176    
Description Flags
Remove pyxml prettyprint use none

Description Toshio Ernie Kuratomi 2012-07-24 17:40:50 UTC
Created attachment 600143 [details]
Remove pyxml prettyprint use

Description of problem:

We're looking into getting rid of PyXML in Fedora 18 as it's dead upstream, suffers unfixed bugs, and has the dubious behaviour of overwriting the python stdlib's  xml module with its own code: https://bugzilla.redhat.com/show_bug.cgi?id=795875

Reviewing xen's use of PyXML, I have the following recommendations:

* It looks like upstream xen actively ported away from PyXML to lxml.  If that's complete the fedora package just has old deps that need to be updated:

- Requires: PyXML
+ Requires: python-lxml 

Adding Requires: python-lxml needs to be done whether or not the PyXML Requires can be removed right away.

Reviewing the code, I also found one place that's still using a PyXML module.  xm create with certain options can attempt to use PyXML to PrettyPrint its output.  I'll attach a patch to trivially fix this to use stdlib functionality instead.

Version-Release number of selected component (if applicable):

Comment 1 Fedora Update System 2012-07-26 22:56:44 UTC
xen-4.1.2-24.fc17 has been submitted as an update for Fedora 17.

Comment 2 Fedora Update System 2012-07-27 09:53:32 UTC
Package xen-4.1.2-24.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing xen-4.1.2-24.fc17'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).

Comment 3 Fedora Update System 2012-08-05 21:26:08 UTC
xen-4.1.2-24.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.