Skip to content

Commit 8fa10ed

Browse files
committed
Merge bitcoin/bitcoin#31428: ci: Allow build dir on CI host
8888ee4 ci: Allow build dir on CI host (MarcoFalke) Pull request description: This is required to pass cross builds on to a different machine after the build. See for example bitcoin/bitcoin#31176, but this pull will also allow someone to implement it outside this repo. ACKs for top commit: davidgumberg: lgtm ACK bitcoin/bitcoin@8888ee4 hebasto: re-ACK 8888ee4. Tree-SHA512: a1e2c32bc1b95efbd0b48287ac5b49e0e1bacbf5a5800845be5352bbdd3e17fa478e90348b2e94e95cf3ae863cdf75ab444089376588f6f8eec438f73a4b5b97
2 parents 809d7e7 + 8888ee4 commit 8fa10ed

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

.github/workflows/ci.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -270,16 +270,16 @@ jobs:
270270
timeout-minutes: 120
271271
env:
272272
FILE_ENV: "./ci/test/00_setup_env_native_asan.sh"
273-
DANGER_CI_ON_HOST_CACHE_FOLDERS: 1
273+
DANGER_CI_ON_HOST_FOLDERS: 1
274274
steps:
275275
- name: Checkout
276276
uses: actions/checkout@v4
277277

278-
- name: Set Ccache directory
279-
run: echo "CCACHE_DIR=${RUNNER_TEMP}/ccache_dir" >> "$GITHUB_ENV"
280-
281-
- name: Set base root directory
282-
run: echo "BASE_ROOT_DIR=${RUNNER_TEMP}" >> "$GITHUB_ENV"
278+
- name: Set CI directories
279+
run: |
280+
echo "CCACHE_DIR=${{ runner.temp }}/ccache_dir" >> "$GITHUB_ENV"
281+
echo "BASE_ROOT_DIR=${{ runner.temp }}" >> "$GITHUB_ENV"
282+
echo "BASE_BUILD_DIR=${{ runner.temp }}/build-asan" >> "$GITHUB_ENV"
283283
284284
- name: Restore Ccache cache
285285
id: ccache-cache

ci/test/02_run_container.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,18 +78,21 @@ if [ -z "$DANGER_RUN_CI_ON_HOST" ]; then
7878
CI_DEPENDS_MOUNT="type=volume,src=${CONTAINER_NAME}_depends,dst=$DEPENDS_DIR/built"
7979
CI_DEPENDS_SOURCES_MOUNT="type=volume,src=${CONTAINER_NAME}_depends_sources,dst=$DEPENDS_DIR/sources"
8080
CI_PREVIOUS_RELEASES_MOUNT="type=volume,src=${CONTAINER_NAME}_previous_releases,dst=$PREVIOUS_RELEASES_DIR"
81+
CI_BUILD_MOUNT=""
8182

82-
if [ "$DANGER_CI_ON_HOST_CACHE_FOLDERS" ]; then
83+
if [ "$DANGER_CI_ON_HOST_FOLDERS" ]; then
8384
# ensure the directories exist
8485
mkdir -p "${CCACHE_DIR}"
8586
mkdir -p "${DEPENDS_DIR}/built"
8687
mkdir -p "${DEPENDS_DIR}/sources"
8788
mkdir -p "${PREVIOUS_RELEASES_DIR}"
89+
mkdir -p "${BASE_BUILD_DIR}" # Unset by default, must be defined externally
8890

8991
CI_CCACHE_MOUNT="type=bind,src=${CCACHE_DIR},dst=$CCACHE_DIR"
9092
CI_DEPENDS_MOUNT="type=bind,src=${DEPENDS_DIR}/built,dst=$DEPENDS_DIR/built"
9193
CI_DEPENDS_SOURCES_MOUNT="type=bind,src=${DEPENDS_DIR}/sources,dst=$DEPENDS_DIR/sources"
9294
CI_PREVIOUS_RELEASES_MOUNT="type=bind,src=${PREVIOUS_RELEASES_DIR},dst=$PREVIOUS_RELEASES_DIR"
95+
CI_BUILD_MOUNT="--mount type=bind,src=${BASE_BUILD_DIR},dst=${BASE_BUILD_DIR}"
9396
fi
9497

9598
if [ "$DANGER_CI_ON_HOST_CCACHE_FOLDER" ]; then
@@ -131,6 +134,7 @@ if [ -z "$DANGER_RUN_CI_ON_HOST" ]; then
131134
--mount "${CI_DEPENDS_MOUNT}" \
132135
--mount "${CI_DEPENDS_SOURCES_MOUNT}" \
133136
--mount "${CI_PREVIOUS_RELEASES_MOUNT}" \
137+
${CI_BUILD_MOUNT} \
134138
--env-file /tmp/env-$USER-$CONTAINER_NAME \
135139
--name "$CONTAINER_NAME" \
136140
--network ci-ip6net \

0 commit comments

Comments
 (0)