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
Some minor changes needed: - Summary is too long and the final dot has to be removed. - Description needs to be wrapped Otherwise, LGTM.
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
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.
Created attachment 1537442 [details] fedora-review output
https://pagure.io/releng/fedora-scm-requests/issue/9994
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-yappi