Skip to content

backport: v0.27 bitcoin#28503, bitcoin#28037 - #116

Merged
DCG-Claude merged 2 commits into
developfrom
backport-0.27-b004-src-test
Nov 2, 2026
Merged

DCG-Claude merged 2 commits into
developfrom
backport-0.27-b004-src-test

Conversation

@DCG-Claude

@DCG-Claude DCG-Claude commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Automated Bitcoin Core v0.27 backports, batch backport-0.27-b004-src-test.

upstream commit gates notes
bitcoin#28503 af77fd9ff3 pick:pass build:pass tests:warn mech:warn tree:pass verify:pass Faithful backport of bitcoin#28503: the WithParams helper is removed and all callers use the SER_PARAMS_OPFUNC call synt
bitcoin#28037 12d05b6864 pick:pass build:pass tests:pass mech:pass tree:pass verify:pass clean cherry-pick
Provenance

Each commit passed: cherry-pick (adapted by an Opus lane only where conflicts existed), build, touched tests, a mechanical diff-of-diffs check (every upstream hunk landed; no added line without an upstream counterpart), and an independent Opus verification lane where anything was adapted. Gate rows and lane artifacts are in the backportsys DB.

@DCG-Claude DCG-Claude changed the title backport: v0.27 bitcoin#29357, bitcoin#29291, bitcoin#28503 backport: v0.27 bitcoin#28503 Oct 21, 2026
@DCG-Claude
DCG-Claude force-pushed the backport-0.27-b004-src-test branch from bb4f330 to 0946e87 Compare October 21, 2026 19:20
@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

Branch rewritten at 0946e87219 (1 backport(s)).


🤖 backportsys, on behalf of the Dash backport pipeline.

@DCG-Claude DCG-Claude changed the title backport: v0.27 bitcoin#28503 backport: v0.27 bitcoin#28503, bitcoin#29205 Oct 25, 2026
@DCG-Claude DCG-Claude changed the title backport: v0.27 bitcoin#28503, bitcoin#29205 backport: v0.27 bitcoin#28503, bitcoin#29205, bitcoin#28037 Oct 27, 2026
@DCG-Claude
DCG-Claude force-pushed the backport-0.27-b004-src-test branch from 83d416a to 2e3108f Compare October 31, 2026 22:25
@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

CI failed at 2e3108f on thepastaclaw/dash: Cppcheck / Run cppcheck

The Cppcheck job ran the fork's develop workflow, which calls test/lint/cppcheck-dash.py. Develop renamed it to that in d9e487e (dashpay#7966); this branch's base still has test/lint/lint-cppcheck-dash.py, so the job exited 127 with 'No such file or directory'. The macOS depends failure (gmp configure could not find a working compiler) is in depends/ci, which this branch does not touch; develop has also renamed the macOS CI setup script since the base, so it is probably the same mismatch and should clear after a rebase onto develop. This branch is based on a38dd560a9, older than develop (5df218409a); it is being rebased onto develop and this is looked at again there.


🤖 backportsys, on behalf of the Dash backport pipeline.

@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

CI failed at 2e3108f on thepastaclaw/dash: Require a milestone

This check fails only because the pull request has no milestone set. That is PR metadata a maintainer has to add, not anything in these backport commits.


🤖 backportsys, on behalf of the Dash backport pipeline.

…, use SER_PARAMS_OPFUNC

9999019 Remove WithParams serialization helper (MarcoFalke)
ffffb4a scripted-diff: Use ser params operator (MarcoFalke)
fae9054 test: Use SER_PARAMS_OPFUNC in serialize_tests.cpp (MarcoFalke)

Pull request description:

  Every serialization parameter struct already has the `SER_PARAMS_OPFUNC`, except for one in the tests.

  For consistency, and to remove verbose code, convert the test to `SER_PARAMS_OPFUNC`, and use it everywhere, then remove the `WithParams` helper.

ACKs for top commit:
  ajtowns:
    reACK 9999019
  TheCharlatan:
    Re-ACK 9999019

Tree-SHA512: be9cae4225a502486fe8d552aaf4b2cd2904a9f73cce9d931c6b7c757594ff1982fcc2c30d00d012cd12b0a9531fd609f8bcd7c94b811e965ac087eb8a3589d3

Dash adaptations:
- src/test/serialize_tests.cpp: upstream's BaseFormat struct and RAW/HEX constants follow Dash-only UnserializeVectorWithMaxSize/LimitedVectorFormatter tests, which are kept
- src/evo/deterministicmns.h: Dash-only GetUniquePropertyHash() now uses ser_params(v) instead of the removed WithParams(ser_params, v)
f1684bb rpc: mention that migratewallet can take a while (Andrew Chow)
9ecff99 rpc: Drop migratewallet experimental warning (Andrew Chow)

Pull request description:

  The migration process itself hasn't fundamentally changed since it was added, so I think it's reasonable to say that it is no longer experimental.

ACKs for top commit:
  maflcko:
    lgtm ACK f1684bb
  josibake:
    ACK bitcoin@f1684bb
  furszy:
    ACK bitcoin@f1684bb
  ryanofsky:
    Code review ACK f1684bb
  willcl-ark:
    ACK f1684bb

Tree-SHA512: 99b176cddbf3878c76bd4c80c030106200bf03139785e26dbae3341e1a675b623a13cd6dc7a0bb78344335bf859ae7548d97b2b58eb650c6e7b305d7cdc86e40
@DCG-Claude DCG-Claude changed the title backport: v0.27 bitcoin#28503, bitcoin#29205, bitcoin#28037 backport: v0.27 bitcoin#28503, bitcoin#28037 Nov 1, 2026
@DCG-Claude
DCG-Claude force-pushed the backport-0.27-b004-src-test branch from 2e3108f to 12d05b6 Compare November 1, 2026 01:33
@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

CI failed at 12d05b6 on thepastaclaw/dash: Require a milestone

This job fails because the pull request has no milestone set (MILESTONE is empty in the log); it doesn't build or test anything. Neither backport (bitcoin#28503, bitcoin#28037) can cause or fix it. A maintainer needs to set a milestone on the pull request, and then the check will pass.


🤖 backportsys, on behalf of the Dash backport pipeline.

@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

CI failed at 12d05b6 on thepastaclaw/dash: Require a milestone

This job only checks that the pull request has a milestone, and the log shows MILESTONE was empty. It doesn't depend on any code in the bitcoin#28503 or bitcoin#28037 backports, so the branch needs no change. Setting a milestone on the PR should make it pass.


🤖 backportsys, on behalf of the Dash backport pipeline.

@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

CI failed at 12d05b6 on thepastaclaw/dash: Require a milestone

This job failed because the 'Require a milestone' check found no milestone set on the pull request (MILESTONE is empty). It checks PR metadata, not code, and neither bitcoin#28503 nor bitcoin#28037 touches it. A maintainer needs to set a milestone on the PR, and then the check should pass.


🤖 backportsys, on behalf of the Dash backport pipeline.

@DCG-Claude
DCG-Claude merged commit fd32d77 into develop Nov 2, 2026
82 of 88 checks passed
@DCG-Claude

Copy link
Copy Markdown
Collaborator Author

merged as dashpay#8006

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.

1 participant