Bug 1368157

Summary: Review Request: python-pika-pool - Pools for pikas (rename of python-pika_pool)
Product: [Fedora] Fedora Reporter: Javier Peña <jpena>
Component: Package ReviewAssignee: Zbigniew Jędrzejewski-Szmek <zbyszek>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: karlthered, package-review, projects.rg, zbyszek
Target Milestone: ---Flags: zbyszek: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-02-13 22:22:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Javier Peña 2016-08-18 14:31:55 UTC
This is a package re-review, to rename the current python-pika_pool into python-pika-pool. According to the Fedora Package naming convention (https://fedoraproject.org/wiki/Packaging:Naming?rd=Packaging:NamingGuidelines#Separators), since the upstream PyPi name is pika-pool, the package name should be python-pika-pool·

Spec URL: https://jpena.fedorapeople.org/python-pika-pool/python-pika-pool.spec
SRPM URL: https://jpena.fedorapeople.org/python-pika-pool/python-pika-pool-0.1.3-5.fc26.src.rpm
Description: Pika connection pooling inspired by:

    flask-pika
    sqlalchemy.pool.Pool
Fedora Account System Username: jpena

Koji scratch build available at http://koji.fedoraproject.org/koji/taskinfo?taskID=15295869

Comment 1 Zbigniew Jędrzejewski-Szmek 2016-08-21 21:08:16 UTC
Please change Summary to something meaningful.

Define %global _description \
blah blah ... \
blah blah

and then use it as %description %_description, %description -n python3-pika-pool %_description, %description -n python2-pika-pool %_description.

The version in Obsoletes is supposed to be fixed, e.g. < 0.1.3-5, see https://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages.

Comment 2 Javier Peña 2016-08-22 10:53:17 UTC
Thanks for the review. I have uploaded the updates files to the same location:

Spec URL: https://jpena.fedorapeople.org/python-pika-pool/python-pika-pool.spec
SRPM URL: https://jpena.fedorapeople.org/python-pika-pool/python-pika-pool-0.1.3-5.fc26.src.rpm

Comment 3 Zbigniew Jędrzejewski-Szmek 2016-08-23 01:07:22 UTC
Haïkel?

Comment 4 Raphael Groner 2016-12-10 19:37:48 UTC
Are you still interested in doing this review?

Comment 5 Raphael Groner 2017-02-01 00:29:01 UTC
Second friendly reminder.

Comment 6 Zbigniew Jędrzejewski-Szmek 2017-02-01 17:23:54 UTC
Must have fallen of the radar. I'll take over the review.

+ package name is OK
+ license is acceptable (BSD)
+ license is specified correctly
+ P/R/BR look correct
+ modern python packaging template is used
+ %python_provides are present
+ builds and installs OK
+ fedora-review has no complaints
+ rpmlint only has bogus spelling fixes

Package is APPROVED.

Comment 7 Gwyn Ciesla 2017-02-02 14:37:11 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/python-pika-pool

Comment 8 Raphael Groner 2017-02-02 18:53:16 UTC
Please build for Fedora 25 at least, too.

Comment 9 Fedora Update System 2017-02-03 10:25:12 UTC
python-pika-pool-0.1.3-6.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-258d703865

Comment 10 Fedora Update System 2017-02-03 23:50:30 UTC
python-pika-pool-0.1.3-6.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-258d703865

Comment 11 Fedora Update System 2017-02-13 22:22:06 UTC
python-pika-pool-0.1.3-6.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2017-02-14 15:48:47 UTC
python-pika-pool-0.1.3-6.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.