Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MPD protocol is not documented #108

Open
anarcat opened this issue Oct 21, 2013 · 7 comments
Open

MPD protocol is not documented #108

anarcat opened this issue Oct 21, 2013 · 7 comments
Assignees

Comments

@anarcat
Copy link

anarcat commented Oct 21, 2013

What does the MPD protocol do? I have tried grepping the web and the documentation, and all I can find are the settings page, which don't really explain what the plugin does.

Anyone care to explain?

@smimram
Copy link
Member

smimram commented Oct 22, 2013

Hi,

The mailing-list would be more appropriate for this kind of questions. The mpd protocol can be used to generate songs by their tags such as mpd:artist=Britney. In order to find songs, Liquidsoap connects to an MPD server and queries it. The server can be configured through mpd.host / mpd.port parameters.

@anarcat
Copy link
Author

anarcat commented Oct 22, 2013

I nevertheless think this is a bug: there is pretty good documentation for other modules, there should be code samples explaining how this works in the documentation. For example, where would I put "mpd:artist=Britney"?

Since documentation is shipped with the liquidsoap source code, it seems to me fair that this is a bug reported against the said source code. If people can brainstorm on how this work, I can formulate a pull request to improve the documentation, after some tests!

@smimram
Copy link
Member

smimram commented Oct 22, 2013

Yes, this could be enhanced in the doc, which is why I did not close the BR. You can use

mpd:tag=value

as any other file (in a playlist for instance).

@anarcat
Copy link
Author

anarcat commented Oct 22, 2013

Thanks! This seems to be the trick, except it doesn't work here, but this may be a 1.0-specific bug... see #111.

@smimram
Copy link
Member

smimram commented Apr 25, 2019

No one should use the MPD protocol in 2019 anymore, closing this.

@smimram smimram closed this as completed Apr 25, 2019
@anarcat
Copy link
Author

anarcat commented Apr 25, 2019

what's wrong with MPD? I still use it and it's working fine... if it shouldn't be used in liquidsoap, seems to be the code should be removed, or it should be documented. either way, something should probably be done with this ticket, no? :)

@smimram
Copy link
Member

smimram commented Apr 26, 2019

Ok fine, I am reopening :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants