[Bug 235815] Review Request: freetennis - Tennis simulation game
bugzilla at redhat.com
bugzilla at redhat.com
Wed May 2 18:56:51 UTC 2007
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: freetennis - Tennis simulation game
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235815
------- Additional Comments From j.w.r.degoede at hhs.nl 2007-05-02 14:56 EST -------
Initial list with some things to fix after a quick initial look.
- The main "%files" is missing a %defattr, this should be added as the first
line after %files
- %files-data can be written mich shorter, it can (and should be) as simple as:
%files data
%defattr(-,root,root,-)
%doc COPYING CHANGES.txt AUTHORS
%{_datadir}/%{name}
Also notice that %defattr is the first line after %files data, putting
%defattr at the end has no effects as %defattr is only applied to files listed
below it, but read on...
- The data subpackage should require the main package, so that it gets removed
with it, but read on...
- The data package doesn't need to double some of the docs from the main package
as it requires it, but read on...
- However since the data is in the same upstream tarbal as the sources, and
this cannot be in a seperate srpm there is no use (what so ever) in having
it in a subpackage, so just completely remove the data sub-package,
and add the single line "%{_datadir}/%{name}" to the main %files list
- Please add an icon, just take any suitable graphics included and resize / edit
it as nescesarry then install it under /usr/share/icons/hicolor/<size>/apps
and add the necessary scripts to update the icon-cache. (See scriptlets wiki
page)
p.s. Do you still need a sponsor? Once this and the deps have been reviewed I
can sponsor you if needed.
--
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.
More information about the package-review
mailing list