Bug 207802
Summary: | Review Request: libpaper - Library and tools for handling papersize | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tom "spot" Callaway <tcallawa> |
Component: | Package Review | Assignee: | Dan Horák <dan> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | pertusus |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-09-23 16:38:33 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779, 207761 |
Description
Tom "spot" Callaway
2006-09-23 13:53:04 UTC
rpmlint from Rawhide gives W: libpaper mixed-use-of-spaces-and-tabs (on the src.rpm) Could the static library be excluded? When running paperconf, it could not find /etc/papersize. /etc/papersize is (re)generated with /usr/sbin/paperconfig, but the package should own it. I set letter as the default. New SRPM: http://www.auroralinux.org/people/spot/review/libpaper-1.1.20-2.fc6.src.rpm New SPEC: http://www.auroralinux.org/people/spot/review/libpaper.spec Review: - no rpmlint output on any package - package name OK - spec file name OK, is in English and is legible - package meets the Packaging Guidelines - license OK (GPL) and is included - source matches upstream 7075f580606a84e63b7d6d9fa3124c31 libpaper_1.1.20.tar.gz 7075f580606a84e63b7d6d9fa3124c31 libpaper_1.1.20.tar.gz.1 - compiles and builds at least on i386 (devel) - BuildRequires are correct - contains no localized files - has shared lib with appropriate ldconfig calls - does not create any directory - no duplicates files, permissions are set properly, uses %defattr - has %clean section - consistent use of macros - contains code - no large docs, %doc is not required during runtime - devel subpackage required and present, contains header - no .la libtool archives - not a GUI application - it works APPROVED The static library can be taken out also with %configure --disable-static. Thanks for the fast review, built in FC-5 and devel. I have spotted some more or less minor issues: * the man pages for functions should be in the devel package %{_mandir}/man3/* * removing unneded files rm -rf $RPM_BUILD_ROOT%{_libdir}/*.la rm -rf $RPM_BUILD_ROOT%{_libdir}/*.a should be without -r and even maybe without f * I think that there is no need to set letter as a default, it is allready the default according to the man paperconf. In my opinion, what should be better is an empty file or a file with a comment along echo '# Simply write the paper name. See papersize(5) for possible values' > $RPM_BUILD_ROOT%{_sysconfdir}/papersize * in man paperconfig there is a reference to /etc/libpaper.d * The file NEWS in debian/ should certainly be in %doc Yes, I really missed the man3 directory. The rest is minor, but reasonable. Also in debian/po subdirectory there are hidden localized files. It would be nice to have them installed. Thanks for the feedback. -3 should resolve all the above issues. |