Bug 445024 - Review Request: python-webhelpers - Helper routines for writing web applications
Summary: Review Request: python-webhelpers - Helper routines for writing web applications
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Brian Pepple
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 445021
Blocks: 445028
TreeView+ depends on / blocked
 
Reported: 2008-05-02 19:16 UTC by Kyle VanderBeek
Modified: 2015-01-14 11:54 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-05-31 21:36:48 UTC
Type: ---
Embargoed:
bdpepple: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Kyle VanderBeek 2008-05-02 19:16:16 UTC
Spec URL: http://www.kylev.com/pylons-rpm/python-webhelpers.spec
SRPM URL: http://www.kylev.com/pylons-rpm/python-webhelpers-0.3.4-1.fc8.src.rpm
Description: Webhelpers is a library of useful and common functions used in generating web pages including HTML generation, escaping, and URL construction.  It is a prerequisite of Pylons.

Comment 1 Felix Schwarz 2008-05-10 21:23:37 UTC
$ rpmlint python-mako-0.1.10-1.fc8.src.rpm 
python-mako.src: W: summary-ended-with-dot Mako template library for Python.

Comment 2 Kyle VanderBeek 2008-05-12 00:23:01 UTC
Good to know I'm not the only guy who makes cut 'n paste errors.  You had me
confused for a moment (this is webhelpers, not mako).  But, the dot problem
existed here to so I fixed it and also added docs, plus fixed a warning about a
spurious executable bit on a doc file:

Spec URL: http://www.kylev.com/pylons-rpm/python-webhelpers.spec
SRPM URL: http://www.kylev.com/pylons-rpm/python-webhelpers-0.3.4-2.fc8.src.rpm

Comment 3 Brian Pepple 2008-05-26 22:54:53 UTC
MD5Sum:
16aacff76caf2e3c2284f1915e2fc57f  WebHelpers-0.3.4.tar.gz

Good:
* Source URL is canonical
* Upstream source tarball verified
* Package name conforms to the Fedora Naming Guidelines
* Group Tag is from the official list
* Valid license tag
* Buildroot has all required elements
* All paths begin with macros
* All necessary BuildRequires listed.
* Files have appropriate permissions and owners (except as noted below).
* Package installs and uninstalls cleanly

Bad:
* rpmlint produces the following errors that should be addressed before
importing into CVS:

python-webhelpers.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/webhelpers/textile.py 0644
python-webhelpers.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/webhelpers/markdown.py 0644

+1 APPROVED.

Comment 4 Kyle VanderBeek 2008-05-28 00:39:46 UTC
New Package CVS Request
=======================
Package Name: python-webhelpers
Short Description: Web helper routines for python, used in Pylons
Owners: kylev
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes

Comment 5 Kevin Fenzi 2008-05-30 19:45:28 UTC
cvs done.

Comment 6 Fedora Update System 2008-05-31 21:44:07 UTC
python-webhelpers-0.3.4-2.fc8 has been submitted as an update for Fedora 8

Comment 7 Fedora Update System 2008-05-31 21:44:42 UTC
python-webhelpers-0.3.4-2.fc9 has been submitted as an update for Fedora 9

Comment 8 Fedora Update System 2008-06-13 02:20:57 UTC
python-webhelpers-0.3.4-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2008-06-13 02:25:12 UTC
python-webhelpers-0.3.4-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 10 Fedora Update System 2008-07-26 06:04:21 UTC
python-webhelpers-0.3.4-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Kim 2015-01-13 22:21:37 UTC
Package Change Request
======================
Package Name: python-webhelpers 
New Branches: epel7
Owners: kylev kim27
InitialCC: 

Please see #1163577 for request for epel 7 status.

Comment 12 Gwyn Ciesla 2015-01-14 11:54:12 UTC
Git done (by process-git-requests).


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