-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
MIME type of merge files set as "video/mp4" #25
Comments
Thanks for reporting. This seems to occur because ffmpeg is told to map all metadata of input files by default - so covers are included as video part and this results in the given mime type. You could you could use the -vn option of ffmpeg, to remove all video data:
Here is what i tested successfully on my system:
But I think this will also remove the cover. I'm not sure what you would expect, but if you don't need covers, you could try the What do you think? |
Update: ffmpeg seems to ignore I would appreciate your feedback, if the attached m4b-tool version produces working files - don't forget to provide the |
Thank you for looking into it. I remade the merged files since before I didn't know about the --no-conversion option. Used that and --fix-mime-type and the metadata looks ok. Will test soon with the iOS Podcasts app but I assume it will be ok. I think for usability, --no-conversion should be default. Same with preserving the original mimetype if possible, if somehow m4b-tool could detect the original and use that. I think that'd be much better and any parameters should be for changing default behavior. "merge" I think should be just that, with original encoding, metadata, etc. |
iOS Podcasts still shows a video window, though it may be due to something else. I'll verify podcast settings though I thought that was ok. Unrelated, using option --no-conversion, tracks seem to not be merged and there's no chapter info. |
Figured out that in the podcast xml file, type needs to be "audio/x-m4a". "audio/x-m4b" is what shows a video window; perhaps the app doesn't read the file metadata. So perhaps the mime type is of less importance for podcast publication though I think it is still preferable to retain the original metadata if possible. I haven't tested the combination of "audio/x-m4a" with files without the "--fix-mime-type" option; if you'd like, I could. |
If
I am absolutely your opinion. Using sane defaults is something that is very important to me and i already tested codec detection and automatically skipping the conversion step. I understand that the extra encoding step is time consuming and produces little lower audio quality, which really does not make sense. That said, So the most reliable way to merge files in a simple command line interface (which is one of
So as Is this ok? |
I'll send you some files. Without --fix-mime-type .m4bs of mime type "audio/mp4" turn into "video/mp4".
Thank you for the help. Yes you can close this. |
Do you have an email I can send files to? I mostly have commercial audiobooks I'm merging and prefer to share another way. I'm finding that with --no-conversion and I believe --skip-cover, plus first removing covers with AtomicParsley, mime type is still set to video unless I use --fix-mime-type. I haven't tested all combos of parameters but that seems to be the case. |
Thank you for offering the original files, but i think we should not share commercial audiobooks for legal reasons. I think you have a specific workflow. Perhaps it would be useful, if you describe the whole process in, what you would like to do, how your directories are structured and which commands you are executing, so that i can try to reproduce the problem and fix the problems that you are facing instead of adding more and more flags that are very specific and not what you need. I think there must be a more generic way to handle this. What do you think? |
sounds good, thank you. It does seem that
With source files in ~/temp2, command is usually:
btw, I've noticed sometimes chapter marks are incorrect. Have you seen that? Maybe source files were incorrectly split (will have to verify) but I think they are ok. When I have a chance to look again, I can make a new issue if that is the case. |
Yes, and there unfortunately is no way around this. The flag is needed because Since an extra transcoding-step is required to fix it and the
Looks reasonable to me. I would recommend a little change for the next
This issue is degenerating a little. To summarize, here are the two things that did not work for you, at least in some cases:
For now, i handle this issue as closed, new issues with more specific description help me to save more time for development. Thank you for your effort. |
Thanks for the clarification. Agree about the issue degrading. It works for me and I don’t mind specifically using some parameters. Just sought some clarification. As for —no-conversion, perhaps some of my steps can be done by default so that —no-conversion can someday be default and work without error. Minor thing for the future if it’s worth it. Will create new issues if I find something else. |
Merging .m4b files with original metadata reported by exiftool:
File Type : M4A
File Type Extension : m4a
MIME Type : audio/mp4
results in:
File Type : MP4
File Type Extension : mp4
MIME Type : video/mp4
I'm having trouble getting them to show up as audio in the default iOS Podcasts app. Seems Podcasts thinks they're video files and displays a video window. Is there anyway to fix already merged files rather than having to re-merge? Maybe some other tool?
The text was updated successfully, but these errors were encountered: