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=459828 --- Comment #2 from Peter Lemenkov <lemenkov@xxxxxxxxx> 2008-08-23 11:09:43 EDT --- REVIEW: MUST Items: + rpmlint is silent. + The package is named according to the Package Naming Guidelines . + The spec file name matches the base package %{name}, in the format %{name}.spec. + 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 package spec file matches the actual license. + The text of the license(s) is included in %doc. + The spec file is written in American English. + The spec file for the package is legible. + The sources used to build the package must match the upstream source: [petro@Sulaco SOURCES]$ md5sum LICENSE* ql2500_fw.bin* 4005328a134054f0fa077bdc37aa64f2 LICENSE 4005328a134054f0fa077bdc37aa64f2 LICENSE.orig e0eee74a60109539f1ea39c89ef05a92 ql2500_fw.bin da9804a0f6c216ea79355ee18fbd3d95 ql2500_fw.bin_mid da9804a0f6c216ea79355ee18fbd3d95 ql2500_fw.bin_mid.orig e0eee74a60109539f1ea39c89ef05a92 ql2500_fw.bin.orig [petro@Sulaco SOURCES]$ + The package successfully compiles and builds into binary rpms on at least one supported architecture (ppc). - MUST: All build dependencies must be listed in BuildRequires. There is something, that I can't understand - this package requires /lib/firmware, and "rpm -qf /lib/firmware" shows that that's a udev's "Provides:". I don't know whethet this firmware should require udev, so I dont know whether this is a blocker.On the other hand there atre the following two rules: - MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. Refer to the Guidelines for examples. - MUST: Packages must not own files or directories already owned by other packages. The rule of thumb here is that the first package to be installed should own the files or directories that other packages may rely upon. This means, for example, that no package in Fedora should ever share ownership with any of the files or directories owned by the filesystem or man package. If you feel that you have a good reason to own a file or directory that another package owns, then please present that at package review time. + A package does not contain any duplicate files in the %files listing. + Permissions on files are set properly. + The package must have a %clean section, which contains rm -rf or $RPM_BUILD_ROOT. + The package consistently uses macros, as described in the macros section of Packaging Guidelines . + The package contains permissable content. + Everything, the package includes as %doc, does not affect the runtime of the application. + At the beginning of %install, the package runs rm -rf $RPM_BUILD_ROOT. + All filenames in rpm packages must be valid UTF-8. And some remarks: * You should use %{_lib} instead of /lib * You should invoke 'install' command with -p key in order to preserve timestamp. Please resolve (or explain me, if I misunderstood something) these issues, and I'll approve it. -- 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