We shouldn't be needing libssl at all; all functions we use are in libcrypto.
Only use the libcrypto part of OpenSSL #58
pull sipa wants to merge 1 commits into bitcoin-core:master from sipa:nolibssl changing 1 files +3 −8-
sipa commented at 5:46 PM on September 1, 2014: contributor
-
Only use the libcrypto part of OpenSSL 3ed6dfae69
-
sipa commented at 6:57 PM on September 1, 2014: contributor
openssl = libssl + libcrypto. We only use things from libcrypto.
-
theuni commented at 7:20 PM on September 1, 2014: contributor
@sipa Looks like I was thrown off at the time by the fact that libcrypto installs headers under openssl/, so I assumed that they may be dependent on some openssl headers (global build config headers, for ex) as a result.
I've tried building and installing libcrypto without libssl to no avail, and distros don't attempt to split them up either. So it seems there's no risk of missing any parent headers. ACK on the change.
Worth renaming the config switch while it's still early to --with-bignum=libcrypto ?
-
theuni commented at 8:03 PM on September 1, 2014: contributor
@sipa sounds good. As for the split, I realize that libssl depends on libcrypto. However, that's not to say that libcrypto doesn't end up sharing some global config headers from the parent project.
For ex, bitcoin's crypto/* files depend on bitcoin-config.h to determine what features it can use. Similarly, libcrypto relies on openssl/opensslconf.h (openssl/crypto.h -> openssl/e_os2.h -> openssl/opensslconf.h). But they're inseparable anyway, so that's no concern to us.
- sipa merged this on Sep 1, 2014
- sipa closed this on Sep 1, 2014
- sipa referenced this in commit f023dc53c4 on Sep 1, 2014
- ysangkok referenced this in commit 11af7015de on Mar 19, 2020