Bug 842850 - Remove dep on salt
Summary: Remove dep on salt
Alias: None
Product: Fedora
Classification: Fedora
Component: salt
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Clint Savage
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 843176
TreeView+ depends on / blocked
Reported: 2012-07-24 18:01 UTC by Toshio Ernie Kuratomi
Modified: 2012-08-21 16:34 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-08-21 16:34:33 UTC
Type: Bug

Attachments (Terms of Use)

Description Toshio Ernie Kuratomi 2012-07-24 18:01:44 UTC
Description of problem:

We're trying to get rid of PyXML in Fedora 18 due to not being maintained upstream, having outstanding bugs, and having the dubious behaviour of overriding the python stdlib's xml module when it is installed.

I've taken a look at salt's Requirement on PyXML and I think we can get rid of it.  salt is using xml.dom.minidom which is available in the stdlib's xml module.  No need for PyXML.  (We can probably remove the Requires: python26-PyXML as well).

Comment 1 Clint Savage 2012-07-24 18:07:50 UTC
Let me have a conversation with the upstream maintainer and see what we can do, but I think you are probably right.

Comment 2 Thomas S Hatch 2012-07-24 22:34:51 UTC
Salt does not have a dep on PyXML. It is only used if someone wants the test suite and output the data to XML. This was added only to integrate the test suite with Jenkins. Removal of the PyXML dep would have no negative bearing on the operation of Salt and no other distro packages require PyXML.

- Thomas S Hatch

Comment 3 Clint Savage 2012-07-29 21:57:22 UTC
Since 0.10.2 is coming out shortly, I'll apply the fix to the spec file there.

Comment 4 Toshio Ernie Kuratomi 2012-08-21 16:34:33 UTC
Looks fixed.  I'll close this bug.  Thanks herlo!

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