Spec URL: http://nicoleau.fabien.free.fr/rpms/SPECS/gestikk.spec SRPM URL: http://nicoleau.fabien.free.fr/rpms/srpms.fc9/gestikk-0.6.1-1.fc9.src.rpm Description: gestikk is an application for nearly all netwm-conform windowmanagers (written in Python) that recognizes predefined mouse gestures and executes defined actions. For exemple by drawing a line you can run Firefox or simulate a key combination. Rebuild under mock (fedora-9-i386) is OK. Rebuild on koji is OK : http://koji.fedoraproject.org/koji/taskinfo?taskID=748648 rpmlint output : [builder@FEDOBOX tmp]$ rpmlint gestikk-0.6.1-1.fc9.noarch.rpm gestikk-0.6.1-1.fc9.src.rpm 2 packages and 0 specfiles checked; 0 errors, 0 warnings. [builder@FEDOBOX tmp]$
I will review this. Stand by for a full review.
MUST items: - rpmlint output: 1 packages and 1 specfiles checked; 0 errors, 0 warnings. - package name: OK - spec file name: OK - Packaging Guidelines: OK; the Python guidelines specify that "BuildRequires: python" must appear. It does not in this spec, but that requirement is picked up transitively via gettext. - licensing guidelines: OK - license: FAIL. The top-level gestikk.py is GPLv2+, but gestikk/condparse.py is under an MIT license. - license file - spec file is American English: OK, although I have a couple of suggestions. See below. - spec file is legible: OK - sources match upstream: OK, md5sum is 7f30737f0b69633b1112688c68f4e3d9 - builds into binary RPM: OK - ExcludeArch: OK - BuildRequires: OK (although I would be happer with an explicit python dependency) - Use of %find_lang: OK - ldconfig: OK - relocatable package: OK - own all created directories: OK - no duplicates in %files: OK, although consider merging "%dir %{python_sitelib}/%{name}" and "%{python_sitelib}/%{name}/*" into just "%{python_sitelib}/%{name}". - proper permissions: OK - %clean section: OK - consistent use of macros: OK - code or permissible content: OK - large documentation files: OK - %doc files not needed at runtime: OK - header files in -devel: OK - static libraries in -static: OK - Requires: pkgconfig: OK - .so files in -devel: OK - -devel requires base package: OK - no libtool archives: OK - desktop file: OK - don't own files/dirs owned by other packages: OK - clean first in %install: OK - filenames are UTF-8: OK SHOULD items: - license file as separate text: you should ask upstream to include the MIT license as a separate file - description and summary contain translations: OK - builds in mock: did not check - builds on all supported arches: did not check - package functions as described: OK (only lightly tested) - scriptlets are sane: OK - subpackages require main package: OK - placement of pkgconfig files: OK - file dependencies: OK
Hi, Thank you for taking this review. I'll contact upstream and update the review ASAP.
We don't need to stall this review for upstream. As long as you ask them to include the MIT license in some future release and fix the License tag in the spec file, we can proceed. I forgot to give you my suggestions for the Summary and %description, though. No offense intended, but I think the English can be improved slightly. Here's what I suggest. Summary: Use mouse gestures to control your PC %description gestikk is a Python application for nearly all netwm-conforming window managers. It recognizes predefined mouse gestures and executes defined actions in response. For example, with the gesture "down - left", you could run Firefox or simulate a window manager keyboard shortcut.
Update : Spec URL: http://nicoleau.fabien.free.fr/rpms/SPECS/gestikk.spec SRPM URL: http://nicoleau.fabien.free.fr/rpms/srpms.fc10/gestikk-0.6.1-2.fc10.src.rpm - License fix - Summary and description update Package builds on koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=1117685 Rpmlint is silent. I've contacted upstream and asked for MIT license file in next release.
Looks good. Please consider the summary and description changes I suggested in comment #4. APPROVED.
Thank you for the review. I've added python BR and used summary and description you propoed in comment #4. New Package CVS Request ======================= Package Name: gestikk Short Description: Use mouse gestures to control your PC Owners: eponyme Branches: F-9 F-10 InitialCC:
cvs done.
As koji seems not closing this bug, I do it manualy. (gestikk is now in stable repository).