Opened 9 years ago

Closed 8 years ago

#8693 closed patch (fixed)

Add support for controlling PulseAudio stream volume.

Reported by: pmhahn+mythtv@… Owned by: JYA
Priority: minor Milestone: 0.25
Component: MythTV - Audio Output Version: 0.23-fixes
Severity: medium Keywords: pulseaudio volume
Cc: Ticket locked: no

Description

PulseAudio? can controll the individual stream volume in addition to controlling the sink volume. Since I use MythTV on my desktop computer, muting and changing the sink-volume changes all sounds, including other sounds like desktop events and other audio streams. This patch adds support to controlling the stream volume by selecting "PCM" for the MixerControl?. Selecting "Master" uses the old schema of controlling the sink volume.

The second part of this patch also sets several properties which help in identifying the right stream with PulseAudio? enabled applications.

Attachments (2)

mythtv-pulseaudio.diff (4.8 KB) - added by pmhahn+mythtv@… 9 years ago.
Add support for controlling PulseAudio? stream volume.
mythtv-pulseaudio.2.diff (4.5 KB) - added by pmhahn+mythtv@… 9 years ago.
patch for audiooutputpulse.cpp from trunk (r26372)

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by pmhahn+mythtv@…

Attachment: mythtv-pulseaudio.diff added

Add support for controlling PulseAudio? stream volume.

comment:1 Changed 9 years ago by robertm

Status: newassigned

comment:2 Changed 9 years ago by JYA

Status: assignedinfoneeded

Your patch do not apply with trunk. As nothing has been modified in a while, I suspect that your patch didn't apply 7 weeks ago either.

Could you please update it?

Thank you

comment:3 Changed 9 years ago by JYA

didn't notice it was for 0.23-1... Please submit a patch for trunk. This area was significantly rewritten

Changed 9 years ago by pmhahn+mythtv@…

Attachment: mythtv-pulseaudio.2.diff added

patch for audiooutputpulse.cpp from trunk (r26372)

comment:4 Changed 9 years ago by pmhahn+mythtv@…

A attached an updated patch for trunk. Only compile testes since I only have one MythTV-PC which is actively used.

comment:5 Changed 9 years ago by robertm

Status: infoneededassigned

comment:6 Changed 8 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: assignedclosed

Add support for controlling PulseAudio? stream volue.

Fixes #8693

Signed-off-by: Jean-Yves Avenard <jyavenard@…>

Branch: master Changeset: 8c5aafc30ffa3423892e9d112287f4d13e48cea5

Note: See TracTickets for help on using tickets.