[FFmpeg-devel] [PATCH] add md5 muxer

Diego Biurrun diego
Mon Sep 28 03:43:59 CEST 2009


On Sun, Sep 27, 2009 at 11:06:57PM +0400, Michael Kostylev wrote:
> 
> On Sun Sep 27 19:53:18 2009
> Reimar D?ffinger wrote:
> 
> >>> we have -f crc, but IMO 32 bit crc are a bit little for a whole file and
> >>> also we use md5sum in the regression tests, so I'd propose to add a -f
> >>> md5 as in attached patch.
> >> why md5 instead of sha1?
> > Because md5 makes more sense for the simple reasons that
> > 1) we already use it in regression tests
> > 2) a md5/md5sum binary is available anywhere, I do not know
> >    something comparable for sha
> 
> Just for the record, codec-regression.sh contains nothing suitable
> for NetBSD because md5sum does not exist and -r is unsupported by md5. OTOH:
> 
> % gmd5sum -b .zshrc
> 0a4998dca23450d99f6fccee3a28d000 *.zshrc
> 
> % md5 .zshrc
> MD5 (.zshrc) = 0a4998dca23450d99f6fccee3a28d000
>  
> % md5 -n .zshrc
> 0a4998dca23450d99f6fccee3a28d000 .zshrc
> 
> % openssl dgst -md5 .zshrc
> MD5(.zshrc)= 0a4998dca23450d99f6fccee3a28d000

So what would work reliably?  Patches are welcome.

Diego



More information about the ffmpeg-devel mailing list