[Bug 782] Review Request: sonic-visualiser-freeworld - A program for viewing and exploring audio data

RPM Fusion Bugzilla noreply at rpmfusion.org
Mon Sep 7 13:35:29 CEST 2009


http://bugzilla.rpmfusion.org/show_bug.cgi?id=782


Hans de Goede <j.w.r.degoede at hhs.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |3
             Status|NEW                         |ASSIGNED




--- Comment #2 from Hans de Goede <j.w.r.degoede at 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.


More information about the rpmfusion-developers mailing list