build: set CXXFLAGS for libressl probe #6729

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2015_09_rightflags_libressl_probe changing 1 files +3 −3

laanwj commented at 3:25 PM on September 27, 2015: member

AC_LANG_PROGRAM checks using g++, not gcc, so set the right flags. Otherwise, SSL_CFLAGS and CRYPTO_CFLAGS are ignored for this check.

build: set CXXFLAGS for libressl probe

AC_LANG_PROGRAM checks using g++, not gcc, so set the right flags.
Otherwise, SSL_CFLAGS and CRYPTO_CFLAGS are ignored for this check.

88458c5d74

laanwj added the label Build system on Sep 27, 2015
laanwj cross-referenced this on Sep 27, 2015 from issue (OpenBSD 5.7) configure: error: Detected LibreSSL: This is NOT supported, and may break consensus compatibility! by ghost
theuni commented at 9:40 PM on September 27, 2015: member

Ugh, whoops. From a quick glance, ACK.
jonasschnelli commented at 6:35 AM on September 28, 2015: contributor

Looks reasonable. utACK.
laanwj cross-referenced this on Sep 28, 2015 from issue doc: Add build guide for OpenBSD 5.7 by laanwj
laanwj referenced this in commit 79df4b0d27 on Sep 28, 2015
laanwj cross-referenced this on Sep 28, 2015 from issue build: remove libressl check (proposal) by laanwj
laanwj commented at 2:17 PM on September 28, 2015: member

Would prefer #6732 to go on, so closing this for now.
laanwj closed this on Sep 28, 2015
laanwj referenced this in commit 5978388476 on Oct 1, 2015
MarcoFalke commented at 7:43 PM on October 8, 2015: member

Note: #6732 got merged.
luke-jr commented at 3:50 AM on January 9, 2016: member

Should this be reconsidered for 0.10/0.11?

EDIT: Nevermind, #6732 makes sense for them too.
luke-jr referenced this in commit 1782cfe9d3 on Jan 9, 2016
luke-jr referenced this in commit ed06991bcf on Jan 10, 2016
rebroad referenced this in commit 07f4ac5903 on Dec 7, 2016
protonn referenced this in commit 121490adc5 on Apr 11, 2017
bitcoin locked this on Sep 8, 2021

Contributors

Labels