[Bug 238270] Review Request: widelands - realtime-strategy game
bugzilla at redhat.com
bugzilla at redhat.com
Sun May 6 15:55:45 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: widelands - realtime-strategy game
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238270
------- Additional Comments From karlikt at gmail.com 2007-05-06 11:55 EST -------
Thanks for review.
(In reply to comment #18)
> MUST FIX
> ========
> * You install pics/wl-logo-64.png as /usr/share/pixmaps/widelands.png, however
> installing icons under /usr/share/pixmaps is obsolete, it should go under
> /usr/share/icons/hicolor/64x64/apps
Changed
> * Add icon update cache code to %post and %postun, see scriptlets page on the
> wiki
Added
> * Remove update-desktop-database from %post(un) this is only needed when you
> install a new mimitype
Removed
> * Remove "Version" and "TryExec" from the .desktop file. Version should be
> set to the actual package version, not 1.0 since thats kinda hard todo for
> this package and since Version isn't actually used by anything just remove it.
> TryExec isn't needed here.
Done.
> * Merge -data and main package into one, no need / use for a seperate package
Done.
> Should Fix
> ==========
> * You do %define buildnum 10 and then everywhere were you use it you write:
> build%{buildnum} why not just do: "%define build build10" and use %{build}
> where you now use build%{buildnum}?
%define build_id build10
> * Why define rel, why not just directly enter it in the release field?
fixed
> * I agree with your assesment made in comment #3 about the locale files being
> to generic named to go into the system dir, however they should still be
> marked %lang XX (just like config files should be marked %config)
It was a little problem, but now it is OK
New URLs:
http://karlik.nonlogic.org/widelands/widelands.spec
http://karlik.nonlogic.org/widelands/widelands-0-0.3.build10.src.rpm
--
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 Fedora-package-review
mailing list