https://bugzilla.redhat.com/show_bug.cgi?id=1460189
--- Comment #2 from Kalev Lember <klem...@redhat.com> ---
Fedora review template-glib-3.25.2-1.fc27.src.rpm 2017-06-09
$ rpmlint template-glib-3.25.2-1.fc27.src.rpm \
template-glib \
template-glib-devel \
template-glib-debuginfo
template-glib.src: W: spelling-error Summary(en_US) templating ->
contemplating, template, tempting
template-glib.src: W: spelling-error %description -l en_US templating ->
contemplating, template, tempting
template-glib.src: E: description-line-too-long C Template-GLib is a templating
library for GLib. It includes a simple template format
template-glib.src: E: description-line-too-long C It separates the parsing of
templates and the expansion of templates for faster expansion.
template-glib.src: E: description-line-too-long C You can also define scope,
custom functions, and more with the embedded expression language.
template-glib.x86_64: W: spelling-error Summary(en_US) templating ->
contemplating, template, tempting
template-glib.x86_64: W: spelling-error %description -l en_US templating ->
contemplating, template, tempting
template-glib.x86_64: E: description-line-too-long C Template-GLib is a
templating library for GLib. It includes a simple template format
template-glib.x86_64: E: description-line-too-long C It separates the parsing
of templates and the expansion of templates for faster expansion.
template-glib.x86_64: E: description-line-too-long C You can also define scope,
custom functions, and more with the embedded expression language.
template-glib.x86_64: W: shared-lib-calls-exit
/usr/lib64/libtemplate_glib-1.0.so.0.2502.0 exit@GLIBC_2.2.5
template-glib-devel.x86_64: W: only-non-binary-in-usr-lib
template-glib-devel.x86_64: E: wrong-script-interpreter
/usr/share/doc/template-glib-devel/examples/simple.py /usr/bin/env python3
template-glib-devel.x86_64: W: doc-file-dependency
/usr/share/doc/template-glib-devel/examples/simple.py /usr/bin/env
4 packages and 0 specfiles checked; 7 errors, 7 warnings.
+ OK
! needs attention
! rpmlint warns about too long description lines. Might be nice to fix that up;
the rest seems like noise and should be fine to ignore.
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the base package name.
+ The package meets the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The license text (COPYING) is included in %license
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match the sources in the srpm
aca745c779f80a1dc9691181768aa4f77b9374646f4b37743c1c9622d266ae09a169edcf757864c435ab428319195d30e033292b81c664e04c767e5e3fe37729
template-glib-3.25.2.tar.xz
aca745c779f80a1dc9691181768aa4f77b9374646f4b37743c1c9622d266ae09a169edcf757864c435ab428319195d30e033292b81c664e04c767e5e3fe37729
Download/template-glib-3.25.2.tar.xz
+ The package builds in koji
(https://koji.fedoraproject.org/koji/taskinfo?taskID=19933441)
n/a ExcludeArch bugs filed
+ BuildRequires look sane
+ locale handling
+ ldconfig in %post and %postun
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
! Several directories created by the package are unowned -- I'll list them at
the end
+ No duplicate files in %files
+ Permissions are properly set
+ Consistent use of macros
+ The package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect the runtime of application
n/a Static libraries should be in -static
+ Development files should be in -devel
+ -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Proper .desktop file handling
+ Doesn't own files or directories already owned by other packages
+ Filenames are valid UTF-8
Just two minor issues I discovered when going through the checklist above:
1) rpmlint warns about too long description lines
2) a number of directories created by the package aren't listed in %files.
should co-own those so that they correctly get removed when uninstalling the
package:
%{_libdir}/girepository-1.0
%{_datadir}/gir-1.0
%{_datadir}/gtk-doc
%{_datadir}/gtk-doc/html
%{_datadir}/vala
Beyond that looks fine to me!
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
