Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 216105 - Review Request: python-twisted-web - Twisted web server, programmable in Python
Summary: Review Request: python-twisted-web - Twisted web server, programmable in Python
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Paul Howarth
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 171543 poker-network 221310
TreeView+ depends on / blocked
 
Reported: 2006-11-17 10:16 UTC by Thomas Vander Stichele
Modified: 2008-02-20 20:29 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-28 23:27:58 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Thomas Vander Stichele 2006-11-17 10:16:31 UTC
Spec URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-web-0.6.0-3.fc5/python-twisted-web.spec
SRPM URL: http://thomas.apestaart.org/download/pkg/fedora-5-x86_64-extras/python-twisted-web-0.6.0-3.fc5/python-twisted-web-0.6.0-3.fc5.src.rpm
Description:
Twisted is an event-based framework for internet applications.

Twisted Web is a complete web server, aimed at hosting web
applications using Twisted and Python, but fully able to serve static
pages, also.

Comment 1 Paul Howarth 2006-11-22 16:44:16 UTC
Review
======

- package and spec file naming OK
- package meets guidelines
- license is MIT, matches spec
- spec file written in English and is legible
- sources match upstream
- package builds OK in mock for FC4-Rawhide, i386 and x86_64
- BR's OK
- no locale issues
- no shared libraries
- not relocatable
- no directory ownership or permissions issues
- %clean section present and correct
- macro usage consistent enough
- code, not content
- docs aren't excessively large
- docs don't affect runtime
- no subpackages
- no devel files present
- not a GUI app, no desktop file needed
- package seems to work OK (tested with bittorrent 5.0.1)

MUSTFIX:

* Include LICENSE as %doc

PROBLEM:

# rpm -qa python-twisted\*
python-twisted-web-0.6.0-4.fc6
python-twisted-core-2.4.0-4.fc6
# rpm -e python-twisted-web
Traceback (most recent call last):
  File "/usr/libexec/twisted-dropin-cache", line 16, in ?
    list(getPlugins(IPlugin))
--- <exception caught here> ---
  File "/usr/lib64/python2.4/site-packages/twisted/plugin.py", line 214, in
getPlugins
    adapted = interface(plugin, None)
  File "/usr/lib64/python2.4/site-packages/zope/interface/interface.py", line
658, in __call__
    adapter = conform(self)
  File "/usr/lib64/python2.4/site-packages/twisted/plugin.py", line 77, in
__conform__
    return self.load()
  File "/usr/lib64/python2.4/site-packages/twisted/plugin.py", line 72, in load
    return namedAny(self.dropin.moduleName + '.' + self.name)
  File "/usr/lib64/python2.4/site-packages/twisted/python/reflect.py", line 370,
in namedAny
    obj = getattr(obj, n)
exceptions.AttributeError: 'module' object has no attribute 'twisted_web'

I think that needs fixing.

NOTES:

* rpmlint output:
E: python-twisted-web no-binary
  This is to be expected for what is essentially a noarch package that has to
  live in an arch-specific place due to the way python works, i.e. not an issue

SUGGESTIONS:

* URL should be: http://twistedmatrix.com/trac/wiki/TwistedWeb
* Include NEWS as %doc



Comment 2 Kevin Fenzi 2006-12-16 19:46:00 UTC
Hey Thomas. 

Any news on the mustfix/suggestion items from this review?


Comment 3 Jef Spaleta 2006-12-23 03:41:17 UTC
thanks to python 2.5 now in the development tree you'll need a new 
Buildrequires: python-devel

You'll find the updated srpm and spec  which builds against current
fedora-development here:

http://jspaleta.thecodergeek.com/Fedora%20SRPMS/python-twisted-web/python-twisted-web-0.6.0-4.src.rpm
http://jspaleta.thecodergeek.com/Fedora%20SRPMS/python-twisted-web/python-twisted-web.spec

I've also included the LICENSE and NEWS fileas as %doc and updated the URL tag
as suggested in the review comment



Comment 4 Thomas Vander Stichele 2006-12-26 12:40:20 UTC
Paul, Kevin,

pulled in Jef's updated spec as is.

Any further comments, or can this be APPROVEd ?

Comment 5 Paul Howarth 2006-12-26 13:26:57 UTC
Has anybody else seen the problem with the dropin cache at package removal time
that I mentioned in Comment #1?

Comment 6 Thomas Vander Stichele 2006-12-26 15:12:24 UTC
I removed all my packages (including the divmod ones) and now I am able to
reproduce it.  Before I had it spuriously and couldn't figure out a way to debug it.

I dug a little deeper now and it seems that the Twisted code logs an exception
using its own log handler when it finds a plugin in the cache that is no longer
on disk.

Since this is the point of updating the cache - removing plugins that no longer
exist - I changed twisted-dropin-cache to not log on that particular error.

I pushed a new python-twisted-core package to extras that should fix this.

Comment 7 Kevin Fenzi 2006-12-26 20:28:12 UTC
I don't see any more blockers here... using the new python-twisted-core from the
buildsys (it's not been pushed yet) I don't see any dropin cache issues.



Comment 8 Paul Howarth 2006-12-27 12:45:58 UTC
OK, Approved.

Comment 9 Thomas Vander Stichele 2006-12-27 13:58:17 UTC
imported, waiting for fc-6 branch

Comment 10 Thomas Vander Stichele 2006-12-28 23:27:58 UTC
built, closing

Comment 11 Lubomir Kundrak 2008-02-17 16:15:48 UTC
Thomas: I'm interested in EL-5 branch. Would you maintain it yourself or you
won't mind if I maintained it?

Comment 12 Lubomir Kundrak 2008-02-20 11:45:39 UTC
This is a blocker for my package and the Fedora maintainer seems not to be
interested. cvsextras commits will be open and I'm willing aprove the Fedora
maintainer once he applies for ACLs.

Package Change Request
======================
Package Name: python-twisted-web
New Branches: EL-5
Cvsextras Commits: yes

Comment 13 Kevin Fenzi 2008-02-20 20:14:40 UTC
cvs done.


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