[Rpm-maint] [rpm-software-management/rpm] Support libgrypt as crypto library (#826)
notifications at github.com
Mon Sep 9 10:42:08 UTC 2019
In a perfect world I'd be much happier with less choice over crypto implementations, but as it is, with our default being something of a cross between a gorilla and an elephant on our back, and the nicer one being license-restricted...
Did a quick performance comparison between the variants, ```time ./rpm -qa|wc -l``` on my laptop, results are fastest from three runs each:
$time ./rpm -qa|wc -l
This comparison is quite favorable too:
$ wc -l rpmio/digest_*.c
It's the simplest of them all code-wise and second fastest overall, beating our current default at both and with a nice license. Not to mention just a fraction of the size of the nss on the library side. What's not to like?
I'll be happy to merge, but I'd actually suggest going one step further: add a separate commit to make libgcrypt the default.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Rpm-maint