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] Suggested enhancements
http://download.fedora.redhat.com/pub/fedora/linux/core/development/SRPMS/python-twisted-1.3.0-4.src.rpm 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. http://cvs.fedora.redhat.com/viewcvs/devel/ 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 to 2.0. 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 :)