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=645761 --- Comment #3 from Domingo Becker <domingobecker@xxxxxxxxx> 2010-11-04 09:10:58 EDT --- (In reply to comment #1) > Most of my comments in bug 645764 also apply here: > > * Full URL is needed on Source Fixed. > * License should be BSD Fixed. > * License Text should be included in the package Fixed. > * Summary should be shortened (A simple yet... is enough) Please, check it. > * There are couple more files for the %doc section (Changelog.txt and > contributors.txt) Fixed. > * Version advised in changlog does not match package Changelog seems to be outdated. I will communicate it upstream. > * Did not check a mock build, but chances are Django is needed as BR: Added as BuildRequires. Same spec and srpm urls as in first comment. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review