[Bug 449504] Review Request: ranpwd - A program to generate random passwords
bugzilla at redhat.com
bugzilla at redhat.com
Sun Jun 22 19:38:11 UTC 2008
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: ranpwd - A program to generate random passwords
https://bugzilla.redhat.com/show_bug.cgi?id=449504
------- Additional Comments From terjeros at phys.ntnu.no 2008-06-22 15:38 EST -------
[ x=ok -=dont't apply !=please fix ?=may fix ]
MUST
[x] rpmlint must be run clean on every package
[x] package must be named according to the Package Naming Guidelines
[x] spec file name must match the base package %{name}
[-] package must meet the Packaging Guidelines
[x] [GPLv2+] package must be licensed with a Fedora approved license
[x] license field in the package spec file must match the actual license
[-] includes the text of the license(s) in its own file: include in %doc
[x] be written in American English
[x] spec file for the package be legible
[!] sources used to build the package must match the upstream source
md5sum: ce5bb201130aad92f0f9a90fc8be624f: ok
however fix date by using e.g. wget -N
[x] compile and build into binary rpms on at least one architecture
http://koji.fedoraproject.org/koji/taskinfo?taskID=675038
[-] not successfully compil an architecture: use ExcludeArch
[x] all build dependencies must be listed in BuildRequires
[x] spec file MUST handle locales properly
[-] shared library files not in any default linker paths: ldconfig
[-] relocatable package: the packager must state this fact
[-] package must own all directories that it creates
[x] not any duplicate files in the %files listing
[x] permissions on files must be set properly
[x] package must have a %clean section, which contains rm -rf %{buildroot}
[x] consistently use macros
[x] must contain code, or permissable content
[-] large docs should go in a -doc subpackage
[-] %doc must not affect the runtime of the application
[-] header files must be in a -devel package
[-] static libraries must be in a -static package
[-] containing pkgconfig(.pc) files must 'Requires: pkgconfig'
[-] library files that end in .so: go in a -devel package
[-] devel pkg: require base package using a fully versioned dependency
[-] no .la libtool archives
[-] gui app include a %{name}.desktop file
[+] must not own files or directories already owned by other packages
[+] %install includes rm -rf %{buildroot}
[+] filenames in rpm packages must be valid UTF-8
SHOULD
[?] nice you if ping upstream about missing license text
[+] translations if description and summary sections
[+] test that the package builds in mock
http://koji.fedoraproject.org/koji/taskinfo?taskID=675038
[+] compile and build into binary rpms on all archs
http://koji.fedoraproject.org/koji/taskinfo?taskID=675038
[+] package functions as described
[-] those scriptlets are sane
[-] subpackages require the base packag fully versioned dep
[-] pkgconfig(.pc) in devel
[-] no explicit file dep outside /etc, /bin/, /sbin, /usr/{sbin,bin}
PEDANTIC
[?] change INSTALL="install -p" to INSTALL="%{__install} -p"
[?] change %{_mandir}/man1/* to %{_mandir}/man1/%{name}.1*
--
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, or are watching someone who is.
More information about the Fedora-package-review
mailing list