Bug 842843 - Remove PyXML dep, add lxml
Summary: Remove PyXML dep, add lxml
Alias: None
Product: Fedora
Classification: Fedora
Component: xen
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Michael Young
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 843176
TreeView+ depends on / blocked
Reported: 2012-07-24 17:40 UTC by Toshio Ernie Kuratomi
Modified: 2012-08-05 22:50 UTC (History)
4 users (show)

Fixed In Version: xen-4.1.2-24.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-08-05 22:50:04 UTC
Type: Bug

Attachments (Terms of Use)
Remove pyxml prettyprint use (662 bytes, patch)
2012-07-24 17:40 UTC, Toshio Ernie Kuratomi
no flags Details | Diff

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.

Note You need to log in before you can comment on or make changes to this bug.