Issue with loading mpv config files

[Gregaras]

Issue

Yewtube will not load mpv config files from mpv config directory.

Context

Inside mpv.py get_config_dir is called:

yewtube/mps_youtube/players/mpv.py

Line 288 in f62a5fd

confpath = os.path.join(paths.get_config_dir(), "mpv-input.conf")

yewtube/mps_youtube/players/mpv.py

Line 321 in f62a5fd

confpath = os.path.join(paths.get_config_dir(), "mpv")

Which, like the code itself suggests, returns the config directory path of Yewtube and not the main config directory path or the one of mpv:

yewtube/mps_youtube/paths.py

Line 56 in f62a5fd

return mps_confdir

Solution

Replacing

yewtube/mps_youtube/players/mpv.py

Line 288 in f62a5fd

confpath = os.path.join(paths.get_config_dir(), "mpv-input.conf")

with

confpath = os.path.join(os.path.split(paths.get_config_dir())[0], "mpv", "input.conf")

seems to make mpv input config loading work but I suppose there might be a better way to solve this. Didn't check the _get_conf_dir function, suppose there will be the same thing as well.

Your Environment

OS: GNU/Linux
Distribution: Manjaro

[iamtalhaasghar]

What is the issue again?

[Gregaras]

What is the issue again?

Did a few edits on the description, thought it was self explanatory though.