Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751925
--- Comment #4 from Thibault North thibault.north@gmail.com 2011-11-10 21:04:10 EST --- Hello Brendan, Thanks for your review.
The %check seems to take an awfully long time, and there are a number of test directories installed.
I'm just wondering if they're really necessary.
I don't really know. For python-numexpr, I was suggested to add these kind of checks. I guess it will help the tracking of some bugs if tests fail at a rebuild.
You could consider a separate doc package also for the examples and documentation, but that's just a suggestion. Full review below, I'll wait for your reply before passing.
Done.
[!] Be licensed with a Fedora approved license and meet the Licensing Guidelines *** You just need to specify the lrucache.py is licensed under AFL
License is set to BSD and AFL: is that good enough?
[?] Has BuildRequires: python-setuptools-devel
I'm really not sure if this is still required?
I don't know. Other python- packages I have seen don't have it.
[!] All build dependencies must be listed in BuildRequires *** perhaps python2-devel rather than python-devel, although this is rather cosmetic
Fixed. Thanks again, Thibault