Skip to content

Commit 8671b0b

Browse files
authored
Merge pull request #126 from frstrtr/ci-steward/retire-btc-linux
ci: retire dormant btc-linux job from build.yml
2 parents 7035158 + 7274023 commit 8671b0b

1 file changed

Lines changed: 11 additions & 68 deletions

File tree

.github/workflows/build.yml

Lines changed: 11 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -345,76 +345,19 @@ jobs:
345345
timeout 180 ./build-qt/c2pool-qt_test_embedded_leak
346346
347347
# ════════════════════════════════════════════════════════════════════════════
348-
# BTC embedded smoke (c2pool-btc target)
348+
# BTC embedded smoke — RETIRED 2026-06-17 (ci-steward/retire-btc-linux).
349+
# Superseded by the coin-matrix.yml `btc` cell ("btc smoke (Linux x86_64)",
350+
# a required check on master). The old btc-linux job was gated solely to
351+
# btc-embedded branch involvement and never fired on master, so removing it
352+
# changes no live coverage.
349353
#
350-
# Gated to runs where the btc-embedded branch is involved — pushes to
351-
# btc-embedded, or PRs whose head or base is btc-embedded. On master
352-
# itself the BTC sources don't yet exist (src/c2pool/main_btc.cpp,
353-
# src/impl/btc/) so this job is intentionally a no-op there; it
354-
# activates as soon as this workflow lands on btc-embedded via merge.
355-
#
356-
# Coverage: cmake --build --target c2pool-btc catches any link/compile
357-
# breakage in the embedded BTC SPV binary (main_btc.cpp + btc module +
358-
# btc_stratum + transitive ltc dependency). Until this CI job existed,
359-
# BTC had ZERO automated coverage on any commit.
360-
#
361-
# Deferred to a follow-up PR landing on btc-embedded directly:
362-
# - src/impl/btc/test/share_test.cpp: `add_subdirectory(test)` is
363-
# commented out in src/impl/btc/CMakeLists.txt because the test
364-
# still references LTC types. Revival is owned by the btc-heap-opt
365-
# arc.
366-
# - src/impl/btc/test/template_parity_test.cpp: standalone consensus
367-
# oracle harness; the file header claims it builds with a bare g++
368-
# invocation but its transitive includes pull in nlohmann_json,
369-
# leveldb_store, core/log — needs proper CMake target wiring to be
370-
# reliable in CI.
354+
# Deferred btc test wiring (unchanged, owned by the btc-heap-opt arc):
355+
# - src/impl/btc/test/share_test.cpp — add_subdirectory(test) disabled in
356+
# src/impl/btc/CMakeLists.txt (still references LTC types).
357+
# - src/impl/btc/test/template_parity_test.cpp — standalone consensus
358+
# oracle harness; needs proper CMake target wiring (nlohmann_json,
359+
# leveldb_store, core/log) to be CI-reliable.
371360
# ════════════════════════════════════════════════════════════════════════════
372-
btc-linux:
373-
name: BTC embedded smoke (Linux x86_64)
374-
if: |
375-
github.ref == 'refs/heads/btc-embedded' ||
376-
github.head_ref == 'btc-embedded' ||
377-
github.base_ref == 'btc-embedded'
378-
runs-on: ubuntu-24.04
379-
steps:
380-
- uses: actions/checkout@v6
381-
382-
- name: Install system dependencies
383-
run: |
384-
sudo apt-get update -qq
385-
sudo apt-get install -y --no-install-recommends \
386-
g++ cmake make libleveldb-dev libsecp256k1-dev
387-
388-
- uses: actions/setup-python@v6
389-
with: { python-version: '3.12' }
390-
391-
- name: Install Conan 2
392-
run: pip install "conan>=2.0,<3.0"
393-
394-
- name: Detect Conan profile
395-
run: |
396-
conan profile detect --force
397-
sed -i 's/compiler.cppstd=.*/compiler.cppstd=20/' "$(conan profile path default)"
398-
399-
- name: Restore Conan cache
400-
uses: actions/cache@v5
401-
with:
402-
path: ~/.conan2
403-
key: conan2-ubuntu24-gcc13-${{ hashFiles('conanfile.txt') }}
404-
405-
- name: Conan install
406-
run: conan install . --build=missing --output-folder=build_btc --settings=build_type=Release
407-
408-
- name: Configure
409-
run: cmake -S . -B build_btc -DCMAKE_TOOLCHAIN_FILE=build_btc/conan_toolchain.cmake -DCMAKE_BUILD_TYPE=Release
410-
411-
- name: Build c2pool-btc
412-
run: cmake --build build_btc --target c2pool-btc -j$(nproc)
413-
414-
- name: Verify binary
415-
run: |
416-
file build_btc/src/c2pool/c2pool-btc
417-
build_btc/src/c2pool/c2pool-btc --help 2>&1 | head -3 || true
418361

419362
# ════════════════════════════════════════════════════════════════════════════
420363
# macOS (matrix: arm64 on GitHub-hosted macos-14, x86_64 on the self-hosted

0 commit comments

Comments
 (0)