build: remove libressl check (proposal) #6732

pull laanwj wants to merge 2 commits into bitcoin:master from laanwj:2015_09_remove_libressl_check changing 2 files +6 −45
  1. laanwj commented at 2:15 pm on September 28, 2015: member

    Now that BIP66 passed, OpenSSL is no longer directly part of the consensus. What matters is that DER signatures are correctly parsed, and secp256k1 crypto is implemented correctly (as well as the other functions we use from OpenSSL, such as random number generation)

    This means that effectively, using LibreSSL is not a larger risk than using another version of OpenSSL.

    Remove the specific check for LibreSSL. This makes it much easier to build on platforms that use LibreSSL by default.

    Includes the still-relevant part of #6729: make sure CHECK_HEADER(ec.h) is called using the right CXXFLAGS, not CFLAGS (as AC_LANG is c++).

  2. laanwj added the label Build system on Sep 28, 2015
  3. laanwj renamed this:
    build: remove libressl check
    build: remove libressl check (proposal)
    on Sep 29, 2015
  4. paveljanik commented at 1:20 pm on September 30, 2015: contributor
    ACK
  5. TheBlueMatt commented at 10:20 pm on September 30, 2015: member
    utACK
  6. theuni commented at 1:56 am on October 1, 2015: member
    ut ACK
  7. jgarzik commented at 9:41 am on October 1, 2015: contributor
    ACK
  8. build: remove libressl check
    Now that BIP66 passed, OpenSSL is no longer directly part of the
    consensus. What matters is that DER signatures are correctly parsed, and
    secp256k1 crypto is implemented correctly (as well as the other
    functions we use from OpenSSL, such as random number generation)
    
    This means that effectively, using LibreSSL is not a larger risk than
    using another version of OpenSSL.
    
    Remove the specific check for LibreSSL.
    
    Includes the still-relevant part of #6729: make sure CHECK_HEADER is
    called using the right CXXFLAGS, not CFLAGS (as AC_LANG is c++).
    5978388476
  9. doc: no longer require use of openssl in OpenBSD build guide a3874c7c3a
  10. laanwj force-pushed on Oct 1, 2015
  11. laanwj merged this on Oct 1, 2015
  12. laanwj closed this on Oct 1, 2015

  13. laanwj referenced this in commit bb882d04e8 on Oct 1, 2015
  14. luke-jr referenced this in commit 1782cfe9d3 on Jan 9, 2016
  15. luke-jr referenced this in commit ed06991bcf on Jan 10, 2016
  16. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-18 00:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me