Bug 842850 - Remove dep on salt
Remove dep on salt
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: salt (Show other bugs)
rawhide
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Clint Savage
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 843176
  Show dependency treegraph
 
Reported: 2012-07-24 14:01 EDT by Toshio Ernie Kuratomi
Modified: 2012-08-21 12:34 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-08-21 12:34:33 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)

  None (edit)
Description Toshio Ernie Kuratomi 2012-07-24 14:01:44 EDT
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 14:07:50 EDT
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 18:34:51 EDT
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 17:57:22 EDT
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 12:34:33 EDT
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.