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=668588
--- Comment #3 from Jason Tibbitts tibbs@math.uh.edu 2011-01-21 18:14:53 EST --- A couple of other things you might have missed:
Need to run rpmlint on the built rpms, too: python26-imaging.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/PIL/_imagingmath.so _imagingmath.so()(64bit) python26-imaging.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/PIL/_imagingcms.so _imagingcms.so()(64bit) python26-imaging.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/PIL/_imagingft.so _imagingft.so()(64bit) python26-imaging.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/PIL/_imaging.so _imaging.so()(64bit) python26-imaging-sane.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/_sane.so _sane.so()(64bit) python26-imaging-tk.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/PIL/_imagingtk.so _imagingtk.so()(64bit) python26-imaging-tk.x86_64: W: no-documentation
Not that you can use the filtering functionality on ancient releases like EL5 anyway, so these aren't really problematic.
EPEL isn't built for s390, so all of the s390 conditional stuff is mostly pointless. (Though I could see value in trying to keep the spec as close as possible to the Fedora one.)
Otherwise this pre-review gets my ACK.