fix(0.11.1:17): require the current revision of each bitcoind line - #81
Merged
Conversation
helix-nine
force-pushed
the
fix/bitcoind-version-range-per-line
branch
from
August 3, 2026 17:22
a600c3c to
100ba80
Compare
Electrs' Auto-Configure task asserts `prune: 0`, which Bitcoin only accepts from the `:8` release of each line — 28.3:8, 29.3:8, 30.2:8, 31.0:8 — where archival stopped being an empty field and became an explicit `prune=0`. Before that, the Pruning field's minimum is hardcoded to 550 no matter how large the disk is. The declared range `>=28.4:17` — the revision that introduced the `peer-local` whitebind host electrs pulls blocks over — only constrained the 28 line: exver sorts 29.3:5 above 28.4:17, so every pre-`:8` build on 29/30/31 satisfied it, including ones with no `peer-local` host at all. On one of those the task opens a form seeded with 0 against a minimum of 550 — not submittable — and since that Bitcoin cannot record `prune=0` at all, `input-not-matches` re-raises the task forever (Start9Labs/fulcrum-startos#14). Each line is now floored at its current published revision, which is also where that line gained `peer-local` (29.4:4, 30.3:4, 31.1:4). Rides 0.11.1:17 rather than claiming :18 — that revision is only on community-beta, not the community-prod reference tier, so it is still unreleased. Its release notes carry both changes. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
helix-nine
force-pushed
the
fix/bitcoind-version-range-per-line
branch
from
August 3, 2026 17:28
100ba80 to
f22488c
Compare
MattDHill
approved these changes
Aug 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Electrs' Auto-Configure task asserts
prune: 0on Bitcoin. Only the:8release of each Bitcoin line accepts that —28.3:8,29.3:8,30.2:8,31.0:8— where archival stopped being an empty field and became an explicitprune=0. Before those, the Pruning field's minimum is hardcoded to 550 regardless of disk size.The declared
>=28.4:17— the revision that introduced thepeer-localwhitebind host electrs pulls blocks over — only constrained the 28 line. exver sorts29.3:5above28.4:17, so every pre-:8build on 29/30/31 satisfied it, including ones with nopeer-localhost at all. On one of those the task opens seeded with0against a minimum of550— not submittable — and because that Bitcoin cannot recordprune=0at all,input-not-matchesre-raises the task forever. Reset + Submit loops. Reported against Fulcrum, which had the identical gap: Start9Labs/fulcrum-startos#14.Each line is now floored at its current published revision, which is also where that line gained
peer-local(29.4:4,30.3:4,31.1:4). Verified against exver:28.4:14,29.3:12,29.4:1,30.2:12,30.3:1,31.0:14rejected;28.4:17+,29.4:4+,30.3:4+,31.1:4+pass. Knots is matched through its.satisfies('28.4:18')/.satisfies('29.4:5')claims, both of which clear the floors.Rebased onto master after #80 merged. This rides
0.11.1:17rather than claiming:18— that revision is only on community-beta, not the community-prod reference tier, so it is still unreleased. Its release notes now carry both changes.Test plan
28.4:17,29.4:4,30.3:4or31.1:4); run Auto-Configure and confirm Pruning accepts0, the form submits, and the task clears.peer-localhost and no restart loop under repeatedblockchain.scripthash.get_historyon an unsubscribed scripthash.29.3:5), confirm electrs reports the dependency as needing an update instead of an unsubmittable form.