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@xxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review