Bug 1679634

Summary: Review Request: python-yappi - Yet Another Python Profiler
Product: [Fedora] Fedora Reporter: Alfredo Moralejo <amoralej>
Component: Package ReviewAssignee: Javier Peña <jpena>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: hguemar, jpena, mail, package-review
Target Milestone: ---Flags: jpena: 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: 2020-05-19 10:32:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
fedora-review output none

Description Alfredo Moralejo 2019-02-21 15:55:43 UTC
Spec URL: https://github.com/amoralej/python-yappi/raw/master/python-yappi.spec
SRPM URL: https://github.com/amoralej/python-yappi/raw/master/python-yappi-1.0-1.fc30.src.rpm

Description:

yappi has been added as new dependency in OpenStack and it's needed for the last release of python-oslo-service

yappi is a new python profiler which includes support for multithreading and cpu-times, as compared with other existing profiles in standard distribution.

project: https://github.com/sumerc/yappi

Fedora Account System Username: amoralej

scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=32946184

Comment 1 hguemar 2019-02-21 16:57:12 UTC
Some minor changes needed:
- Summary is too long and the final dot has to be removed.
- Description needs to be wrapped

Otherwise, LGTM.

Comment 2 Alfredo Moralejo 2019-02-22 11:19:57 UTC
I've just updated the spec to fix the mentioned issues:

Spec URL: https://github.com/amoralej/python-yappi/raw/master/python-yappi.spec
SRPM URL: https://github.com/amoralej/python-yappi/raw/master/python-yappi-1.0-1.fc30.src.rpm

Comment 3 Javier Peña 2019-02-22 12:02:18 UTC
Review notes (fedora-review.txt will be added as an attachment due to the size limit):

- gcc is required for the build, regardless of the fedora-review output
- The comment about header files in the -debugsource subpackage is expected
- The comment about the /usr/lib/.build-id directory is expected, and shared with lots of other Fedora packages.

The package is APPROVED.

Comment 4 Javier Peña 2019-02-22 12:03:01 UTC
Created attachment 1537442 [details]
fedora-review output

Comment 5 Alfredo Moralejo 2019-02-25 09:55:47 UTC
https://pagure.io/releng/fedora-scm-requests/issue/9994

Comment 6 Gwyn Ciesla 2019-02-25 14:51:22 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/python-yappi