[Bug 222374] Review Request: paprefs - Management tool for PulseAudio

[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: paprefs - Management tool for PulseAudio


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





------- Additional Comments From eric.moret at epita.fr  2007-01-23 14:13 EST -------
Created an attachment (id=146339)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=146339&action=view)
New spec

I agree that a name such as pulseaudio-preferences would make more sense. The
reason I named it paprefs was for the sake of consistency. All current gtk
based packages for pulseaudio have the name of their executable: pavumeter,
padevchooser, pavucontrol, paman while all pulseaudio backend package names
start with pulseaudio. Maybe should we rename the gtk based packages to the
more meaningful pulseaudio-whatever.

I am not utterly familiar with desktop-file-install, could you point me to the
page that describe the deprecation of some of those parameters. ie: removing
the "--vendor=" statement resulted in an rpmbuild failure with the below error:


+ desktop-file-install --dir
/var/tmp/paprefs-0.9.5-1-root-emoret/usr/share/applications --remove-category
Application
/var/tmp/paprefs-0.9.5-1-root-emoret/usr/share/applications/paprefs.desktop
Must specify the vendor namespace for these files with --vendor

Re: BuildRequires I took the paman spec file as a template to derive the spec
for paprefs. I have addressed all concerns regarding the issues you kindly
outlined in Comment #1 and I think most would also apply to all gtk based
pulseaudio packages (pavumeter, padevchooser, pavucontrol, paman), so you may
want to provide feedback for those other packages as well.

-- 
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.