Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: typespeed - Test your typing speed and get your fingers' CPS https://bugzilla.redhat.com/show_bug.cgi?id=447639 fedora@xxxxxxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ ------- Additional Comments From fedora@xxxxxxxxxxxxxxxxxxxx 2008-07-14 16:54 EST ------- The package 9ad47f3cc9af37116e7c4c47e54e55e1 typespeed-0.6.4-2.fc9.src.rpm fixes all blockers: OK - MUST: %files section fixed OK - MUST: desktop file is valid OK - SHOULD: ChangeLog is UTF-8 OK - SHOULD: Description was reworked Two more suggestions: - Please remove INSTALL from %doc. It's generic and not needed when installing from rpm - Consider adding an icon to the desktop file. I suggest ether "keyboard" or simply "terminal" since this is a console application. Anyway: None of these is a blocker, so this package is APPROVED. -- 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, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review