Thread: [ANNOUNCE] Alpha release of Open Media Player
View Single Post
pali is offline pali
2012-08-16 , 15:47
Posts: 2,154 | Thanked: 8,464 times | Joined on May 2010
#1218
Originally Posted by gidzzz View Post
Thanks, the button works like a charm with new policy-settings-rx51, but I don't quite get this patch. All I can see it does is it changes the note and removes support for pause/resume on disconnection/connection of wired headsets. Isn't adding 'or CSSU(-devel) installed' to 'Requires "Prevent playback interruptions" disabled' all that has to (and can) be changed on OMP's side?
My first code for headset button in OMP working only if there is no running libplayback application. It turning on/off jackbias via amixer.

But with new patched policy.plc old amixer code in OMP can break policy.plc and nothing will work.

I really suggest to remove old hack code. In merge request is only deleted function updateWiredHeadset which call amixer, nothing more.

If somebody want headset button working it really needs CSSU update. CSSU is right place to update system packages... Also only today was policy-settings-rx51 pushed to cssu-devel but in next days it will be moved to cssu-testing. So merge request can wait until package will be in cssu testing or stable.
 

The Following 4 Users Say Thank You to pali For This Useful Post: