aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorAva Chow <github@achow101.com>2024-10-01 13:30:02 -0400
committerAva Chow <github@achow101.com>2024-10-01 13:30:02 -0400
commit110183746150428e6385880c79f8c5733b1361ba (patch)
tree49ca493995cf8384ead6e92fb2747a72c1107130 /src
parent1147e72953d1f262111a4b1d5a438a8394511bc7 (diff)
parent5de225f5c145368f70cb5f870933bcf9df6b92c8 (diff)
Merge bitcoin/bitcoin#30959: [28.x] backports and finalizev28.0
5de225f5c145368f70cb5f870933bcf9df6b92c8 doc: 28.0 Release Notes (Ava Chow) 98745e03ffc9f69f901b827e19e4d8d645a27112 doc: generate manpages (Ava Chow) 5feef9ce7ebf608e45aa9682512599494c0d46d8 build: Bump to 28.0 (Ava Chow) 7fcd7b85c64ffbcd67d8ff1add46d258b26b2029 validation: Disable CheckForkWarningConditions for background chainstate (Martin Zumsande) e24a25d882ba8c8dacc1eede6b9373fe7601f85b test: Use shell builtins in run_command test case (Ava Chow) Pull request description: * #30952 * #30962 * Finalize 28.0 (or rc3 if additional backports are needed) ACKs for top commit: sipa: utACK 5de225f5c145368f70cb5f870933bcf9df6b92c8 Tree-SHA512: b42948a04d4250f2c9ef3331a39a4c3d7de9ceb9f4f294dd283599d08f3e2b7147297ef9ec1c4276e291a015fc2daa5a72c1f1c33fb517e8ea5c740c4459bf32
Diffstat (limited to 'src')
-rw-r--r--src/test/system_tests.cpp2
-rw-r--r--src/validation.cpp3
2 files changed, 3 insertions, 2 deletions
diff --git a/src/test/system_tests.cpp b/src/test/system_tests.cpp
index 07d15f5552..c9dd9c82cd 100644
--- a/src/test/system_tests.cpp
+++ b/src/test/system_tests.cpp
@@ -54,7 +54,7 @@ BOOST_AUTO_TEST_CASE(run_command)
}
{
// Return non-zero exit code, with error message for stderr
- const std::string command{"python3 -c 'import sys; print(\"err\", file=sys.stderr); sys.exit(2)'"};
+ const std::string command{"sh -c 'echo err 1>&2 && false'"};
const std::string expected{"err"};
BOOST_CHECK_EXCEPTION(RunCommandParseJSON(command), std::runtime_error, [&](const std::runtime_error& e) {
const std::string what(e.what());
diff --git a/src/validation.cpp b/src/validation.cpp
index a6b7f3d361..59beb5cbda 100644
--- a/src/validation.cpp
+++ b/src/validation.cpp
@@ -2023,7 +2023,8 @@ void Chainstate::CheckForkWarningConditions()
// Before we get past initial download, we cannot reliably alert about forks
// (we assume we don't get stuck on a fork before finishing our initial sync)
- if (m_chainman.IsInitialBlockDownload()) {
+ // Also not applicable to the background chainstate
+ if (m_chainman.IsInitialBlockDownload() || this->GetRole() == ChainstateRole::BACKGROUND) {
return;
}