https://bugzilla.redhat.com/show_bug.cgi?id=1344276



--- Comment #28 from Ken Dreyer <ktdre...@ktdreyer.com> ---
Please accept my apologies for the delay and lack of communication! That is
bad.

I can continue the review or let someone else take it.

Looks like the LICENSE file should be listed as %license under %files.

Would you please use the newer %py2_build and %py2_install macros?
https://fedoraproject.org/wiki/Packaging:Python

Can this package work with Python 3 instead of Python 2? Maybe it would be a
good idea to add a comment to the .spec file explaining why this will not work
with Python 3 if that is the case.
https://fedoraproject.org/wiki/FinalizingFedoraSwitchtoPython3 is coming
eventually. You could conditionalize py2/py3 with "%if %{?rhel} < 8" if you
want to share the same .spec file across RHEL 7 and Fedora.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to