Bug 811689 - Review Request: python-tw2-forms - Forms for ToscaWidgets2
Summary: Review Request: python-tw2-forms - Forms for ToscaWidgets2
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 720818
Blocks: 811732 811769 818293
TreeView+ depends on / blocked
 
Reported: 2012-04-11 18:21 UTC by Ralph Bean
Modified: 2014-11-12 22:07 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-22 15:34:35 UTC
Type: ---
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Ralph Bean 2012-04-11 18:21:29 UTC
Spec URL: http://threebean.org/rpm/python-tw2-forms.spec
SRPM URL: http://threebean.org/rpm/python-tw2-forms-2.0.2-1.fc17.src.rpm
Description: 
ToscaWidgets is a web widget toolkit for Python to aid in the creation,
packaging and distribution of common view elements normally used in the web.

tw2.forms contains the basic form widgets.

This package requires python-tw2-core which is currently in process at https://bugzilla.redhat.com/show_bug.cgi?id=720818

Comment 1 Ralph Bean 2012-04-30 21:55:33 UTC
Updated to make some changes I learned from my last package review.

Spec URL: http://threebean.org/rpm/python-tw2-forms.spec
SRPM URL: http://threebean.org/rpm/python-tw2-forms-2.0.2-2.fc17.src.rpm

koji scratch builds fail only because python-tw2-core is still in updates-testing.  If you add updates-testing to your mock config, fedora-review should be able to run just fine.

koji - f17 - http://koji.fedoraproject.org/koji/taskinfo?taskID=4037157
koji - el6 - http://koji.fedoraproject.org/koji/taskinfo?taskID=4037175

Comment 2 Tom "spot" Callaway 2012-05-02 18:23:24 UTC
== REVIEW ==

Good:

- rpmlint checks return nothing
- package meets naming guidelines
- package meets packaging guidelines
- license (MIT) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 
- %check okay

Nice clean simple little Python package. APPROVED.

Comment 3 Ralph Bean 2012-05-02 18:42:35 UTC
New Package SCM Request
=======================
Package Name: python-tw2-forms
Short Description: Forms for ToscaWidgets2
Owners: ralph
Branches: f17 el6
InitialCC:

Comment 4 Gwyn Ciesla 2012-05-03 13:12:23 UTC
Git done (by process-git-requests).

Comment 5 Fedora Update System 2012-05-03 14:47:31 UTC
python-tw2-forms-2.0.2-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/python-tw2-forms-2.0.2-2.fc17

Comment 6 Fedora Update System 2012-05-03 17:12:28 UTC
python-tw2-forms-2.0.2-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/python-tw2-forms-2.0.2-2.el6

Comment 7 Fedora Update System 2012-05-04 03:11:12 UTC
python-tw2-forms-2.0.2-2.fc17 has been pushed to the Fedora 17 testing repository.

Comment 8 Fedora Update System 2012-05-22 15:34:35 UTC
python-tw2-forms-2.0.2-2.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 9 Fedora Update System 2012-05-26 07:39:54 UTC
python-tw2-forms-2.0.2-2.fc17 has been pushed to the Fedora 17 stable repository.

Comment 10 Ralph Bean 2014-11-12 20:25:19 UTC
Package Change Request
======================
Package Name: python-tw2-forms
New Branches: epel7
Owners: ralph
InitialCC:

Comment 11 Gwyn Ciesla 2014-11-12 22:07:27 UTC
Git done (by process-git-requests).


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