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 <mcla...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-review+

--- Comment #10 from Matthias Clasen <mcla...@redhat.com> 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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to