https://bugzilla.redhat.com/show_bug.cgi?id=1275517 Randy Barlow <rbarlow@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rbarlow@xxxxxxxxxx --- Comment #1 from Randy Barlow <rbarlow@xxxxxxxxxx> --- 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review