sai80082 / Jellyfin-lyrics

Download lyrics for all of your music
19 stars 8 forks source link

Error while writing lyrics #3

Closed Kakoluz closed 4 months ago

Kakoluz commented 4 months ago

I've ran the script in my linux server, and found that it did nothing. All songs fails to write. I have picked one that was actually successfully fetched online (the script is modified to pick m4a files that I use)

2024-05-30 13:15:14,722 - INFO - Processing file 431 of 1063 - /mnt/nas/Media/Music organized/Hiroyuki Sawano/Avid/Hands Up to the Sky.m4a 2024-05-30 13:15:17,930 - INFO - Found Lyrics for the song: Hands Up to the Sky 2024-05-30 13:15:17,953 - ERROR - Error in writing lyrics for the song: /mnt/nas/Media/Music organized/Hiroyuki Sawano/Avid/Hands Up to the Sky.m4a

the error is not clear, it's not permissions related as I'm running the script as root to avoid that, the target directory is a SMB share but other programs are writing to it fine and can modify from the console as well. So I am a bit lost.

Have been looking at the code but can't find the issue.

UPDATE: I've found the issue, there are a few checks missing and a variable used when undefined. I've modified the script and will PR it later.

sai80082 commented 4 months ago

PR #4 merged.