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

Do not overwrite existing lyrics #3

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

jgoguen
Copy link
Contributor

@jgoguen jgoguen commented Jul 21, 2024

Instead of overwriting the lyrics file, which may have been locally modified after download, skip processing if the lyrics file is already present.

Skipping 06 Vengeance Venom, lyrics file exists: /opt/docker-data/jellyfin/media/music/Leaves' Eyes/King of Kings (Deluxe Version)/06 Vengeance Venom.lrc
[exact_search] requesting: http://lrclib.net/api/get?track_name=09+Haraldskv%C3%A6di&artist_name=Leaves%27+Eyes&album_name=King+of+Kings+%28Deluxe+Version%29&duration=204

@jgoguen jgoguen force-pushed the no-overwrite-lyrics branch 2 times, most recently from 65b72da to a652e2a Compare July 21, 2024 23:36
@techno-disaster
Copy link
Owner

meep thanks for the change, could you also add a --force file using clap like in the previous PR?

@jgoguen
Copy link
Contributor Author

jgoguen commented Jul 22, 2024

What I'd like to do, just to make it easier so I'm not adding clap in two PRs, is I'll get #4 in a good state then once it's merged I'll rebase and update this one with a --force flag.

README.md Outdated Show resolved Hide resolved
Instead of overwriting the lyrics file, which may have been locally
modified after download, skip processing if the lyrics file is already
present.

```
Skipping 06 Vengeance Venom, lyrics file exists: /opt/docker-data/jellyfin/media/music/Leaves' Eyes/King of Kings (Deluxe Version)/06 Vengeance Venom.lrc
[exact_search] requesting: http://lrclib.net/api/get?track_name=09+Haraldskv%C3%A6di&artist_name=Leaves%27+Eyes&album_name=King+of+Kings+%28Deluxe+Version%29&duration=204
```
@jgoguen jgoguen force-pushed the no-overwrite-lyrics branch from a652e2a to 7bb90db Compare July 26, 2024 02:04
Copy link
Owner

@techno-disaster techno-disaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg thanks!

@techno-disaster techno-disaster merged commit 2a05b03 into techno-disaster:main Aug 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants