337034 2014-07-04 08:34:33 +0000 Knotification should use PA directly when available 2014-08-26 14:12:51 +0000 1 1 3 Frameworks and Libraries frameworks-knotifications general unspecified unspecified Linux RESOLVED WONTFIX NOR normal --- 0 web mklapetek cfeck jlp kde.kfoar kdelibs-bugs-null silvan.calarco sitter 0 oldest_to_newest 1459239 0 web 2014-07-04 08:34:33 +0000 KDE 4 and KDE 5 use phonon for playing notification and so are unable to use media.role=event PulseAudio property. This create weird bug like this: https://bugs.kde.org/show_bug.cgi?id=324975 Reproducible: Always Steps to Reproduce: 1. Mute notification volume in kmix 2. play a KDE notification Actual Results: Knotity play sound Expected Results: No sound Two solution: - Use PA when available in notifybyaudio.cpp - Patch PulseAudio (if not already available) to be able to set PA properties Temp fix, use this script as external notification executable: $ cat /usr/local/bin/notify.sh #!/bin/bash paplay --property="media.role=event" "$@" 1459322 1 cfeck 2014-07-04 14:02:31 +0000 Isn't Phonon our abstraction layer? What needs to be changed in Phonon, so that applications do not need to use PA directly? 1459828 2 web 2014-07-07 12:18:00 +0000 A way to pass PA properties. 1460052 3 sitter 2014-07-08 08:29:20 +0000 internal mapping https://projects.kde.org/projects/kdesupport/phonon/phonon/repository/revisions/master/entry/phonon/pulsesupport.cpp#L227 internal map application https://projects.kde.org/projects/kdesupport/phonon/phonon/repository/revisions/master/entry/phonon/pulsesupport.cpp#L1039 internal property creation https://projects.kde.org/projects/kdesupport/phonon/phonon/repository/revisions/master/entry/phonon/pulsesupport.cpp#L1195 internal override setting https://projects.kde.org/projects/kdesupport/phonon/phonon/repository/revisions/master/entry/phonon/pulsesupport.cpp#L1231 if stuff isn't behaving then it's because pulseaudio ignores overrides (which is the only way to set properties when using VLC (https://trac.videolan.org/vlc/ticket/9989)... 1460053 4 sitter 2014-07-08 08:32:42 +0000 Oh and in case you want to fix this in an multimedia library independent way here is the api proposal that needs implementing to provide a cache based playback (which in turn internally could use pulseaudio directoy as pulseaudio provides a sample cache api already). http://markmail.org/thread/3yysp76oz7zjyr3t#query:+page:1+mid:3yysp76oz7zjyr3t+state:results 1460054 5 sitter 2014-07-08 08:35:16 +0000 s/directoy/directly fwiw ^^ 1460128 6 cfeck 2014-07-08 14:50:19 +0000 knotify requests Phonon::NotificationCategory for its audio output from Phonon. If I understand comment #3 correctly, Phonon already has support for passing the category down to pulseaudio. If this does not work correctly, the bug is either in Phonon, or in pulseaudio. Marking as NEEDSINFO, until someone clarifies the status of this bug. 1460194 7 sitter 2014-07-08 21:01:06 +0000 Pulseaudio bug I say. I briefly talked with Colin Guthrie about this a while ago. What happens (and this only ever affects the vlc backend because of the videolan bug I mentioned in comment #3) is that phonon defines override variables in the envrionment before letting vlc create the pulseaudio sink. However pulseaudio then incorrectly merges the diverging data incorrectly, giving the data that vlc provides preference over the override which ultimately sets the role to video and not whatever was set by phonon. This then bypasses the notification volume set in kmix as that doesn't apply to video role sinks and that's why it uses volume 1.0. 1460237 8 cfeck 2014-07-09 07:31:41 +0000 Thanks for the update. In other words, what knotify does is correct, and does not need to be changed.