Comment 60 for bug 486154

Revision history for this message
Robert Schroll (rschroll) wrote :

Dmitrijs Ledkovs wrote:
> I'm part of Ubuntu Reviewers team. Your patch is good to brute-force
> revert the change but what about people who want both options and switch
> between the two behavior? As discussed in this long bug report it's not
> possible yet due to many issues raised and the patch doesn't bring
> configurable behavior.

The correct way to allow configuration, IMHO, is to use Pulse Audio's module-x11-bell. It can be turned on and off at will just by loading and unloading that module. module-x11-bell is loaded *by default* in Lucid, so it's ready to be used in all it's configurable glory. But metacity traps system bell events and keeps Pulse Audio from seeing them.

This patch would remove this trapping and *allow configuration* through Pulse Audio. It would also ensure that both metacity and compiz use the same system for ringing the bell. It doesn't completely solve the bug, but it gets us significantly closer.