https://bugzilla.redhat.com/show_bug.cgi?id=1404305 Rafael Fonseca <rdossant@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(rdossant@redhat.c | |om) | --- Comment #4 from Rafael Fonseca <rdossant@xxxxxxxxxx> --- (In reply to Randy Barlow from comment #3) > It installs now! I have one thing you need to fix before I can approve, and > a few suggestions that are at your option: > > Must fix > ======== > > * The Python package must be named python3-bcc instead of bcc-python3, > as per the Python packaging guidelines: > > https://fedoraproject.org/wiki/Packaging:Python#Provides Fixed. > Optional suggestions > ==================== > > * fedora-review doesn't think you need to BuildRequire gcc or gcc-c++. Fixed. > * It would be good to include a link to an upstream pull request, or a > comment > explaining the purpose of the patch and why it isn't submitted upstream > in the spec file. Fixed. Just to summarize, the problem comes from using %cmake which makes building shared libs the default instead of static. > * The Python package should Require: bcc with the %{?_isa} macro. Fixed. > * There does appear to be a test suite upstream. It would be good to add > a %check section that runs it. Won't fix. BCC test suit is complicated to run on a builder. If you look at the tests, they require root privileges and they make changes at the machine they are running on (like changing IP address). > * Consider making bcc-doc and gcc-tools noarch packages. python3-gcc > should be a noarch package as well since it installs to /usr/lib/. Fixed. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx