http://bugzilla.rpmfusion.org/show_bug.cgi?id=782
Hans de Goede <j.w.r.degoede(a)hhs.nl> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |3
Status|NEW |ASSIGNED
--- Comment #2 from Hans de Goede <j.w.r.degoede(a)hhs.nl> 2009-09-07 13:35:29 ---
Full review done, looks good, only issue I have is that I don't like the
Conflicts. Looking at %install it should be very easy to just install this as
/usr/bin/sonic-visualiser-freeworld
And also rename the icon and update the .desktop to refer the renamed icon
(and rename the .desktop too).
This way we don't need the Conflicts, I know having both installed makes little
sense, but AFAIK Conflicts will confuse yum and esp. PackageKit, so by avoiding
the conflict, we allow users to use gui tools to install this even if they also
have the Fedora version.
Also, please update the icon cache scripts to match the new template:
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache
--
Configure bugmail:
http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.