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=620556 Matthias Clasen <mclasen@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+ --- Comment #10 from Matthias Clasen <mclasen@xxxxxxxxxx> 2010-08-18 08:41:37 EDT --- Package name: ok Spec file name: ok Packaging guidelines: ok License: ok License field: ok License file: ok Spec file language: ok Spec file readable: ok Upstream sources: ok Buildable: ok ExcludeArch: ok BuildRequires: ok Locale handling: ok ldconfig: ok system libraries: ok relocatable: ok directory ownership: ok, could make it slightly more explicit by replacing %{_includedir}/* with %{_includedir}/folks duplicate files: ok file permissions: ok macro use: ok permissible content: ok large docs: ok %doc content: ok header files: ok static libraries: ok shared libraries: ok -devel deps: ok libtool archives: ok gui applications: ok file ownership: ok utf8 filenames: ok 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review