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

Randy Barlow <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #1 from Randy Barlow <[email protected]> ---
Hello Dennis!

My review here is informal (as I am also currently seeking to become a package
maintainer), but here are a few notes:

0) You've put a BuildRequires on python3-setuptools in the section that deals
with Python 2 (probably a typo).

1) Python 3 systems will get Requires ImageMagick twice - I think the second
Requires line is redundant.

rpmlint only complains about a few spellings, but I think they are all OK so I
wouldn't worry about them. Nice work!

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to