Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 195223 - Review Request: pavucontrol: Volume control for PulseAudio
Summary: Review Request: pavucontrol: Volume control for PulseAudio
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 195221
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-06-14 15:24 UTC by Pierre Ossman
Modified: 2007-11-30 22:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-09-09 18:05:22 UTC
Type: ---
Embargoed:
j: fedora-cvs+

Attachments (Terms of Use)

Description Pierre Ossman 2006-06-14 15:24:16 UTC 
Spec URL: http://homes.drzeus.cx/~drzeus/polypaudio/pavucontrol.spec
SRPM URL: http://homes.drzeus.cx/~drzeus/polypaudio/pavucontrol-0.9.1-1.src.rpm
Description:
Polypaudio Volume Control (pavucontrol) is a simple GTK based volume control
tool ("mixer") for the Polypaudio sound server. In contrast to classic mixer
tools this one allows you to control both the volume of hardware devices and
of each playback stream separately.
Comment 1 Pierre Ossman 2006-07-09 11:56:12 UTC 
Spec URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol.spec
SRPM URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol-0.9.2-1.src.rpm
Comment 2 Pierre Ossman 2006-08-20 10:58:04 UTC 
Spec URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol.spec
SRPM URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol-0.9.3-1.src.rpm
Comment 3 Brian Pepple 2006-09-06 23:42:12 UTC 
Quick note:

1. Desktop file handling does not conform to Fe guidelines.  Refer to
http://fedoraproject.org/wiki/Packaging/Guidelines#head-254ddf07aae20a23ced8cecc219d8f73926e9755
Comment 4 Pierre Ossman 2006-09-08 05:10:49 UTC 
I read that portion of the guidelines as relating to the case of when you have
to include a .desktop file yourself (i.e. when upstream doesn't have one). If
not, I would have expected a "rm" somewhere in the script to first kill off the
.desktop file the Makefiles have installed.
Comment 5 Brian Pepple 2006-09-08 12:47:02 UTC 
You would simply add something like:

desktop-file-install --vendor fedora --delete-original	\
  --dir $RPM_BUILD_ROOT%{_datadir}/applications   	\
  --add-category X-Fedora			        \
  $RPM_BUILD_ROOT%{_datadir}/applications/%{name}.desktop

to the install section of your spec file.
Comment 6 Rex Dieter 2006-09-08 13:09:17 UTC 
So that the .desktop file (name) doesn't vary from upstream, I'd suggest 
using:
--vendor=""
instead.
Comment 7 Rex Dieter 2006-09-08 13:14:01 UTC 
spec looks clean,simple, just
* MUST: add to %install section
desktop-file-install \
  --dir $RPM_BUILD_ROOT%{_datadir}/applications         \
  --add-category="X-Fedora" --vendor=""                  \
  $RPM_BUILD_ROOT%{_datadir}/applications/%{name}.desktop

do that, and pending my confirmation for building in mock and rpmlint sanity 
checking, I'll APPROVE this.
Comment 8 Pierre Ossman 2006-09-08 21:00:57 UTC 
Spec URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol.spec
SRPM URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol-0.9.4-1.src.rpm
Comment 9 Brian Pepple 2006-09-08 22:33:09 UTC 
(In reply to comment #6)
> So that the .desktop file (name) doesn't vary from upstream, I'd suggest 
> using:
> --vendor=""
> instead.

Rex, maybe I'm missing something here.  Why does it matter if you are setting
the vendor?
Comment 10 Rex Dieter 2006-09-08 22:58:18 UTC 
> Why does it matter if you are setting the vendor?

What *matters* is that .desktop files not get renamed, and adding --vendor does
just that.
Comment 11 Brian Pepple 2006-09-08 23:18:17 UTC 
(In reply to comment #10)
> > Why does it matter if you are setting the vendor?
> 
> What *matters* is that .desktop files not get renamed, and adding --vendor does
> just that.

I was aware that adding the vendor changes the desktop filename, but I guess my
question is why does it matter if the desktop file gets renamed?  Plenty of
packages in extras do it.
Comment 12 Rex Dieter 2006-09-09 00:23:30 UTC 
> why does it matter if the desktop file gets renamed? 

Lots of reasons, one of which is menu editing.

> Plenty of packages in extras do it.

I know, but they (mostly) shouldn't have.  But, since the files were renamed
once, they probably ought to stay that way (else we'd comitt the sin of renaming
them *again*).
Comment 13 Pierre Ossman 2006-09-09 11:17:57 UTC 
Spec URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol.spec
SRPM URL: http://homes.drzeus.cx/~drzeus/pulseaudio/pavucontrol-0.9.4-2.src.rpm

BR added.
Comment 14 Rex Dieter 2006-09-09 12:34:55 UTC 
One possible simplification is replace in %files:
%dir %{_datadir}/pavucontrol
%{_datadir}/pavucontrol/pavucontrol.glade
with
%{_datadir}/pavucontrol/
Comment 15 Rex Dieter 2006-09-09 12:36:08 UTC 
mock builds fine, rpmlint is happy:
$rpmlint pavucontrol-0.9.4-2.fc6.i386.rpm 386.rpm 
W: pavucontrol incoherent-version-in-changelog 0.9.4-2 0.9.4-2.fc6

APPROVED.
Comment 16 Lennart Poettering 2007-07-30 17:34:38 UTC 
Package Change Request
======================
Package Name: pavucontrol
Updated Fedora Owners: lpoetter,drzeus-bugzilla
Comment 17 Lennart Poettering 2007-07-30 17:37:26 UTC 
Sorry, I forgot the justification for the owner change: I am upstream for this
package and Pierre agreed to co-maintain this package with me from now on.
Comment 18 Jason Tibbitts 2007-07-30 17:37:51 UTC 
CVS done.
Note You need to log in before you can comment on or make changes to this bug.