Spec URL: http://fab.fedorapeople.org/packages/SRPMS/metromap.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/metromap-0.1.2-1.fc10.src.rpm Project URL: http://metromap.antex.ru/ Description: A simple pygtk+2 application for finding paths in metro (subway) maps. Maps for Moscow, St. Petersburg, Kiev, London, and Berlin are included. Others can be downloaded. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1096182 rpmlint output: [fab@laptop24 noarch]$ rpmlint metromap-0.1.2-1.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [fab@laptop24 SRPMS]$ rpmlint metromap-0.1.2-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
- Licensing issues should be cleaned up: a) There is no indication of GPL version in code or docs at all. (There is no indication of license at all, but that's another question). As such, License tag should be without version - "GPL+". b) Maps data was taken from pmetro.nm.ru. And it have the following license in Russian language (http://pmetro.nm.ru/License.html): Данные, использованные в программе, свободны для некоммерческого использования, а также для GPL проектов. My translation: Data used in program is free for non-commercial use and also for GPL projects. That is not very clear situation. Could you query license for data upstream? In particular they should tell what are terms for: * use * modification * redistribution - Source url is wrong. Instead of http://downloads.sourceforge.net/metromap/metromap-0.1.2.tar.bz2 it should be: http://downloads.sourceforge.net/metro-map/metromap-0.1.2.tar.bz2 - pygtk2-devel is not required in BuildRequires. It should have pygtk2 in Requires instead.
(In reply to comment #1) > - Licensing issues should be cleaned up: > > a) There is no indication of GPL version in code or docs at all. (There is no > indication of license at all, but that's another question). > As such, License tag should be without version - "GPL+". > > b) Maps data was taken from pmetro.nm.ru. And it have the following license in > Russian language (http://pmetro.nm.ru/License.html): > Данные, использованные в программе, свободны для некоммерческого использования, > а также для GPL проектов. > > My translation: > Data used in program is free for non-commercial use and also for GPL projects. > > That is not very clear situation. Could you query license for data upstream? > In particular they should tell what are terms for: > * use > * modification > * redistribution The license files says GPLv2. Fixed > - Source url is wrong. Instead of > http://downloads.sourceforge.net/metromap/metromap-0.1.2.tar.bz2 > it should be: > http://downloads.sourceforge.net/metro-map/metromap-0.1.2.tar.bz2 fixed > - pygtk2-devel is not required in BuildRequires. It should have pygtk2 in > Requires instead. fixed Updated files: Spec URL: http://fab.fedorapeople.org/packages/SRPMS/metromap.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/metromap-0.1.2-2.fc10.src.rpm
It's only one License file, not files.
Created attachment 334403 [details] Added Russian translation for .desktop > The license files says GPLv2. Fixed Version in COPYING doesn't tell what version of GPL it is under: https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F Would be also good to add Russian translation for .desktop (attached file).
+ rpmlint output clean: 2 packages and 0 specfiles checked; 0 errors, 0 warnings. + 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. + File, containing the text of the licenses for the package 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 matches the upstream source, as provided in the spec URL. f3a92b70f7a282ad7b0ff52bec645ab2 metromap-0.1.2.tar.bz2 f3a92b70f7a282ad7b0ff52bec645ab2 metromap-0.1.2.tar.bz2.orig + The package successfully compiles and builds into binary rpms on at least one supported architecture (x86_64). + All build dependencies are listed in BuildRequires. + The spec file handles locales properly. + Does not contain shared libraries. + The package does not designed to be relocatable. + A package owns all directories that it creates. + A package does not contain any duplicate files in the %files listing. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf %{buildroot}. + The package consistently uses macros. + The package contains code, or permissable content. + Does not contain large documentation files. + Includes only doc files in %doc. + No headers. + No static libraries. + The package does not contain pkgconfig(.pc) files. + The package does not contain library files with a suffix (e.g. libfoo.so.1.1). + No devel packages. + The package does not contain any .la libtool archives. + Includes %{name}.desktop file. Properly installed with desktop-file-install. + The package does not own files or directories already owned by other packages. + At the beginning of %install, the package runs rm -rf %{buildroot}. + All filenames in the package are valid UTF-8. SHOULD: Source package does not include license text for data. Query upstream to include it. Source package does not specify what GPL version it is under. Query upstream what intended version is and to correct the sources. This package is APPROVED.
(In reply to comment #4) > Created an attachment (id=334403) [details] > Added Russian translation for .desktop > > > The license files says GPLv2. Fixed > Version in COPYING doesn't tell what version of GPL it is under: > https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F The license in the spec file is GPL+ for now. > Would be also good to add Russian translation for .desktop (attached file). Name[ru]=Карты метро GenericName=Path finder in metro maps Comment[ru]=Навигация по метро Comment=Simple program for finding paths in subway/metro maps Comment[ru]=Программа для нахождения путей в метро I guess that 'Comment[ru]=Навигация...' is the 'GenericName', right? (In reply to comment #5) > SHOULD: > Source package does not include license text for data. Query upstream > to include it. done > Source package does not specify what GPL version it is under. Query upstream > what intended version is and to correct the sources. done > This package is APPROVED. Thanks for the review.
New Package CVS Request ======================= Package Name: metromap Short Description: Simple program for finding paths in subway/metro maps Owners: fab Branches: F-9 F-10 InitialCC:
(In reply to comment #6) > I guess that 'Comment[ru]=Навигация...' is the 'GenericName', right? Yeah, right.
cvs done.
Thanks, Kevin
metromap-0.1.2-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/metromap-0.1.2-3.fc10
metromap-0.1.2-3.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/metromap-0.1.2-3.fc9
metromap-0.1.2-3.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update metromap'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-2566
metromap-0.1.2-3.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update metromap'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-2569
metromap-0.1.2-3.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
metromap-0.1.2-3.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.