Skip to content

Commit 988058d

Browse files
committed
Merge bitcoin/bitcoin#24254: build: Add Boost.Process usage check
abc057c build: Add Boost.Process usage check (Hennadii Stepanov) Pull request description: This PR adds a check that Boost.Process can be used without linking any libraries (header-only). Disable the functionality if that is not the case. Fixes bitcoin/bitcoin#24314. ACKs for top commit: fanquake: ACK abc057c Tree-SHA512: ed2a32b1f751ec6f88cc7220766edd4cdab93c1d0515c892aa3094ee8d5b13ef569830d6e7a7a00c0197b117585dc526d00d943cc99a1f8c8a66ac4e20fe2061
2 parents 9011d4d + abc057c commit 988058d

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

configure.ac

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1437,8 +1437,20 @@ if test "$use_external_signer" != "no"; then
14371437
use_external_signer="no";
14381438
;;
14391439
*)
1440-
use_external_signer="yes"
1441-
AC_DEFINE([ENABLE_EXTERNAL_SIGNER], [1], [Define if external signer support is enabled])
1440+
AC_MSG_CHECKING([whether Boost.Process can be used])
1441+
AC_LINK_IFELSE([AC_LANG_PROGRAM([[#include <boost/process.hpp>]])],
1442+
[have_boost_process="yes"],
1443+
[have_boost_process="no"])
1444+
AC_MSG_RESULT([$have_boost_process])
1445+
if test "$have_boost_process" == "yes"; then
1446+
use_external_signer="yes"
1447+
AC_DEFINE([ENABLE_EXTERNAL_SIGNER], [1], [Define if external signer support is enabled])
1448+
else
1449+
if test "$use_external_signer" = "yes"; then
1450+
AC_MSG_ERROR([External signing is not supported for this Boost version])
1451+
fi
1452+
use_external_signer="no";
1453+
fi
14421454
;;
14431455
esac
14441456
fi

0 commit comments

Comments
 (0)