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

Reply via email to