Bug 451405 - Review Request: python-weberror - Web Error handling and exception catching middleware
Summary: Review Request: python-weberror - Web Error handling and exception catching m...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
Depends On: 451406
TreeView+ depends on / blocked
Reported: 2008-06-14 07:07 UTC by Ricky Zhou
Modified: 2014-10-13 23:00 UTC (History)
5 users (show)

Fixed In Version: 0.10.1-1.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-02-24 05:15:19 UTC
Type: ---
j: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Ricky Zhou 2008-06-14 07:07:17 UTC
Spec URL: http://ricky.fedorapeople.org/pkgs/python-weberror/python-weberror.spec
SRPM URL: http://ricky.fedorapeople.org/pkgs/python-weberror/python-weberror-0.8-1.fc9.src.rpm
Description: WebError is WSGI middleware that performs error handling and exception catching.

Comment 1 Ricky Zhou 2008-06-14 07:12:13 UTC
Add dependency on the python-tempita review bug (#451406).  This will also
depend on WebOb (which hasn't been submitted yet).  

Comment 2 Jason Tibbitts 2008-06-21 20:05:31 UTC
Currently this builds but doesn't install due to the missing WebOb dependency. 
Did you intend to submit that or was someone else going to do so?

Comment 3 Ricky Zhou 2008-06-21 20:11:07 UTC
Yup, this was intentional (as part of an effort to get everything packaged to be
ready to support turbogears2).  I spoke to spot, who had already packaged, but
not submitted a WebOb package, and I'll be submitting that package myself shortly.  

Comment 4 Brian Pepple 2008-07-08 00:21:13 UTC
Remove it from the queue until python-Web0b is submitted; just clear the
whiteboard when it's ready.

Comment 5 Ricky Zhou 2008-07-17 04:46:35 UTC
Readd to queue (python-webob has been submitted)

Some changes:
* Thu Jul 17 2008 Ricky Zhou <ricky> - 0.8-3
- Update Requires for python-webob rename.
- Add BuildRequires on python-webob and python-webtest.

* Mon Jul 07 2008 Ricky Zhou <ricky> - 0.8-2
- Add %%check section.

Spec URL: http://ricky.fedorapeople.org/pkgs/python-weberror/python-weberror.spec

Comment 6 Luke Macken 2008-10-21 20:10:08 UTC

* Tue Oct 21 2008 Luke Macken <lmacken> - 0.9-1
- Update to 0.9
- Add python-tempita to the BuildRequires

Comment 7 Luke Macken 2008-10-21 21:45:33 UTC

* Tue Oct 21 2008 Luke Macken <lmacken> - 0.9-2
- Add a patch to remove the wsgiref requirement, as it is now in Python2.5

Comment 8 Jason Tibbitts 2008-12-07 03:10:24 UTC
This one has been sitting for a bit.  I'll take a look.

It builds fine on rawhide; rpmlint says:
  python-weberror.noarch: W: incoherent-version-in-changelog 0.9-3 ['0.9-2.fc11', 
The last changelog entry is for release 3 but the package is still at release 2.

I note that 0.9.1 is out.  Did you want to update?

Comment 9 Ricky Zhou 2008-12-07 03:28:37 UTC
Strange, I didn't see any weird release when I looked - maybe Luke fixed that.  Here's a new release with the updated version:

Spec URL: http://ricky.fedorapeople.org/pkgs/python-weberror/python-weberror.spec
SRPM URL: http://ricky.fedorapeople.org/pkgs/python-weberror/python-weberror-0.9.1-1.fc10.src.rpm

Comment 10 Jason Tibbitts 2008-12-07 04:38:15 UTC
As for the release weirdness, note that the spec in comment 7 and the spec inside the src.rpm from comment 7 are not identical.

In any case, let me do a full review.

I might suggest that you expand the "WSGI" acronym in the description; I was curious as to what SGI was up to until I looked it up.  I don't think it's a big deal, though.

I do want to make sure that you're aware that this seems to bundle its own copy of at least part of jquery, which might fall afoul of future javascript packaging guidelines.

* source files match upstream.  sha256sum:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint is silent.
* final provides and requires are sane:
   python-weberror = 0.9.1-1.fc11
   python(abi) = 2.6

* %check is present and all tests pass:
   Ran 16 tests in 0.339s

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.


Comment 11 Luke Macken 2009-01-06 22:23:00 UTC
New Package CVS Request
Package Name: python-weberror
Short Description: Web Error handling and exception catching middleware
Owners: lmacken, ricky
Branches: F-10 F-9 EL-5

Comment 12 Kevin Fenzi 2009-01-07 01:23:47 UTC
cvs done.

Comment 13 Jason Tibbitts 2009-02-20 02:34:11 UTC
I note that this package is in rawhide but doesn't seem to have been pushed to any  of the release branches.  I see a build for F-10 but nothing for F-9 (not sure how to check EPEL).

If you really wanted this package in the release branches, please build it for them and issue updates in bodhi.  Otherwise, please go ahead and close this ticket.

Comment 14 Fedora Update System 2009-02-24 05:13:48 UTC
python-weberror-0.10.1-1.fc10 has been submitted as an update for Fedora 10.

Comment 15 Fedora Update System 2009-02-24 05:13:55 UTC
python-weberror-0.10.1-1.fc9 has been submitted as an update for Fedora 9.

Comment 16 Luke Macken 2009-02-24 05:15:19 UTC
Tagged, built, and submitted for F-10 and F-9 updates.  Thanks!

Comment 17 Fedora Update System 2009-02-24 20:49:01 UTC
python-weberror-0.10.1-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2009-02-24 20:56:08 UTC
python-weberror-0.10.1-1.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Ralph Bean 2014-10-13 17:24:40 UTC
Package Change Request
Package Name: python-weberror
New Branches: epel7
Owners: ralph

Comment 20 Kevin Fenzi 2014-10-13 23:00:06 UTC
Git done (by process-git-requests).

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