search

nekowinston / mpd-rich-presence

Discord Rich Presence for MPD
MIT License
10 stars 3 forks source link

feat: Add support for UNIX socket #5

Closed nathanielevan closed 11 months ago

nathanielevan commented 12 months ago

[why] At present this app only supports TCP connection to an MPD host. I use a UNIX socket for MPD, and this quick patch should add support for it. Let me know what you think. Cheers.

[how] I added a new configuration variable use_socket that takes a Boolean value. If set to true, mpd.Dial() should take "unix" as its first argument, indicating that it's connecting to a UNIX socket, followed by the location of that socket in the filesystem, set by the configuration variable host. The port variable is therefore unused in this mode.

nekowinston commented 11 months ago

Sorry for not responding to this PR in time - I had it on my todo list, but admittedly as pretty low priority since I'm no longer using this program myself. If you want to take it over, I'm happy to archive this repo and redirect to an active fork.

nathanielevan commented 11 months ago

Oh sorry, I closed this PR because I committed my changes to the main branch instead of a feature branch -- I'll make a new one real quick.