Red Hat Bugzilla – Bug 156453
[PATCH] python-twisted enhancements
Last modified: 2007-11-30 17:11:05 EST
Here's a patch that merges some enhancements from my old python-twisted package
to the one in Extras.
- IIRC python-crypto and pyOpenSSL are required for a lot of the
package's functionality, so added the deps.
- IIRC the freevo rpms require "Twisted", so added the provides.
- Cleanups to installed stuff and docs.
- Honoring $RPM_OPT_FLAGS.
Oh, and by the way, twisted 2.0 is out.
Created attachment 113889 [details]
Is this is the same python-twisted in FC4 core?
Yep, the core one is a derivative of this package, and it looks most of my patch
is applicable for it too. I had no idea of this being in core nowadays.
By the way, there's no python-twisted available in the public FC CVS.
If python-twisted will ship in FC4, we'll want to remove it from Extras.
Please provide a patch for the exact python-twisted in rawhide and make the
request to remove it from Extras.
FWIW, it's in Core only for the Xen tools at this point. They don't work with
2.0 based on a mail to xen-devel from one of the twisted developers and there's
work underway to have twisted no longer be a dependency for the Xen control
tools. Once that happens, twisted can ping pong back to extras and be updated
I'm not against adding patches to it while it's there though :)
I had provides: Twisted originally as well, I was asked to remove it since no
released package requires it (and apparently "but other people have made a
package requiring it" was not an argument).
I'm pretty sure I mentioned the following point last time this patch was
proposed (in f.us bugzilla), but please *don't* remove the trial stuff !!! It is
an important part of twisted and used by other projects, like buildbot and
flumotion. It is *not* simply only internal testsuite stuff.
as for Twisted 2.0, Jeff Pitman has made packages after Twisted's splitup - they
should probably be used as a base. I'm not sure if he's done with them now.
Feel free to close this if you don't agree with the patch, I have no problem
with that. Just submitted it in case some of it would be useful now that I
finally got rid of my local (unmaintained) package.
I don't disagree with everything in the patch, I just want to make sure the
trial stuff stays in there :)