Skip to content

Commit 6c261d2

Browse files
committed
ci: Use bash by default for all platforms
This change enforces fail-fast behavior for all platforms. See: https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#exit-codes-and-error-action-preference
1 parent 257fd27 commit 6c261d2

File tree

1 file changed

+18
-13
lines changed

1 file changed

+18
-13
lines changed

.github/workflows/ci.yml

+18-13
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,12 @@ env:
2121
CI_FAILFAST_TEST_LEAVE_DANGLING: 1 # GHA does not care about dangling processes and setting this variable avoids killing the CI script itself on error
2222
MAKEJOBS: '-j10'
2323

24+
defaults:
25+
run:
26+
# Enforce fail-fast behavior for all platforms.
27+
# See: https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#exit-codes-and-error-action-preference
28+
shell: bash
29+
2430
jobs:
2531
test-each-commit:
2632
name: 'test each commit'
@@ -185,19 +191,20 @@ jobs:
185191
arch: x64
186192

187193
- name: Get tool information
194+
shell: pwsh
188195
run: |
189196
cmake -version | Tee-Object -FilePath "cmake_version"
190197
Write-Output "---"
191198
msbuild -version | Tee-Object -FilePath "msbuild_version"
192199
$env:VCToolsVersion | Tee-Object -FilePath "toolset_version"
193200
py -3 --version
194201
Write-Host "PowerShell version $($PSVersionTable.PSVersion.ToString())"
202+
bash --version
195203
196204
- name: Using vcpkg with MSBuild
197205
run: |
198-
Set-Location "$env:VCPKG_INSTALLATION_ROOT"
199-
Add-Content -Path "triplets\x64-windows.cmake" -Value "set(VCPKG_BUILD_TYPE release)"
200-
Add-Content -Path "triplets\x64-windows-static.cmake" -Value "set(VCPKG_BUILD_TYPE release)"
206+
echo "set(VCPKG_BUILD_TYPE release)" >> "${VCPKG_INSTALLATION_ROOT}\triplets\x64-windows.cmake"
207+
echo "set(VCPKG_BUILD_TYPE release)" >> "${VCPKG_INSTALLATION_ROOT}\triplets\x64-windows-static.cmake"
201208
202209
- name: vcpkg tools cache
203210
uses: actions/cache@v4
@@ -214,7 +221,7 @@ jobs:
214221

215222
- name: Generate build system
216223
run: |
217-
cmake -B build --preset vs2022-static -DCMAKE_TOOLCHAIN_FILE="$env:VCPKG_INSTALLATION_ROOT\scripts\buildsystems\vcpkg.cmake" ${{ matrix.generate-options }}
224+
cmake -B build --preset vs2022-static -DCMAKE_TOOLCHAIN_FILE="${VCPKG_INSTALLATION_ROOT}\scripts\buildsystems\vcpkg.cmake" ${{ matrix.generate-options }}
218225
219226
- name: Save vcpkg binary cache
220227
uses: actions/cache/save@v4
@@ -226,13 +233,13 @@ jobs:
226233
- name: Build
227234
working-directory: build
228235
run: |
229-
cmake --build . -j $env:NUMBER_OF_PROCESSORS --config Release
236+
cmake --build . -j $NUMBER_OF_PROCESSORS --config Release
230237
231238
- name: Run test suite
232239
if: matrix.job-type == 'standard'
233240
working-directory: build
234241
run: |
235-
ctest --output-on-failure --stop-on-failure -j $env:NUMBER_OF_PROCESSORS -C Release
242+
ctest --output-on-failure --stop-on-failure -j $NUMBER_OF_PROCESSORS -C Release
236243
237244
- name: Run functional tests
238245
if: matrix.job-type == 'standard'
@@ -243,25 +250,23 @@ jobs:
243250
BITCOINUTIL: '${{ github.workspace }}\build\bin\Release\bitcoin-util.exe'
244251
BITCOINWALLET: '${{ github.workspace }}\build\bin\Release\bitcoin-wallet.exe'
245252
TEST_RUNNER_EXTRA: ${{ github.event_name != 'pull_request' && '--extended' || '' }}
246-
shell: cmd
247-
run: py -3 test\functional\test_runner.py --jobs %NUMBER_OF_PROCESSORS% --ci --quiet --tmpdirprefix=%RUNNER_TEMP% --combinedlogslen=99999999 --timeout-factor=%TEST_RUNNER_TIMEOUT_FACTOR% %TEST_RUNNER_EXTRA%
253+
run: py -3 test\functional\test_runner.py --jobs $NUMBER_OF_PROCESSORS --ci --quiet --tmpdirprefix="${RUNNER_TEMP}" --combinedlogslen=99999999 --timeout-factor=${TEST_RUNNER_TIMEOUT_FACTOR} ${TEST_RUNNER_EXTRA}
248254

249255
- name: Clone corpora
250256
if: matrix.job-type == 'fuzz'
251257
run: |
252-
git clone --depth=1 https://github.com/bitcoin-core/qa-assets "$env:RUNNER_TEMP\qa-assets"
253-
Set-Location "$env:RUNNER_TEMP\qa-assets"
254-
Write-Host "Using qa-assets repo from commit ..."
258+
git clone --depth=1 https://github.com/bitcoin-core/qa-assets "${RUNNER_TEMP}\qa-assets"
259+
cd "${RUNNER_TEMP}\qa-assets"
260+
echo "Using qa-assets repo from commit ..."
255261
git log -1
256262
257263
- name: Run fuzz tests
258264
if: matrix.job-type == 'fuzz'
259265
working-directory: build
260266
env:
261267
BITCOINFUZZ: '${{ github.workspace }}\build\bin\Release\fuzz.exe'
262-
shell: cmd
263268
run: |
264-
py -3 test\fuzz\test_runner.py --par %NUMBER_OF_PROCESSORS% --loglevel DEBUG %RUNNER_TEMP%\qa-assets\fuzz_corpora
269+
py -3 test\fuzz\test_runner.py --par $NUMBER_OF_PROCESSORS --loglevel DEBUG "${RUNNER_TEMP}\qa-assets\fuzz_corpora"
265270
266271
asan-lsan-ubsan-integer-no-depends-usdt:
267272
name: 'ASan + LSan + UBSan + integer, no depends, USDT'

0 commit comments

Comments
 (0)