search

cniw / mpv-discordRPC

Discord Rich Presence integration for mpv Media Player
MIT License
133 stars 10 forks source link

Fixes pypresence wrong python version issues #22

Closed naxay07 closed 2 years ago

naxay07 commented 2 years ago

When pypresence selected as rpc wrapper, a typo in the code causes it to refer to python 2 instead of python 3 in macOS and most Linux systems. As a result code exits with the error ImportError: No module named pypresence and this can be observed in the mpv output in CLI. This change has not been tested on Windows.

cniw commented 2 years ago

This is like revert of this https://github.com/cniw/mpv-discordRPC/commit/4692413931f84051dbc120ef8566eefff23e94be commit, but if it can make the script avoiding an error and make it works more universal i think it's good enough. LGTM

nb: python 2 was already reach the end-of-life since January 1st 2020 so you shouldn't have that in your system nowaday.