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=733925 --- Comment #2 from Michael Schwendt <mschwendt@xxxxxxxxx> 2011-09-03 04:57:35 EDT --- Please at least skim over the following pages, especially with regard to search terms relevant to your package. For example, you could have searched for a section covering static and shared libraries: https://fedoraproject.org/wiki/Packaging:ReviewGuidelines https://fedoraproject.org/wiki/Packaging:Guidelines https://fedoraproject.org/wiki/Packaging:NamingGuidelines http://fedoraproject.org/wiki/Licensing#SoftwareLicenses * MUST: rpmlint must be run on the source rpm and all binary rpms the build produces. The output should be posted in the review. Which does not imply that only a reviewer should do this. It's a tool for packagers. > Libdatrie-0.2.4-1.src.rpm rpmlint says something about this. > Name: libdatrie > Group: Development/Libraries Base library packages typically go into group "System Environment/Libraries". https://fedoraproject.org/wiki/Packaging:DistTag > Release: 1 https://fedoraproject.org/wiki/Packaging/NamingGuidelines https://fedoraproject.org/wiki/Packaging:DistTag If you don't like the dist tag, a brief rationale would be appreciated. > Summary: An Implementation of Double-Array Trie In the installer and package tools, leading "An", "A", "The" merely decrease readability. Many book/report titles also omit them to be even more concise. Summary: Implementation of Double-Array Trie > License: LGPLv2.1+ rpmlint says something about this. > BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag > %install > rm -rf %{buildroot} > %clean > rm -rf %{buildroot} https://fedoraproject.org/wiki/Packaging:Guidelines#.25clean > %package devel > Summary: Development files for libdatrie Notice you don't have a leading article here either. ;) > Requires: libdatrie https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package > %defattr(-,root,root,-) Either at the beginning of all sections or omitted because of: http://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions > %{_libdir}/libdatrie.so.1* https://fedoraproject.org/wiki/Packaging:Guidelines#Shared_Libraries > %{_libdir}/libdatrie.a > %{_libdir}/libdatrie.la https://fedoraproject.org/wiki/Packaging:Guidelines#Packaging_Static_Libraries > %package doc > Requires: libdatrie Clearly the documentation subpackage does not depend on the library. See also: https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires > Group: Development/Libraries https://fedoraproject.org/wiki/Packaging:Guidelines#Documentation > %files doc > %{_datadir}/doc/datrie/ > %{_datadir}/doc/libdatrie/ Any particular reason why the documentation is not put into %{_defaultdocdir}/%{name}-%{version} - but split up into two directories plus the base library package adding another doc directory? > %doc AUTHORS COPYING README What about file NEWS? > %{_datadir}/man/man1/trietool-0.2.1.gz There is %{_mandir} = %{_datadir}/man -- 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