On Tue, Nov 15, 2016 at 01:29:15PM +0000, Andras Salamon wrote: > On 2016-11-14 18:08:24 -0800, Kevin J. McCarthy wrote: > > On Mon, Nov 14, 2016 at 11:59:35PM +0100, Evgeni Golov wrote: > > > > For md5, we can use a built-in, or even just use perl's Digest::MD5. > > > built-in like coreutils' md5sum? would prefer that over perl, tbh :) > > > > Yes, I meant md5sum. I'm thinking just to use a wrapper script that > > first tries md5sum, and falls back to a perl blurb if it can't be found. > > What is the precise purpose of requiring an external md5, other than the > one included in the distribution?
When cross-building, mutt_md5 ends up being built for the target system, and so isn't able to run as part of the build process. The mutt_md5 program is used for header cache versioning. The build pipes the headers of core mutt data structures through cpp and then generates an md5 sum of that. That sum is then stored inside each header cache file. We could switch to something else, but that would force-invalidate everyone's header caches. > The versions of Perl shipped with MacOS 10.10 to 10.12 have Digest::MD5; > the Ruby and Python versions don't seem to include an MD5 module. It > doesn't look like Digest::MD5 is a core Perl module so there may be > systems out there with Perl but without Digest::MD5. Looks like it's a core module to me: http://perldoc.perl.org/index-modules-D.html It looks like it was added in version 5.8: http://perldoc.perl.org/perl58delta.html#New-Modules-and-Pragmata released in 2002, so I feel pretty safe counting on it to be there. > MacOS (and possibly other BSD variants) have /sbin/md5 but no md5sum. > This has a slightly different output format. Do you happen to have the output format handy? Thank you! -- Kevin J. McCarthy GPG Fingerprint: 8975 A9B3 3AA3 7910 385C 5308 ADEF 7684 8031 6BDA
signature.asc
Description: PGP signature