Skip to content

Commit 657d1ab

Browse files
committed
CI: use more optimized builds sometimes.
1. Don't --enable-debugbuild for sanitizer runs (slow) or -O3 compilations. 2. Use -O3 build for splicing and postgres tests runs (speed them a little), make sure builds without --enable-debugbuild work. 3. Remove clang test run, because we already use that for minimum-btc-version test: this saves another 2 hour CI job. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
1 parent d44ebbd commit 657d1ab

1 file changed

Lines changed: 8 additions & 9 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -116,19 +116,23 @@ jobs:
116116
- CFG: compile-gcc
117117
VALGRIND: 1
118118
COMPILER: gcc
119+
DEBUG_BUILD: --enable-debugbuild
119120
- CFG: compile-gcc-O3
120121
VALGRIND: 1
121122
COMPILER: gcc
122123
COPTFLAGS_VAR: COPTFLAGS="-O3 -Werror"
124+
DEBUG_BUILD:
123125
# While we're at it let's try to compile with clang
124126
- CFG: compile-clang
125127
VALGRIND: 1
126128
COMPILER: clang
129+
DEBUG_BUILD: --enable-debugbuild
127130
- CFG: compile-clang-sanitizers
128131
COMPILER: clang
129132
ASAN: 1
130133
UBSAN: 1
131134
VALGRIND: 0
135+
DEBUG_BUILD:
132136
steps:
133137
- name: Checkout
134138
uses: actions/checkout@v4
@@ -156,7 +160,7 @@ jobs:
156160
COMPAT: 1
157161
run: |
158162
set -e
159-
./configure --enable-debugbuild CC="$COMPILER" ${{ matrix.COPTFLAGS_VAR }}
163+
./configure ${{ matrix.DEBUG_BUILD }} CC="$COMPILER" ${{ matrix.COPTFLAGS_VAR }}
160164
161165
uv run make -j $(nproc) testpack.tar.gz
162166
@@ -395,15 +399,10 @@ jobs:
395399
TEST_DB_PROVIDER: sqlite3
396400
COMPILER: gcc
397401
TEST_NETWORK: regtest
398-
# While we're at it let's try to compile with clang
399-
- NAME: clang
400-
CFG: compile-clang
401-
TEST_DB_PROVIDER: sqlite3
402-
COMPILER: clang
403-
TEST_NETWORK: regtest
402+
# We do a clang run, but with minimum BTC version, to avoid YA test run.
404403
# And of course we want to test postgres too
405404
- NAME: postgres
406-
CFG: compile-gcc
405+
CFG: compile-gcc-O3
407406
COMPILER: gcc
408407
TEST_DB_PROVIDER: postgres
409408
TEST_NETWORK: regtest
@@ -423,7 +422,7 @@ jobs:
423422
EXPERIMENTAL_DUAL_FUND: 1
424423
# And splicing!
425424
- NAME: splicing
426-
CFG: compile-gcc
425+
CFG: compile-gcc-O3
427426
TEST_DB_PROVIDER: sqlite3
428427
COMPILER: gcc
429428
TEST_NETWORK: regtest

0 commit comments

Comments
 (0)