Differential D11913

[avalanche] Make the payout script mandatory in the proof builder
ClosedPublic
Actions

Authored by Fabien on Sep 5 2022, 13:13.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCb1ca9766c52e: [avalanche] Make the payout script mandatory in the proof builder

Summary

This is a step toward deprecating the legacy proof format that will help reviewing the actual deprecation diff later.

Test Plan

ninja all check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Sep 5 2022, 13:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 5 2022, 13:13

Fabien requested review of this revision.Sep 5 2022, 13:13

Harbormaster completed remote builds in B19926: Diff 34815.Sep 5 2022, 13:30

Fix includes

Fabien added a child revision: D11914: [avalanche] Deprecate the legacy proof format.Sep 5 2022, 14:20

Harbormaster completed remote builds in B19927: Diff 34816.Sep 5 2022, 14:25

PiRK accepted this revision.Sep 6 2022, 08:11

This revision is now accepted and ready to land.Sep 6 2022, 08:11

Closed by commit rABCb1ca9766c52e: [avalanche] Make the payout script mandatory in the proof builder (authored by Fabien). · Explain WhySep 7 2022, 10:04

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCb1ca9766c52e: [avalanche] Make the payout script mandatory in the proof builder.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

proofbuilder.h

2 lines

test/

peermanager_tests.cpp

6 lines

6 lines

18 lines

8 lines

7 lines

3 lines

test/

serialize_tests.cpp

4 lines

Diff 34846

View Options

src/avalanche/proofbuilder.h

View Options

src/avalanche/test/peermanager_tests.cpp

View Options

src/avalanche/test/processor_tests.cpp

View Options

src/avalanche/test/proof_tests.cpp

View Options

src/avalanche/test/proofpool_tests.cpp

View Options

src/avalanche/test/util.h

View Options

src/avalanche/test/util.cpp

View Options

src/test/serialize_tests.cpp

[avalanche] Make the payout script mandatory in the proof builderClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 34846

src/avalanche/proofbuilder.h

src/avalanche/test/peermanager_tests.cpp

src/avalanche/test/processor_tests.cpp

src/avalanche/test/proof_tests.cpp

src/avalanche/test/proofpool_tests.cpp

src/avalanche/test/util.h

src/avalanche/test/util.cpp

src/test/serialize_tests.cpp

[avalanche] Make the payout script mandatory in the proof builder
ClosedPublic
Actions

Revision Contents
Changeset List