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=478917 --- Comment #4 from Thomas Sailer <t.sailer@xxxxxxxxxxxxxx> 2009-04-18 13:40:44 EDT --- Fedora review globus-xio-2.7-1.fc10.src.rpm 2008-04-18 (btw: the posted spec file and the spec file contained in the srpm are not consistent) + OK ! needs attention ! rpmlint output (including the license file as doc gets rid of the warning) + Package is named according to Fedora Package Naming Guidelines + Specfile name matches the package base name + Package follows the Fedora packaging guidelines + License meets guidelines and is acceptable to Fedora ASL 2.0 + License matches the actual package license ! The package contains the license file + Spec file is written in American English + Spec file is legible + Upstream sources match sources in the srpm the source repacking makes it impossible to compare md5sum's, thus: $ diff -urN globus_xio-2.7 globus_xio-2.7-dl reports no differences + Package scratch-builds n/a ExcludeArch bugs filed + BuildRequires list all build dependencies n/a %find_lang instead of %{_datadir}/locale/* n/a binary RPM with shared library files must call ldconfig in %post and %postun + Does not use Prefix: /usr + Package owns all directories it creates + No duplicate files in %files + %files has %defattr + %clean contains rm -rf $RPM_BUILD_ROOT + Consistent use of macros + Package must contain code or permissible content + Large documentation files should go in -doc subpackage n/a Files marked %doc should not affect package + Header files should be in -devel n/a Static libraries should be in -static ! Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig' + libfoo.so must go in -devel + -devel must require the fully versioned base + Packages should not contain libtool .la files n/a Packages containing GUI apps must include %{name}.desktop file + Packages must not own files or directories owned by other packages + %install begins with rm -rf $RPM_BUILD_ROOT + Filenames must be valid UTF-8 -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review