ci: Put space and non-ASCII char in `BASE_BUILD_DIR`
What changed, and why it matters
This is a harmless CI (continuous integration) configuration change for Bitcoin Core. It adds a space and a non-ASCII character to a build directory path used only in automated testing, so the test environment exercises how the code handles unusual directory names. There is no security issue here.
No action needed. This is a benign CI testing improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies .github/actions/configure-environment/action.yml to set BASE_BUILD_DIR to a path containing a space and non-ASCII characters (₿🧪). The stated purpose is to restore prior test coverage for word-splitting and UTF-8 path handling that was lost when GitHub Actions workflows began overriding BASE_BUILD_DIR. The change is purely a testing hardening measure and does not alter shipped code, introduce vulnerabilities, or affect production Bitcoin nodes.
Changed components
.github/actions/configure-environment/action.ymlInspect captured patch +3 / −1
diff --git a/.github/actions/configure-environment/action.yml b/.github/actions/configure-environment/action.yml
index e2a26b71..d3c928cf 100644
--- a/.github/actions/configure-environment/action.yml
+++ b/.github/actions/configure-environment/action.yml
@@ -7,7 +7,9 @@ runs:
shell: bash
run: |
echo "BASE_ROOT_DIR=${{ runner.temp }}" >> "$GITHUB_ENV"
- echo "BASE_BUILD_DIR=${{ runner.temp }}/build" >> "$GITHUB_ENV"
+ # Space and non-ASCII symbols mimic BASE_SCRATCH_DIR in ci/test/00_setup_env.sh,
+ # to test word-splitting and UTF-8 path handling on CI.
+ echo "BASE_BUILD_DIR=${{ runner.temp }}/build_ ₿🧪_" >> "$GITHUB_ENV"
echo "CCACHE_DIR=${{ runner.temp }}/ccache_dir" >> $GITHUB_ENV
echo "DEPENDS_DIR=${{ runner.temp }}/depends" >> "$GITHUB_ENV"
echo "BASE_CACHE=${{ runner.temp }}/depends/built" >> $GITHUB_ENV
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.