[Bug 237813] Review Request: taxipilot - Game where you pilot a taxi in space

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: taxipilot - Game where you pilot a taxi in space


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237813





------- Additional Comments From karlikt@xxxxxxxxx  2007-05-03 17:45 EST -------
1. rpmlint returns
[paczka@localhost ~]$ rpmlint /var/lib/mock/fedora-development-i386-core/result
W: taxipilot dangling-symlink /usr/share/doc/HTML/en/taxipilot/common
/usr/share/doc/HTML/en/common
W: taxipilot symlink-should-be-relative /usr/share/doc/HTML/en/taxipilot/common
/usr/share/doc/HTML/en/common
E: taxipilot zero-length /usr/share/apps/taxipilot/game_data/game_data_tag

2. "Data files (maps, pixmaps, sounds) go in  %{_datadir}/%{name} , not
%{_datadir}/games/%{name} . According to the FHS, the use of /usr/share/games is
optional, and we recommend not using it for consistency so that games are
packaged like all other applications."
In the rpm, datafiles are in /usr/share/apps. Please check it.

3. "Installed .desktop files MUST follow the [desktop-entry-spec], paying
particular attention to validating correct usage of Name, GenericName,
[Categories], [StartupNotify] entries."
In .desktop is not defined GenericName and the shortcut is after installation in
Games without any category, check it.

4. "- MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec."
There is %{_libdir}/libEXT_wavpo.la

5. Add NEWS to %doc, please.

I did not check all require items, but these should be checked/fixed.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]