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=668542 Jochen Schmitt <jochen@xxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |jochen@xxxxxxxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |jochen@xxxxxxxxxxxxxxx Flag| |fedora-review? --- Comment #1 from Jochen Schmitt <jochen@xxxxxxxxxxxxxxx> 2011-01-11 13:11:45 EST --- Good: + Base name of the SPEC files matches to package name. + Package fullfill naming guidelines + Package is build for noarch + Package cleans the buildroot in the %clean and %install section + Package contains no subpackages Bad: - This is not the most current release. On upstream I have found a note about the release of 1.27c - I would make the removement of the CRs in the example file into a loop. - Why to you refer to %{_datadir}/doc/%{name} directory in the %doc stanza - Please create a separate font package, because yoyur package contains fonts - Package contains no verbatin copy of the license text. Please contact upstream for including on to the upstream distributation. -- 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