docs: add slskd example to readme #139
Conversation
| read -t5 -r line | ||
| path=$(echo "$line" | awk -F'"localDirectoryName":"' '{print $2}' | awk -F'"' '{print $1}' | awk '{printf "%s", $0}') |
There was a problem hiding this comment.
i haven't tested, but would this work? just incase a filename has quotes in it
| read -t5 -r line | |
| path=$(echo "$line" | awk -F'"localDirectoryName":"' '{print $2}' | awk -F'"' '{print $1}' | awk '{printf "%s", $0}') | |
| path="$(jq -r ".localDirectoryName")" |
no need for read either since jq will receive stdin
might need to add jq to the docker image too
There was a problem hiding this comment.
actually, it looks like the json comes as an argument to the script, not stdin
There was a problem hiding this comment.
I did a lot of trial and error to make it work using only the tools included in the official docker image
There was a problem hiding this comment.
sure, but I think adding jq as a dependency here https://github.com/sentriz/betanin/blob/master/Dockerfile#L38 would be fine. i think it's worth it to parse the json properly incase the directory path includes special chars like "
There was a problem hiding this comment.
I don't think adding jq to the betanin docker image would help ^^
(the script is run by slskd)
There was a problem hiding this comment.
I could make a pr on the slskd Dockerfile : https://github.com/slskd/slskd/blob/master/Dockerfile
But I don't know if it would be accepted.
There was a problem hiding this comment.
ahhaha oh yes, you're right, ofc 😁
slskd recently allowed to run a script whenever a directory download is completed. The implementation is quite basic but it works nicely with betanin.