baudm / mplayer.py

Lightweight Python wrapper for MPlayer
http://pypi.python.org/pypi/mplayer.py
GNU Lesser General Public License v3.0
117 stars 32 forks source link

noconfig breaks audio detection #31

Closed rohdef closed 6 years ago

rohdef commented 6 years ago

Hey, I had an issue with audio detection. I skimmed the network of projects from this one for solutions and found a fork with a solution. Don't know why there isn't a pull request for it, however, changing it in this project solved the problem:

https://github.com/tb0hdan/mplayer.py/commit/a392ca594dfe7a25278881581342b9910ea27ff4

Could we please get this corrected in the official project too?

rohdef commented 6 years ago

Dispite that fix, so it now detects pulseaudio as expected on my system the video I'm trying to play is not playing, it just freezes. If you have any ideas for that let me know. I'm using apart from the path to the file the code is identical to: https://github.com/baudm/mplayer.py/wiki/QtPlayer (that is I just copied it and corrected the path). Also seems that it is not exiting correctly when exiting the player

baudm commented 6 years ago

Cherry-picked his commit here: https://github.com/baudm/mplayer.py/commit/44da3eccd62430a951eb0660c34bc9f094fb0af6

I think the freezing issue is specific to the MPlayer and PulseAudio interaction and is outside the scope of the wrapper. That said, I'm closing this issue now.