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

id3 tags not writing all the time? #103

Closed
ohnx opened this issue Apr 16, 2016 · 8 comments
Closed

id3 tags not writing all the time? #103

ohnx opened this issue Apr 16, 2016 · 8 comments

Comments

@ohnx
Copy link
Contributor

ohnx commented Apr 16, 2016

i feel like the artist and album tag don't write properly when it tries to write album art to the file?

try this yourself, see if you have the same problem.

this might be more of an issue with the id3 module used - is it node-ffmetadata? (this seems to point to it, but i might be wrong...)

@benkaiser
Copy link
Owner

Yeah I feel the same thing, I have suspected this for a while since when syncing music to my phone it seems to show the wrong coverart for ~80% of my files.

When I get around to having some more time for stretto I'll see if I can work out what's causing the issue.

@ohnx
Copy link
Contributor Author

ohnx commented Apr 16, 2016

https://github.com/parshap/node-ffmetadata/issues

seems like a couple of others have had this issue, too.

@ohnx
Copy link
Contributor Author

ohnx commented Apr 16, 2016

There are other libraries such as https://github.com/Jack12816/id3-writer

@benkaiser
Copy link
Owner

Yeah, we'll have to look into it.

Yeah I saw id3-writer, the thing I don't like about it is it depends on an external binary, and even if that does have builds for all platforms, it means we have to pack a different binary in with each electron build for each platform. The thing I like about node-ffmetadata is that it depends on ffmpeg and we already bundle ffmpeg to do the youtube video to audio conversion.

@ohnx
Copy link
Contributor Author

ohnx commented Apr 17, 2016

Worst case, we can emscripten TagLib ;)

@benkaiser
Copy link
Owner

Now that you mention it. This actually looks really cool: https://github.com/bgrins/videoconverter.js
But it has many issues and I don't want to dump the binary ffmpegs just yet.

@ohnx
Copy link
Contributor Author

ohnx commented Oct 3, 2016

Poked around and apparently the issue is due to a certain argument. Not able to test right now, but does this work? https://www.npmjs.com/package/ffmetadata-ohnx-fork (as a replacement of the current one)

@ohnx
Copy link
Contributor Author

ohnx commented Oct 22, 2016

#120 should close this, too.

@ohnx ohnx closed this as completed Oct 23, 2016
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

No branches or pull requests

2 participants