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=461011 --- Comment #5 from Kevin Kofler <kevin@xxxxxxxxxxxxxxxx> 2008-09-09 12:04:41 EDT --- MUST Items: + rpmlint output OK (no warnings or errors) + named and versioned according to the Package Naming Guidelines + spec file name matches base package name + Packaging Guidelines: ! License is listed as GPLv3+, is actually GPLv2+ + No known patent problems + No emulator, no firmware, no binary-only or prebuilt components + Complies with the FHS + proper changelog, tags, BuildRoot, BuildRequires, Summary, Description ! Requires: kdelibs4 >= 4 should be >= 4.1 if mentioned, but it isn't needed at all, as the autoreqs take care of it (libplasma.so.2, i.e. kdebase-workspace-libs 4.1 is automatically dragged in, and that Requires a kdelibs4 >= 4.1 already) + no non-UTF-8 characters + upstream TODO is included, no other documentation + RPM_OPT_FLAGS are used (%cmake_kde4 macro) + debuginfo package is valid + no static libraries nor .la files + no duplicated system libraries + no rpaths + no configuration files, so %config guideline doesn't apply + no init scripts, so init script guideline doesn't apply + no GUI executables (in the proper sense), so no .desktop file needed + ... and thus no desktop-file-install needed either + no timestamp-clobbering file commands + _smp_mflags used ! scriptlets: mkdir %{buildroot} fails in Rawhide because the parent directory of the new RPM's new buildroot is not getting created by mock (unless that has been fixed), it's probably easiest to just omit it and let make install create the directories recursively. (The buildroot used by the Rawhide RPM is secure against race conditions, by the way, it's under the user's home directory.) + not a web application, so web application guideline doesn't apply + no conflicts + complies with all the legal guidelines ! COPYING not packaged as %doc ! source does not match upstream 1.0 release at: http://sourceforge.net/project/showfiles.php?group_id=238004&package_id=289266&release_id=622496 The source tarball is polluted with .svn directories, probably a checkout rather than an export. Please use the upstream tarball instead! ! source tarball should be specified using full URL according to the guidelines for SourceForge-hosted packages https://fedoraproject.org/wiki/Packaging/SourceURL -> http://downloads.sourceforge.net/lancelot-menu/lancelot-1.0.tar.bz2 + builds on at least one arch (F9 Koji scratch build on all arches) + no non-working arches, so no ExcludeArch needed + no missing BuildRequires (builds in Koji) + no translations, so translation/locale guidelines don't apply + ldconfig correctly called in %post and %postun + package not relocatable ! unowned directory %{_kde4_appsdir}/desktoptheme/default/lancelot/ -> please remove the * from %{_kde4_appsdir}/desktoptheme/default/lancelot/* (other than that, no package-specific directories which would have to be owned, doesn't own directories owned by another package) + no duplicate files in %files + permissions correct, defattr used correctly + %clean section present and correct + macros used where possible + no non-code content + no large documentation files, so no -doc package needed + no %doc files required at runtime + no header files which would need to be in a -devel subpackage + no static libraries, so no -static package needed + no .pc files, so no Requires: pkgconfig needed + no devel symlinks which would need to be in a -devel subpackage + plugins in %{_kde4_libdir}/kde4/ are correctly NOT in a -devel subpackage + no -devel package, so "-devel should require %{name} = %{version}-%{release}" is irrelevant + no .la files + no GUI executables, so no .desktop file needed + buildroot is deleted at the beginning of %install + all filenames are valid UTF-8 SHOULD Items: + license already included upstream + no translations for description and summary provided by upstream + package builds in mock, on all architectures (F9 Koji scratch build) * not tested functionality (will do once the other stuff is fixed) + scriptlets are sane + no subpackages other than -devel, so "Usually, subpackages other than devel should require the base package using a fully versioned dependency." is irrelevant + no .pc files, so "placement of .pc files" is irrelevant + no file dependencies Please fix the following MUST FIX items: * License is listed as GPLv3+, is actually GPLv2+ * remove unneeded Requires: kdelibs4 >= 4, a more precise dependency is already implied automatically * remove the explicit mkdir %{buildroot} (at least until the mock used by Koji gets fixed) * add COPYING as %doc * use the upstream tarball from: http://sourceforge.net/project/showfiles.php?group_id=238004&package_id=289266&release_id=622496 * Source: should have full URL: http://downloads.sourceforge.net/lancelot-menu/lancelot-1.0.tar.bz2 * remove the * from %{_kde4_appsdir}/desktoptheme/default/lancelot/* (to own the directory) Once these are fixed, I'll do a short test to make sure it actually works and then 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