Page MenuHomePhabricator
Differential D5261

Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi wallet.
ClosedPublic
Actions

Authored by markblundeberg on Feb 11 2020, 13:38.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rSTAGING41fde531cfe5: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi…
rABC41fde531cfe5: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi…
Summary

PR10816 backport ; fixes misbackport in D1061

Test Plan

ninja check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markblundeberg created this revision.Feb 11 2020, 13:38
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 11 2020, 13:38
teamcity edited the summary of this revision. (Show Details)Feb 11 2020, 13:38

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those Bitcoin Core PRs have been inserted into the summary for reference.

teamcity added a comment.Feb 11 2020, 13:38

Build Bitcoin-ABC / Diffs / Diff Testing started.

markblundeberg added a child revision: D5262: Merge #13030: [bugfix] [wallet] Fix zapwallettxes/multiwallet interaction..Feb 11 2020, 13:45
teamcity added a comment.Feb 11 2020, 13:48

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B9434: Diff 16278.Feb 11 2020, 13:48
Fabien accepted this revision.Feb 11 2020, 13:50
This revision is now accepted and ready to land.Feb 11 2020, 13:50
markblundeberg updated this revision to Diff 16284.Feb 11 2020, 14:24

minor text fix (will not change result after child)

teamcity added a comment.Feb 11 2020, 14:46

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Feb 11 2020, 14:55

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B9436: Diff 16284.Feb 11 2020, 14:55
Closed by commit rABC41fde531cfe5: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi… (authored by Pieter Wuille <pieter.wuille@gmail.com>, committed by Mark Lundeberg <markblundeberg@users.noreply.github.com>). · Explain WhyFeb 11 2020, 15:00
This revision was automatically updated to reflect the committed changes.
Mark Lundeberg <markblundeberg@users.noreply.github.com> added a commit: rABC41fde531cfe5: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi….
Mark Lundeberg <markblundeberg@users.noreply.github.com> added a commit: rSTAGING41fde531cfe5: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi….Apr 7 2020, 22:58

Revision Contents
Changeset List

PathSize
src/
wallet/
20 lines

Diff 16289

View Options

src/wallet/init.cpp

Loading...