-
-
Notifications
You must be signed in to change notification settings - Fork 105
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
Replace AtomicParsley with lofty-rs #334
Comments
Couldn't we just use |
Looks like I misread, I thought you had said
|
That's not entirely true. It supports all the tags currently used by Also, metadata written with I don't know if you consider these metadata to be important, but if they aren't, I could make a PR in a few days implementing the use of (*) the flag is |
That is a point of concern for this project. I don't have much time these days, but this is worth investigating. I will if I ever find the time. Or if you do, and can submit a PR, I'd be happy to test and review it. Reusing the embedded The help would be very much appreciated. Otherwise, I'm pretty happy with the state of things as it is. |
Hi, I was working on using I wanted to develop a WASM version of |
That would be amazing, feel free to reach out if you need any help. Thank you! |
Related: #82
In certain cases,
AtomicParsley
falls short. Like embedding long rDNSatom keys. wez/atomicparsley#44In the spirit of #305, we could replace it with a wasm bundled version of https://github.com/Serial-ATA/lofty-rs
Which should allow us to support more formats beyond AAC and even export the original audio with only metadata embedded. #168
And be free of native dependencies.
The text was updated successfully, but these errors were encountered: