[Bug 431335] Review Request: decibel-audio-player - Music player for GNOME

[bugzilla at redhat.com]

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: decibel-audio-player - Music player for GNOME


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





------- Additional Comments From debarshi.ray at gmail.com  2008-02-19 12:38 EST -------
(In reply to comment #1)

> However, it must install to %{python_sitelib}  as per
> http://fedoraproject.org/wiki/Packaging/Python  
> At the moment it's /usr/share/decibel-audio-player
 
Those Python files are only meant to be used by Decibel, and not any other
program. Only system-wide files (or modules) should go in %{python_sitelib},
isn't it?

> Speaking of decibel-audio-player, why patch the .desktop to be Decibel Music
> Player?

To ensure consistency in the desktop menus. We have "Banshee Music Player",
"Exaile Music Player", etc.. So the trend seems to be "<name> Music Player" for
such applications.

What do you suggest?

> Why not use the default BuildRoot of 
> %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) ?

That used to be the default BuildRoot. Not anymore. See:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b4fdd45fa76cbf54c885ef0836361319ab962473


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.