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

iTunes Atom Support for MP4 #130

Closed
lazka opened this issue Jul 4, 2014 · 5 comments
Closed

iTunes Atom Support for MP4 #130

lazka opened this issue Jul 4, 2014 · 5 comments
Labels

Comments

@lazka
Copy link
Member

@lazka lazka commented Jul 4, 2014

Originally reported by: Christoph Reiter (Bitbucket: lazka, GitHub: lazka)


From jukow...@gmail.com on November 29, 2012 00:39:42

Hi,

It would be really great if the MP4 class could read/write the following:

Chapter Marks [chpl]
MediaKind [stik]
Copyright
Encoding Tool

I have sent (via email) an example file that has all of the above set.

I currently use MP4Chaps and MP4Tags (from the libmp4v2 library) to set these.

Many thanks,

Geoff

Original issue: http://code.google.com/p/mutagen/issues/detail?id=130


@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Sep 22, 2014

Original comment by McAnix (Bitbucket: McAnix, GitHub: McAnix):


Hello, I've submitted a patch for MediaKind, EncodingTool and Copyright, what is the format for chpl?

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Sep 28, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


Issue #90 was marked as a duplicate of this issue.

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Sep 28, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


For chpl see https://www.adobe.com/content/dam/Adobe/en/devnet/flv/pdfs/video_file_format_spec_v10.pdf

It's not in 'ilst', so this would need special handling.

(I've just checked a bunch of mp4s for no-handled atoms, for future reference: Counter({'stik': 1009, 'rtng': 923, 'plID': 839, 'cnID': 835, 'atID': 787, 'geID': 787, 'sfID': 723, 'akID': 594, 'cmID': 512, 'tves': 169, 'tvsn': 169, 'rati': 168, 'rndu': 8, 'rpdu': 8, 'NITR': 2, 'apID': 2}))

@lazka lazka removed the trivial label Apr 27, 2016
@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 7, 2017

Some more: taglib/taglib#758

lazka added a commit that referenced this issue Feb 19, 2017
@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Feb 21, 2017

Should all be there now. For chapters I've opened #295

@lazka lazka closed this Feb 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.