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=477190 --- Comment #3 from Tom "spot" Callaway <tcallawa@xxxxxxxxxx> 2008-12-19 11:51:54 EDT --- Looking at the code, it looks like it is GPLv3+. Why are the man pages separate from the upstream source? Seems like they'd be a logical fit to go inside the tarball, or at the very least, uploaded to the fedorahosted site. If you opt for the latter route, please provide full upstream URLs. Do you still need Source1? It doesn't seem to be used anymore. You also don't need the Requires: python >= 2.4 Fedora's RPM will detect the python bits in the package and add a proper versioned Requires on python(abi). For example, in rawhide, we get: [spot@velociraptor ~]$ rpm -qp /home/spot/rpmbuild/RPMS/noarch/cas-0.13-113.fc11.noarch.rpm --requires /usr/bin/python config(cas) = 0.13-113.fc11 crash python >= 2.4 python(abi) = 2.6 rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PartialHardlinkSets) <= 4.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 Thus, your manual python Requires is unnecessary. Also, the last sentence of the description doesn't make much sense. Can you fix that up a bit? :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review