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

Missing cover image when converting into OGG #23

Open
xtemp09 opened this issue Aug 15, 2015 · 3 comments
Open

Missing cover image when converting into OGG #23

xtemp09 opened this issue Aug 15, 2015 · 3 comments

Comments

@xtemp09
Copy link

xtemp09 commented Aug 15, 2015

I converted a FLAC into OGG and found that cover art is missing.

@lordmulder
Copy link
Owner

The Ogg/Vorbis Encoder (oggenc2.exe) does not offer a way to pass an artwork image – that's why.

Maybe artwork could be added, in a separate step, via vorbiscomment.exe, but I'll have to see ...

@xtemp09
Copy link
Author

xtemp09 commented Aug 23, 2015

Opusenc2 doesn't support art embedding; output APE files lack cover image too. It will require third-party tool(s) to add that feature. Looking through temporary directory, I found that LameXP uses lxp_tag for mp3 metadata.

Might I point at a multifunctional library, like TagLib (http://taglib.github.io), that can kill several birds with one stone?

@lordmulder
Copy link
Owner

Nope, 'tag.exe' is used only to embed is only used to emebed ApeTags
into Monkey's Audio files. LAME supports adding meta tags to the MP3 it
creates files just fine.

Since OggEnc2 doesn't support cover artwork, we could use VorbisComment
to add cover artwork to Ogg/Vorbis files. But VorbisComment doesn't
provide a way to easily add cover artwork either. It just takes the
"raw" data. For cover artwork, we would have to put together a
METADATA_BLOCK_PICTURE structure manually. I will have to look into that.

On 23.08.2015 22:15, xtemp09 wrote:

Opusenc2 doesn't support art embedding; output APE files lack cover
image too. It will require third-party tool(s) to add that feature.
Looking through temporary directory, I found that LameXP uses lxp_tag
for mp3 metadata.

Might I point at a multifunctional library, like TagLib
(http://taglib.github.io), that can kill several birds with one stone?


Reply to this email directly or view it on GitHub
#23 (comment).

https://www.youtube.com/v/VF9UMona74w?autoplay=1

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

No branches or pull requests

2 participants