Skip to content

Bump rust-bitcoin minimums to the 0.32.10x line - #1879

Merged
benalleng merged 1 commit into
payjoin:masterfrom
caarloshenriq:deps/bump-bitcoin-minimums
Sep 14, 2026
Merged

benalleng merged 1 commit into
payjoin:masterfrom
caarloshenriq:deps/bump-bitcoin-minimums

Conversation

@caarloshenriq

Copy link
Copy Markdown
Contributor

Closes #1818 (option b: adopt the 0.32.10x line as the floor, rather than pinning away from it).

Change

  • bitcoin: 0.32.9 → 0.32.102
  • bitcoin-units: 0.1.3 → 0.1.101
  • Cargo-minimal.lock and Cargo-recent.lock regenerated via
    contrib/update-lock-files.sh

Rationale

rust-bitcoin publishes two lines under the 0.32 major, and cargo compares patch numbers numerically, so an open ceiling always resolves to the higher one regardless of which line is declared. The workspace's minimum sat at 0.32.9, which meant Cargo-minimal.lock tested a different line than Cargo-recent.lock and every downstream consumer of published payjoin 1.0.0, that crate declares bitcoin ^0.32.9 on crates.io with no upper bound, so cargo add payjoin today already resolves 0.32.102 regardless of what this repo's CI pins.

Pinning down to stable-only would only create a CI/reality mismatch, not protect anyone downstream. MSRV is not a blocker: 0.32.102 and 0.1.101 both declare rust-version 1.60.0, well under this workspace's 1.85 floor. Accepted risk: 0.32.100 was yanked before 101/102 shipped, so this line has had at least one bad release, worth keeping an eye on stability here going forward.

Pull Request Checklist

Please confirm the following before requesting review:

rust-bitcoin publishes two lines under the 0.32 major: the stable
line (0.32.11) and a 0.32.1xx line (0.32.102). Cargo compares patch
versions numerically, so 102 outranks 11, and an open ceiling
resolves to the 1xx line regardless of which one the manifest
names.

The declared minimum drives Cargo-minimal.lock and the open
ceiling drives Cargo-recent.lock, so the workspace already
exercised both lines, one per leg of contrib/test.sh, without
anyone deciding that on purpose. The minimum sat at 0.32.9, two
lines behind what Cargo-recent.lock and every downstream consumer
of the published payjoin crate already resolve.

Raise the minimum to 0.32.102 so both legs converge on the same
line instead of silently diverging. bitcoin-units moves to 0.1.101
for the same reason: bitcoin 0.32.102 requires bitcoin-units
^0.1.101, so the floor has to move together with it.
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 34867980010

Coverage remained the same at 86.646%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 16549
Covered Lines: 14339
Line Coverage: 86.65%
Coverage Strength: 341.5 hits per line

💛 - Coveralls

@benalleng benalleng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 845651a

This seems like an appropriate bump just to keep our minimal-lock up-to-date with critical bitcoin stuff

@benalleng
benalleng merged commit 1f35dd2 into payjoin:master Sep 14, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pin or bump the rust-bitcoin constraint: ^0.32.9 currently accepts the 0.33 pre-release prep

3 participants