In D6073#148948, @majcosta wrote:Most of the changes in the original backport were already done in D1106, except for a few changes related to PartiallySignedTransactions, which we already moved so that is why the files changed are different. The line change in wallet/rpcwallet.cpp also just brings up to parity with Core.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
May 14 2020
May 14 2020
nakihito accepted D6073: [backport#14906] refactor: Make explicit CMutableTransaction -> CTransaction conversion..
nakihito updated the summary of D6061: Merge #9332: Let wallet importmulti RPC accept labels for standard scriptPubKeys.
nakihito added inline comments to D6061: Merge #9332: Let wallet importmulti RPC accept labels for standard scriptPubKeys.
nakihito committed rSTAGING76d513094857: Remove Core release note file, update previous release notes, and fix generate… (authored by nakihito).
Remove Core release note file, update previous release notes, and fix generate…
nakihito committed rABC76d513094857: Remove Core release note file, update previous release notes, and fix generate… (authored by nakihito).
Remove Core release note file, update previous release notes, and fix generate…
nakihito requested changes to D6073: [backport#14906] refactor: Make explicit CMutableTransaction -> CTransaction conversion..
Backport doesn't match the original PR.
diff looks correct.
nakihito updated the diff for D6074: Remove Core release note file, update previous release notes, and fix generate() deprecation message.
Actually remove the Core release note file.
nakihito updated the diff for D6074: Remove Core release note file, update previous release notes, and fix generate() deprecation message.
Removed note from current release notes.
nakihito committed rSTAGING3e4f45f38e83: Fix string layout in rpc/rawtransaction.cpp (authored by nakihito).
Fix string layout in rpc/rawtransaction.cpp
nakihito committed rABC3e4f45f38e83: Fix string layout in rpc/rawtransaction.cpp (authored by nakihito).
Fix string layout in rpc/rawtransaction.cpp
May 13 2020
May 13 2020
nakihito requested review of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito updated the summary of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito planned changes to D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito updated the diff for D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
Moved PR9332 to its own patch (D6061) and rebased off it.
nakihito retitled D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness) from Merge #9332 and #14454: Let wallet importmulti RPC accept labels for standard scriptPubKeys and ProcessImport() cleanup to Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito planned changes to D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
In D6051#148328, @deadalnix wrote:I started reviewing this but this is incredibly difficult. If the problem backport PR9332 is that some test is broken, isn't it possible to modify the PR slightly so that it doesn't break the test?
May 12 2020
May 12 2020
nakihito updated the summary of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito updated the summary of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito updated the summary of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito requested review of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito planned changes to D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito requested review of D6051: Partial Merge #14454: ProcessImport() cleanup (excluding witness).
nakihito committed rSTAGING46afc3475dab: Merge #14150: Add key origin support to descriptors (authored by Pieter Wuille <pieter.wuille@gmail.com>).
Merge #14150: Add key origin support to descriptors
nakihito committed rABC46afc3475dab: Merge #14150: Add key origin support to descriptors (authored by Pieter Wuille <pieter.wuille@gmail.com>).
Merge #14150: Add key origin support to descriptors
May 11 2020
May 11 2020
Fixed formatting.
May 8 2020
May 8 2020
Reformatted comment.
May 7 2020
May 7 2020
The removed return values are duplicates of hdseedid. It even says in the help text that its an alias for hdseedid. The rest of the PR does not actually replace this value with an alternative because the alternative is already there. See relevant changes:
src/wallet/rpcwallet.cpp
https://github.com/bitcoin/bitcoin/pull/14021/files?file-filters%5B%5D=.cpp#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceL2421
https://github.com/bitcoin/bitcoin/pull/14021/files?file-filters%5B%5D=.cpp#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceL2459
https://github.com/bitcoin/bitcoin/pull/14021/files?file-filters%5B%5D=.cpp#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceL3663
https://github.com/bitcoin/bitcoin/pull/14021/files?file-filters%5B%5D=.cpp#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceL3726-L3729
nakihito requested changes to D6000: [backport#15638] [build] Move wallet load functions to wallet/load unit.
Small nit and test plan should include autotools build:
../configure -enable-deprecated-build-system make check
nakihito added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.
In D5956#146767, @deadalnix wrote:It seems like there is already a machinery to select the folder in which business happen, why does this needs to be all duplicated with a different name?
nakihito updated the diff for D5956: Add -datadir option to allow specification of seeder's data directory.
Error outputs to std::cerr and changed GetSeederDir() to call GetDefaultDataDir() directly when -seederdir is not set.
nakihito updated the test plan for D5956: Add -datadir option to allow specification of seeder's data directory.
May 6 2020
May 6 2020
May 5 2020
May 5 2020
nakihito updated the test plan for D5956: Add -datadir option to allow specification of seeder's data directory.
nakihito updated the diff for D5956: Add -datadir option to allow specification of seeder's data directory.
Moved seeder specific functions out of regular node code.
nakihito updated the test plan for D5956: Add -datadir option to allow specification of seeder's data directory.
nakihito committed rSTAGINGe55251f4329b: Finish PR14987 and clean up some functions (authored by nakihito).
Finish PR14987 and clean up some functions
nakihito committed rSTAGINGfa91619ce8b3: Pass some of wallet/rpcwallet RPC results and examples to RPCHelpMan (authored by nakihito).
Pass some of wallet/rpcwallet RPC results and examples to RPCHelpMan
nakihito committed rSTAGING70ba2634e165: Finish passing rpcwallet RPCs Results and Examples to RPCHelpMan (authored by nakihito).
Finish passing rpcwallet RPCs Results and Examples to RPCHelpMan
nakihito committed rSTAGINGa65bffe62fab: Pass some more (3/4) rpcwallet RPCs Results and Examples to RPCHelpMan (authored by nakihito).
Pass some more (3/4) rpcwallet RPCs Results and Examples to RPCHelpMan
nakihito committed rSTAGING50beb5f36aea: Pass more (2/4) rpcwallet RPCs Results and Examples to RPCHelpMan (authored by nakihito).
Pass more (2/4) rpcwallet RPCs Results and Examples to RPCHelpMan
nakihito committed rSTAGING69f5edb69071: Pass pruneblockchain RPC Results and Examples to RPCHelpMan (authored by nakihito).
Pass pruneblockchain RPC Results and Examples to RPCHelpMan
nakihito committed rSTAGINGdf9019c00825: Pass rpc/avalanche Results and Examples to RPCHelpMan (authored by nakihito).
Pass rpc/avalanche Results and Examples to RPCHelpMan
nakihito committed rSTAGING499a48084018: Pass rpc/misc RPC Results and Exmaples to RPCHelpMan (authored by nakihito).
Pass rpc/misc RPC Results and Exmaples to RPCHelpMan
nakihito committed rSTAGINGc03c766cf334: Finish Passing rpc/blockchain RPC results and examples to RPCHelpMan (authored by nakihito).
Finish Passing rpc/blockchain RPC results and examples to RPCHelpMan
nakihito committed rSTAGING4bc2f9217b1e: Pass some rpc/blockchain RPC results and examples to RPCHelpMan (authored by nakihito).
Pass some rpc/blockchain RPC results and examples to RPCHelpMan
nakihito committed rSTAGINGc006172f5b02: Finish Passing rpc/rawtransaction RPC results and examples to RPCHelpMan (authored by nakihito).
Finish Passing rpc/rawtransaction RPC results and examples to RPCHelpMan
nakihito committed rSTAGINGf2d4686b9352: Pass wallet/rpcdump RPC results and examples to RPCHelpMan (authored by nakihito).
Pass wallet/rpcdump RPC results and examples to RPCHelpMan
nakihito committed rSTAGING22f5cdd265bc: Pass rpc/rawtransaction RPC results and examples to RPCHelpMan (authored by nakihito).
Pass rpc/rawtransaction RPC results and examples to RPCHelpMan
nakihito committed rSTAGING8cc917edc1d8: Pass rpc/net RPC Results and Examples to RPCHelpMan (authored by nakihito).
Pass rpc/net RPC Results and Examples to RPCHelpMan
nakihito committed rSTAGING9f1d09d7937d: Pass rpc/mining RPC results and examples to RPCHelpMan (authored by nakihito).
Pass rpc/mining RPC results and examples to RPCHelpMan
Rename IsGood() to IsReliable()
nakihito committed rSTAGING6cc421370619: Pass zmq RPC results and examples to RPCHelpMan (authored by nakihito).
Pass zmq RPC results and examples to RPCHelpMan
nakihito committed rSTAGINGae8d0e0ef626: Pass rpc/server RPC results and examples to RPCHelpMan (authored by nakihito).
Pass rpc/server RPC results and examples to RPCHelpMan
nakihito committed rSTAGINGe6cc692608b8: Pass rpc/abc RPC Results and Examples to RPCHelpMan (authored by nakihito).
Pass rpc/abc RPC Results and Examples to RPCHelpMan
Remove TestNode()
nakihito committed rSTAGINGf7690c0f382b: Merge #17192: util: Add CHECK_NONFATAL and use it in src/rpc (authored by Wladimir J. van der Laan <laanwj@protonmail.com>).
Merge #17192: util: Add CHECK_NONFATAL and use it in src/rpc
nakihito committed rSTAGING43d44b2e899b: Merge #14885: rpc: Assert named arguments are unique in RPCHelpMan (authored by MarcoFalke <falke.marco@gmail.com>).
Merge #14885: rpc: Assert named arguments are unique in RPCHelpMan
nakihito committed rSTAGINGc7f7fe772158: Merge #15943: tests: Fail if RPC has been added without tests (authored by MarcoFalke <falke.marco@gmail.com>).
Merge #15943: tests: Fail if RPC has been added without tests
nakihito committed rSTAGING02ab89366a5a: Complete PR14796 by cleaning up some old functions and names (authored by nakihito).
Complete PR14796 by cleaning up some old functions and names
nakihito committed rSTAGINGa0950966fd85: Finish passing the remainder of wallet/rpcwallet RPC argument descriptions to… (authored by nakihito).
Finish passing the remainder of wallet/rpcwallet RPC argument descriptions to…
nakihito committed rSTAGING245bf8cc7f0a: Start passing some wallet/rpcwallet RPC argument descriptions to RPCHelpMan (authored by nakihito).
Start passing some wallet/rpcwallet RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGING28c8d7fa85b5: Pass rpc/mining RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/mining RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGINGb8e343ee301f: Pass wallet/rpcdump RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass wallet/rpcdump RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGINGc69d41b2bc44: Pass rpc/rawtransaction RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/rawtransaction RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGING5850294bc6e7: Pass rpc/misc RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/misc RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGINGb17d1db1baaf: Pass rpc/blockchain RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/blockchain RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGING3413afd1ce58: Pass rpc/net RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/net RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGINGf7eb5edff8aa: Pass rpc/server RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/server RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGING969731122b25: Pass rpc/avalanche RPC argument descriptions to RPCHelpMan (authored by nakihito).
Pass rpc/avalanche RPC argument descriptions to RPCHelpMan
nakihito committed rSTAGING28b437a49d59: Make use of ADDR_SOFT_CAP outside just the seeder test suite (authored by nakihito).
Make use of ADDR_SOFT_CAP outside just the seeder test suite