LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead #7520

pull paveljanik wants to merge 1 commits into bitcoin:master from paveljanik:20160211_LibreSSL_compile_fix changing 1 files +3 −1
  1. paveljanik commented at 8:25 PM on February 11, 2016: contributor

    OpenBSD is using LibreSSL by default instead of openssl. The build ends with

    CXX      libbitcoin_server_a-init.o
    In file included from addrman.h:14:0,
                     from init.cpp:12:
    init.cpp: In function 'bool AppInit2(boost::thread_group&, CScheduler&)':
    init.cpp:1095:61: error: 'OPENSSL_VERSION' was not declared in this scope
         LogPrintf("Using OpenSSL version %s\n", OpenSSL_version(OPENSSL_VERSION));
                                                                 ^
    

    This change adds LIBRESSL_VERSION_TEXT macrocheck and adds conditionals before these macros. If none is defined, nothing is printed at all.

    Fixes #7515. Fixes #7447.

  2. laanwj added the label Needs backport on Feb 11, 2016
  3. laanwj added the label Build system on Feb 11, 2016
  4. paveljanik cross-referenced this on Feb 11, 2016 from issue build options for OpenBSD incomplete/out-of-date by ghost
  5. paveljanik force-pushed on Feb 12, 2016
  6. LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead a0a17b3e44
  7. paveljanik force-pushed on Feb 12, 2016
  8. luke-jr commented at 6:06 AM on February 12, 2016: member

    utACK a0a17b3e441ac82bc64ca183d458258ded26c9ed

  9. laanwj commented at 12:05 PM on February 12, 2016: member

    utACK

  10. laanwj merged this on Feb 12, 2016
  11. laanwj closed this on Feb 12, 2016

  12. laanwj referenced this in commit 621940e040 on Feb 12, 2016
  13. paveljanik cross-referenced this on Feb 23, 2016 from issue Fix for #7080 breaks LibreSSL ??? by AliceWonderMiscreations
  14. laanwj removed the label Needs backport on Sep 26, 2016
  15. laanwj added this to the milestone 0.12.2 on Sep 26, 2016
  16. laanwj added the label Needs backport on Sep 26, 2016
  17. fanquake removed the label Needs backport on Mar 7, 2018
  18. barton2526 cross-referenced this on Aug 29, 2021 from issue LibreSSL doesn't define OPENSSL_VERSION, use LIBRESSL_VERSION_TEXT instead by barton2526
  19. bitcoin 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: 2026-05-20 06:55 UTC