https://bugzilla.redhat.com/show_bug.cgi?id=1315020 Jonathan Underwood <jonathan.underwood@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #10 from Jonathan Underwood <jonathan.underwood@xxxxxxxxx> --- Ok, this is looking great. I'm going to set it as passed review, but I'd urge you to consider adding more tests during %check to check you can import other modules than just the fileio modules. You can do simple checks just to see if the module successfully imports. Eg. (taken from python-lz4.spec): PYTHONPATH=$RPM_BUILD_ROOT%{python2_sitearch} %{__python2} -c "import lz4" These sorts of simple tests do actually catch breakage sometimes, so do have use. Thanks for your contribution to Fedora, and thanks for being so quick to respond to the review. APPROVED -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review