notandy / ympd

Standalone MPD Web GUI written in C, utilizing Websockets and Bootstrap/JS
http://www.ympd.org
GNU General Public License v2.0
515 stars 143 forks source link

ability to select the output #50

Open ilgiznurgaliev opened 9 years ago

ilgiznurgaliev commented 9 years ago

Hi, Could you add the ability to select the output?

Thanks for your work. It's the best web gui for mpd.

\Ilgiz.

rhoering commented 9 years ago

I second this request. :)

mxjeff commented 9 years ago

:+1: for this feature request

kurt-vd commented 9 years ago

I just added this into my fork https://github.com/kurt-vd/ympd The merge request is pending ...

ilgiznurgaliev commented 9 years ago

Hi,

It's strange, but now I get the following error:

ympd -h 127.0.0.1 -p 6600 -w 8082

Mongoose error: Cannot bind to port

Is there somthing I missed?

2015-04-28 15:11 GMT+03:00 Kurt Van Dijck notifications@github.com:

I just added this into my fork https://github.com/kurt-vd/ympd The merge request is pending ...

— Reply to this email directly or view it on GitHub https://github.com/notandy/ympd/issues/50#issuecomment-97038945.

Best regards Ilgiz Nurgaliev

kurt-vd commented 9 years ago

Please run $ sudo netstat --tcp --listen -n -p | grep 8082 and see what other process has already bound to port 8082

Kurt

ilgiznurgaliev commented 9 years ago

Hi, No any process listen 8082 port, that's why it looks strange.

media ~ $ netstat --tcp --listen -n -p | grep 8082

media ~ $ lsof -i :8082 media ~ $ whoami root media ~ $ yaourt -Qs ympd ==> WARNING: Building package as root is dangerous. Please run yaourt as a non-privileged user. local/ympd 1.2.3-1 A standalone MPD Web GUI

Let me know pls if you need any other info? I'll be happy to help.

Thanks. Ilgiz.

2015-04-28 21:52 GMT+03:00 Kurt Van Dijck notifications@github.com:

Please run $ sudo netstat --tcp --listen -n -p | grep 8082 and see what other process has already bound to port 8082

Kurt

— Reply to this email directly or view it on GitHub https://github.com/notandy/ympd/issues/50#issuecomment-97169184.

Best regards Ilgiz Nurgaliev

ilgiznurgaliev commented 9 years ago

It works if I do not use -h and -p options:

media ~ $ ympd -w 8082 MPD Connecting to 127.0.0.1:6600 MPD connected.

Now I'm able to browse webui, but I couldn't find output settings? How can I change output? There is also ympd-git package in aur, probably I should install it instead?

2015-04-28 22:13 GMT+03:00 Ильгиз Нургалиев ilgiznurgaliev@gmail.com:

Hi, No any process listen 8082 port, that's why it looks strange.

media ~ $ netstat --tcp --listen -n -p | grep 8082

media ~ $ lsof -i :8082 media ~ $ whoami root media ~ $ yaourt -Qs ympd ==> WARNING: Building package as root is dangerous. Please run yaourt as a non-privileged user. local/ympd 1.2.3-1 A standalone MPD Web GUI

Let me know pls if you need any other info? I'll be happy to help.

Thanks. Ilgiz.

2015-04-28 21:52 GMT+03:00 Kurt Van Dijck notifications@github.com:

Please run $ sudo netstat --tcp --listen -n -p | grep 8082 and see what other process has already bound to port 8082

Kurt

— Reply to this email directly or view it on GitHub https://github.com/notandy/ympd/issues/50#issuecomment-97169184.

Best regards Ilgiz Nurgaliev

Best regards Ilgiz Nurgaliev

Kernald commented 9 years ago

While there may be an issue, this has nothing to do with this one, maybe create a separate issue?

Le mar. 28 avr. 2015 21:20, ilgiznurgaliev notifications@github.com a écrit :

It works if I do not use -h and -p options:

media ~ $ ympd -w 8082 MPD Connecting to 127.0.0.1:6600 MPD connected.

Now I'm able to browse webui, but I couldn't find output settings? How can I change output? There is also ympd-git package in aur, probably I should install it instead?

2015-04-28 22:13 GMT+03:00 Ильгиз Нургалиев ilgiznurgaliev@gmail.com:

Hi, No any process listen 8082 port, that's why it looks strange.

media ~ $ netstat --tcp --listen -n -p | grep 8082

media ~ $ lsof -i :8082 media ~ $ whoami root media ~ $ yaourt -Qs ympd ==> WARNING: Building package as root is dangerous. Please run yaourt as a non-privileged user. local/ympd 1.2.3-1 A standalone MPD Web GUI

Let me know pls if you need any other info? I'll be happy to help.

Thanks. Ilgiz.

2015-04-28 21:52 GMT+03:00 Kurt Van Dijck notifications@github.com:

Please run $ sudo netstat --tcp --listen -n -p | grep 8082 and see what other process has already bound to port 8082

Kurt

— Reply to this email directly or view it on GitHub https://github.com/notandy/ympd/issues/50#issuecomment-97169184.

Best regards Ilgiz Nurgaliev

Best regards Ilgiz Nurgaliev

— Reply to this email directly or view it on GitHub https://github.com/notandy/ympd/issues/50#issuecomment-97174972.

ilgiznurgaliev commented 9 years ago

Yes, I agree. Sorry for that. kurt-vd, I have tried your fork and it works perfectly. Thanks a lot for your work. Now I'm able to turn on/off music in different rooms of my house.

image

ilgiznurgaliev commented 9 years ago

One issue has been found today if password defined in mpd.conf:

media ~ $ ympd -w 8082 MPD Connecting to 127.0.0.1:6600 MPD connected. MPD mpd_run_status: you don't have permission for "outputs" Segmentation fault (core dumped)

kurt-vd commented 9 years ago

Hey,

One issue has been found today if password defined in mpd.conf:

media ~ $ ympd -w 8082 MPD Connecting to 127.0.0.1:6600 MPD connected. MPD mpd_run_status: you don't have permission for "outputs" Segmentation fault (core dumped)

Interesting. I don't have permissions configured in MPD, as you would have guessed.

I'm not sure I can contribute there. Can you share your permissions-related configuration of MPD (with blanked passwords). I'm curious how one would set it up, and I'm too lazy to experiment until I find your situation.

Kurt

ilgiznurgaliev commented 9 years ago

password "MyPassword@read,add,control,admin"

That issue is not really critical for me, I just experimented with this.

kurt-vd commented 9 years ago

One issue has been found today if password defined in mpd.conf:

media ~ $ ympd -w 8082 MPD Connecting to 127.0.0.1:6600 MPD connected. MPD mpd_run_status: you don't have permission for "outputs" Segmentation fault (core dumped)

I was able to recreate the problem. I solved the issue by properly handling mpd errors when processing outputs. It works now! I don't know exactly why it crashed, maybe something inside libmpd_client.