search

realies / soulseek-docker

🐳 Soulseek Docker Container
https://hub.docker.com/r/realies/soulseek/
MIT License
217 stars 35 forks source link

Issues with spaces in volumes #77

Closed indadb closed 1 year ago

indadb commented 1 year ago

Using last docker container manager in Synology DSM 7.2 I am receiveing errors with next volumes due to spaces between words

-v /volume1/descargas/Soulseek Downloads:/data/Soulseek Downloads \ -v /volume1/docker/soulseek/logs:/data/Soulseek Chat Logs \ -v /volume2/music:/data/Soulseek Shared Folder \

Probably it´s because of the kernel but, can be fixed in any new update?

Thanks and regards eerror

realies commented 1 year ago

I don't have a Synology NAS to try this, but spaces in folder paths should work fine. You can mount any directory anywhere on the container filesystem and select it in Soulseek's settings. It will probably break the links in the sidebar.

https://github.com/realies/soulseek-docker/blob/3ff36e240b634e7b95134bfaf53baa50bbd9b7d6/Dockerfile#L20-L23 Looking at this, you should be able to mount the folders at their /usr/share/novnc/{downloads,shared,logs} paths.

indadb commented 1 year ago

The fact is that the issue happens with the path configured in docker settings, not in the folders I select...

I mean:

/volume1/descargas/Soulseek Downloads ---> it´s my local path and works fine /data/Soulseek Downloads ---> it´s the mount path and trigger an error (if i modify such as /data/Soulseek_Downloads it works.

It partially works but because I can modify download folder in Soulseek seetings but I can´t see how modify logs folder in soulseek gui seetings.

Thanks and regards

realies commented 1 year ago

@indadb, mount at /usr/share/novnc/{downloads,shared,logs} instead of /data/{Soulseek Downloads,Soulseek Chat Logs,Soulseek Shared Folder}.

indadb commented 1 year ago

It works!! thank you very much @realies I really aprecciate your work and support