Skip to content

Commit 3e3f53c

Browse files
author
MarcoFalke
committed
Merge bitcoin/bitcoin#23328: ci: Disable syscall sandbox in valgrind functional tests
fa44406 ci: Disable syscall sandbox in valgrind functional tests (MarcoFalke) Pull request description: Otherwise this will fail: ``` $ valgrind ./src/bitcoind -regtest -datadir=/tmp -sandbox=log-and-abort ==204660== Memcheck, a memory error detector ==204660== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al. ==204660== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info ==204660== Command: ./src/bitcoind -regtest -datadir=/tmp -sandbox=log-and-abort ==204660== Bad system call (core dumped) ACKs for top commit: practicalswift: cr ACK fa44406 Tree-SHA512: 41853e6d5697d99bd5775a9b9017859290b3119e83726036d7e58856fcbb3459ef794b129bfaba6eca54b9bc034e2baf74cc2c177767ad5d5af3f3be2a45507f
2 parents c53e95f + fa44406 commit 3e3f53c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ci/test/00_setup_env_native_valgrind.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,6 @@ export CONTAINER_NAME=ci_native_valgrind
1111
export PACKAGES="valgrind clang llvm python3-zmq libevent-dev bsdmainutils libboost-dev libboost-system-dev libboost-filesystem-dev libboost-test-dev libdb5.3++-dev libminiupnpc-dev libnatpmp-dev libzmq3-dev libsqlite3-dev"
1212
export USE_VALGRIND=1
1313
export NO_DEPENDS=1
14-
export TEST_RUNNER_EXTRA="--exclude rpc_bind,feature_bind_extra" # Excluded for now, see https://github.com/bitcoin/bitcoin/issues/17765#issuecomment-602068547
14+
export TEST_RUNNER_EXTRA="--nosandbox --exclude rpc_bind,feature_bind_extra" # Excluded for now, see https://github.com/bitcoin/bitcoin/issues/17765#issuecomment-602068547
1515
export GOAL="install"
1616
export BITCOIN_CONFIG="--enable-zmq --with-incompatible-bdb --with-gui=no CC=clang CXX=clang++" # TODO enable GUI

0 commit comments

Comments
 (0)