Opened 6 years ago
Closed 4 years ago
#4347 closed enhancement (fixed)
support writing crc elements to level 1 elements of matroska
Reported by: | dericed | Owned by: | |
---|---|---|---|
Priority: | wish | Component: | avformat |
Version: | git-master | Keywords: | mkv bounty |
Cc: | Blocked By: | ||
Blocking: | Reproduced by developer: | no | |
Analyzed by developer: | no |
Description
Summary of the enhancement request:
The Matroska specification states "All level 1 elements should include a CRC-32." http://matroska.org/technical/specs/index.html
I propose supporting an option to add CRC sub-elements while muxing to MKV. Just a dream.
Change History (14)
comment:1 Changed 6 years ago by gjdfgh
comment:2 Changed 6 years ago by dericed
ACtually I'm not sure and do have the same question. However based on the upcoming work at the Preforma project, http://preforma-project.eu/, I think the development of a Matroska implementation checker would feasibly cover this feature. The use of Matroska within a digital preservation context makes the CRC feature very interesting.
comment:3 Changed 6 years ago by dericed
- Summary changed from support write crc elements to level 1 elements of matroska to support writing crc elements to level 1 elements of matroska
comment:4 Changed 5 years ago by dericed
Anyone interested in working on adding crc support to the mkv muxer if this can be sponsored?
comment:5 Changed 5 years ago by dericed
Here is a sample Matroska file with Level 1 CRC Elements. http://archive.org/download/159753000/blackpool_1-0_manutd-regmo.blogspot.com.mkv
comment:6 Changed 5 years ago by dericed
mediainfo can now verify EBML CRC-32 elements in Matroska. I also note that mkvclean can write level 1 CRC-32 elements while re-muxing.
comment:7 Changed 5 years ago by cehoyos
- Keywords mkv added; matroska crc removed
comment:8 Changed 5 years ago by dericed
- Keywords bounty added
comment:9 follow-up: ↓ 10 Changed 5 years ago by dericed
Offering $600 bounty on this feature.
comment:10 in reply to: ↑ 9 Changed 4 years ago by jamrial
comment:11 follow-up: ↓ 12 Changed 4 years ago by dericed
Thanks @jamrial, looking forward to testing it.
comment:12 in reply to: ↑ 11 Changed 4 years ago by jamrial
Replying to dericed:
Thanks @jamrial, looking forward to testing it.
I just sent the set to the mailing list.
comment:13 Changed 4 years ago by dericed
Resolved in https://git.ffmpeg.org/gitweb/ffmpeg.git/commit/3bcadf822711720ff0f8d14db71ae47cdf97e652. Thanks @jamrial.
comment:14 Changed 4 years ago by dericed
- Resolution set to fixed
- Status changed from new to closed
What for? Is there even anything that checks these CRC elements?