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 198287 - Review Request: python-paste - Tools for using a Web Server Gateway Interface stack
Summary: Review Request: python-paste - Tools for using a Web Server Gateway Interface...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Patrice Dumas
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 198288 198289
Blocks: FE-ACCEPT 189338
TreeView+ depends on / blocked
 
Reported: 2006-07-10 20:47 UTC by Luke Macken
Modified: 2016-09-20 02:37 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-19 20:17:00 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Luke Macken 2006-07-10 20:47:19 UTC
Spec URL: http://people.redhat.com/lmacken/python-paste.spec
SRPM URL: http://people.redhat.com/lmacken/python-paste-0.9.3-1.src.rpm
Description:
These provide several pieces of "middleware" (or filters) that can be nested
to build web applications.  Each piece of middleware uses the WSGI (PEP 333)
interface, and should be compatible with other middleware based on those
interfaces.

Comment 1 Patrice Dumas 2006-07-14 19:13:49 UTC
I had that for my Paste packaging attempt: 

# clean docs directory
pushd docs
rm StyleGuide.txt *.css */*.css rebuild doc.conf template.tmpl */*.js
popd

rpmlint has some errors that should certainly be fixed:
E: python-paste non-executable-script
/usr/lib/python2.4/site-packages/paste/util/scgiserver.py 0644
E: python-paste non-executable-script
/usr/lib/python2.4/site-packages/paste/debug/doctest_webapp.py 0644

For scgiserver.py I believe it should be patched to have the
shebang removed. For doctest_webapp.py it isn't obvious, maybe
it can be run standalone, in that case the permissions should 
be fixed.

Comment 2 Patrice Dumas 2006-07-15 09:59:00 UTC
There is an inconsistent use of RPM_BUILD_ROOT and %buildroot
(also in other Paste related python packages)

Comment 3 Luke Macken 2006-07-15 16:23:06 UTC
http://people.redhat.com/lmacken/python-paste.spec
http://people.redhat.com/lmacken/python-paste-0.9.3-2.src.rpm

* Sat Jul 15 2006 Luke Macken <lmacken> - 0.9.3-2
- Clean up docs directory
- Remove shebang from from non-executable scripts
- Use consistent build root variables


Comment 4 Patrice Dumas 2006-07-19 16:52:57 UTC
It seems to me that %doc would be better with

%doc docs/*

Comment 5 Luke Macken 2006-07-19 17:42:13 UTC
http://people.redhat.com/lmacken/python-paste.spec
http://people.redhat.com/lmacken/python-paste-0.9.3-3.src.rpm

* Wed Jul 19 2006 Luke Macken <lmacken> - 0.9.3-3
- Fix doc inclusion

Comment 6 Patrice Dumas 2006-07-19 19:25:53 UTC
I am not that comfortable with python packagin, but I have to
start one day...

As a first remark, the striping of shebangs with sed is not
wrong, but prone to errors since even if the first lines of 
these files isn't a shebang anymore there won't be any error.

* rpmlint output:
E: python-paste zero-length
/usr/lib/python2.4/site-packages/Paste-0.9.3-py2.4.egg-info/not-zip-safe

I guess it is a usefull file?

* package rightly named
* open source licence, licence not included but not in the 
  upstream package
* spec legible
* source match upstream
c1b8f08d0a7e9ccb97e6ee524b26d39f  Paste-0.9.3.tar.gz
* BuildRequires are clean
* Owns the paste directory it creates
* %files section right, using a python scriplet
* doc is small and don't affect runtime
* no gui no lib

APPROVED

Comment 7 Luke Macken 2006-07-19 20:17:00 UTC
I modified the shebang removal expression to be a bit smarter :)

     %{__sed} -i -e '/^#!.*/,1 d' paste/util/scgiserver.py
paste/debug/doctest_webapp.py

Imported into CVS, added to owners list, FC5 branch requested, tagged and built
for FC6.

Thanks!



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