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

Front Cover Tag #66

Open
JoooostB opened this issue Nov 11, 2016 · 6 comments · May be fixed by #106
Open

Front Cover Tag #66

JoooostB opened this issue Nov 11, 2016 · 6 comments · May be fixed by #106

Comments

@JoooostB
Copy link

JoooostB commented Nov 11, 2016

I'm able to tag my podcasts, but can't figure out how to add a Front Cover image to the downloaded files. Can you help me out?

@manolomartinez
Copy link
Owner

Hi, the library we use for tagging, stagger, does not support this, I think. But it's pretty easy to do in greg, using a download handler:

Suppose you have eyeD3 installed. Then you could do something like this. First, you create an albumart script (which you need to make executable, and place it somewhere in your PATH) with something like

wget $1 -O $2/$3.mp3
eyeD3 --add-image $4:OTHER $2/$3.mp3

And then, in your greg.conf,

[yourpodcast]

downloadhandler = albumart {link} {directory} {filename_title} /path/to/album_art.jpg

This setup would download your podcasts using wget and then add the album art using eyeD3

@manolomartinez
Copy link
Owner

While the above will work, I have just learned from stagger's dev that this library does support album arts. I'll try to expose that functionality through greg too.

@JoooostB
Copy link
Author

Thank you very much for the detailed instructions, really appreciate it :)

@sabberworm
Copy link
Contributor

Now that greg uses eyeD3 instead of stagger, can you reopen this, @manolomartinez?

@manolomartinez
Copy link
Owner

Thanks for reminding me. If you feel like submitting a PR that'd be great. Otherwise I'll try to get to it as soon as I can.

@manolomartinez manolomartinez linked a pull request Apr 27, 2021 that will close this issue
@manolomartinez
Copy link
Owner

@sabberworm, could you please check whether the above PR works?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants