Bug 468224

Summary: Review Request: python-zope-sqlalchemy - Minimal Zope/SQLAlchemy transaction integration
Product: [Fedora] Fedora Reporter: Luke Macken <lmacken>
Component: Package ReviewAssignee: Brian Pepple <bdpepple>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, pfrields
Target Milestone: ---Flags: bdpepple: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-01-06 22:31:18 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 468230    

Description Luke Macken 2008-10-23 16:33:49 UTC
Spec URL: http://lmacken.fedorapeople.org/rpms/python-zope-sqlalchemy.spec
SRPM URL: http://lmacken.fedorapeople.org/rpms/python-zope-sqlalchemy-0.3-1.fc9.src.rpm
Description:
The aim of this package is to unify the plethora of existing packages
integrating SQLAlchemy with Zope's transaction management. As such it seeks
only to provide a data manager and makes no attempt to define a zopeish way to
configure engines.

Comment 1 Brian Pepple 2008-11-16 23:33:00 UTC
MD5Sum:
3edc2b0f02e778483de64b60ff214df9  zope.sqlalchemy-0.3.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
* Buildroot has all required elements
* All paths begin with macros
* All necessary BuildRequires listed.
* Package builds in Mock.
* Package installs and uninstalls cleanly

Bad:
* rpmlint produces the following, which based on the source files should be 'ZPLv2.1':
python-zope-sqlalchemy.noarch: W: invalid-license ZPL

the license tag can be fixed when you import it into CVS.

+1 APPROVED

Comment 2 Luke Macken 2008-11-19 15:39:15 UTC
New Package CVS Request
=======================
Package Name: python-zope-sqlalchemy
Short Description: Minimal Zope/SQLAlchemy transaction integration
Owners: lmacken
Branches: F-9 F-10 EL-5

Comment 3 Kevin Fenzi 2008-11-23 03:58:37 UTC
cvs done.

Comment 4 Luke Macken 2009-01-06 22:31:18 UTC
Tagged and built, and queued up in bodhi.  Thanks!

Comment 5 Fedora Update System 2009-01-06 22:33:07 UTC
python-zope-sqlalchemy-0.3-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/python-zope-sqlalchemy-0.3-1.fc10

Comment 6 Fedora Update System 2009-01-06 22:33:29 UTC
python-zope-sqlalchemy-0.3-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/python-zope-sqlalchemy-0.3-1.fc9