doc: add clarifying comment that creating the .bitcoin file is to avoid any mainnet datadir access #33503

pull kevkevinpal wants to merge 1 commits into bitcoin:master from kevkevinpal:commentNoDataDirAccess changing 1 files +1 −0

kevkevinpal commented at 6:03 PM on September 29, 2025: contributor

referenced from #33486 (comment)

This adds a clarifying comment to tell the reader that the creation of the ${HOME}/.bitcoin file is to ensure that there is no mainnet datadir access happening from any of the tests
doc: add clarifying comment that creating the .bitcoin file is to avoid any mainnet datadir access d66bf8fbd5
DrahtBot added the label Docs on Sep 29, 2025
DrahtBot commented at 6:03 PM on September 29, 2025: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33503.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
in ci/test/03_test_script.sh:95 in d66bf8fbd5
```
  91 | @@ -92,6 +92,7 @@ if [ "$USE_BUSY_BOX" = "true" ]; then
  92 |  fi
  93 |  
  94 |  # Make sure default datadir does not exist and is never read by creating a dummy file
  95 | +# Ensuring that no mainnet datadir access is happening from any of the tests
```
laanwj commented at 1:29 PM on October 23, 2025:

This is not only about the mainnet datadir, but any default datadir. i'm not convinced that this comment adds any information beyond that in the previous line, which already states that.
kevkevinpal commented at 2:59 PM on October 23, 2025: contributor

closing due to lack of interest and may not add any substantial value
kevkevinpal closed this on Oct 23, 2025

Contributors

Labels

Linked (view graph)

#33486 ci: check if file or directory already exists for ${HOME}/.bitcoin instead of failing

doc: add clarifying comment that creating the .bitcoin file is to avoid any mainnet datadir access #33503

Code Coverage & Benchmarks

Reviews