[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-10-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||python26-sqlalchemy-0.6.4-1
   ||.el5
 Resolution||ERRATA
Last Closed||2010-10-25 12:37:06

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-10-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2010-10-05 
13:26:41 EDT ---
python26-sqlalchemy-0.6.4-1.el5 has been pushed to the Fedora EPEL 5 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python26-sqlalchemy'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/python26-sqlalchemy-0.6.4-1.el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-10-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-10-04 
13:06:20 EDT ---
python26-sqlalchemy-0.6.4-1.el5 has been submitted as an update for Fedora EPEL
5.
https://admin.fedoraproject.org/updates/python26-sqlalchemy-0.6.4-1.el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-10-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

--- Comment #5 from Kevin Fenzi ke...@tummy.com 2010-09-29 14:45:09 EDT ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||steve.tray...@cern.ch
 AssignedTo|nob...@fedoraproject.org|steve.tray...@cern.ch
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

--- Comment #1 from Steve Traylen steve.tray...@cern.ch 2010-09-28 14:50:55 
EDT ---

Review of python26-sqlalchemy.

PASS: specfiles match:
In fact no but python-sqlalchemy has set the precedent.

PASS: source files match upstream:
$ curl -s 
http://pypi.python.org/packages/source/S/SQLAlchemy/SQLAlchemy-0.5.8.tar.gz |
md5sum -
11292211f2634151d240025d58791210  -
[st...@bottom SPECS]$ md5sum ../SOURCES/SQLAlchemy-0.5.8.tar.gz 
11292211f2634151d240025d58791210  ../SOURCES/SQLAlchemy-0.5.8.tar.gz

PASS: package meets naming and versioning guidelines.
PASS: spec is properly named, cleanly written, and uses macros consistently.
PASS: dist tag is present.
PASS: build root is correct.
PASS: license field matches the actual license.
MIT. Very clearly

PASS: license is open source-compatible.
PASS: license text included in package.
FAIL: latest version is being packaged.
PASS: BuildRequires are proper.
PASS:  compiler flags are appropriate.
PASS:  %clean is present. 
PASS: package builds in mock.
 epel5.
PASS: package installs properly.
IGNORE: debuginfo package looks complete.
FAIL: rpmlint is silent.
python26-sqlalchemy.noarch: W: incoherent-version-in-changelog 0.5.84-
['0.5.8-4.el5', '0.5.8-4']

FAIL: final provides and requires are sane
$ rpm -qp --provides python26-sqlalchemy-0.5.8-4.el5.noarch.rpm 
python26-sqlalchemy = 0.5.8-4.el5
$ rpm -qp --requires python26-sqlalchemy-0.5.8-4.el5.noarch.rpm 
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1

In particular python(abi) = 2.6 is not present.

PASS: %check is present and all tests pass:
Hopefully we can get the python-nose tests in before release.
Probably worth waiting if possible.

PASS: no shared libraries are added to the regular linker search paths.
PASS: owns the directories it creates. 
PASS: doesn't own any directories it shouldn't.
PASS: no duplicates in %files.
PASS:  file permissions are appropriate.
PASS: scriptlets match those on ScriptletSnippets page.
PASS:  code, not content.
PASS:  documentation is small, so no -docs subpackage is necessary.
PASS: %docs are not necessary for the proper functioning of the package.
IGNORE: no headers.
IGNORE: no pkgconfig files.
NONE:  no libtool .la droppings.
IGNORE: desktop files valid and installed properly.

FAILS:
1) 0.6.4 has been released recently is this a better choice.
2) The release number is missing from the changelog comment.
3) I think a python(abi) = 2.6 should be in the requires. 


COMMENTS:
CFLAGS=$RPM_OPT_FLAGS is irrelevant on a noarch package and should
be removed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

--- Comment #2 from BJ Dierkes wdier...@rackspace.com 2010-09-28 15:41:45 EDT 
---
Thank you.  I've made the requested changes:

SPEC: http://5dollarwhitebox.org/tmp/python26-sqlalchemy.spec
SRPMS: http://5dollarwhitebox.org/tmp/python26-sqlalchemy-0.6.4-1.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Steve Traylen steve.tray...@cern.ch 2010-09-28 15:52:16 
EDT ---
All seems pretty perfect to me.
Approved.

Steve.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

BJ Dierkes wdier...@rackspace.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 606557] Review Request: python26-sqlalchemy - Modular and flexible ORM library for python26

2010-09-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=606557

--- Comment #4 from BJ Dierkes wdier...@rackspace.com 2010-09-28 16:30:51 EDT 
---
New Package SCM Request
===
Package Name: python26-sqlalchemy
Short Description: Modular and flexible ORM library for python26 
Owners: derks
Branches: el5
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review