Skip to content

Conversation

ryanofsky
Copy link
Contributor

This change adds an optional wallet=filename argument to RPCs that access the
wallet so they can operate on wallets other than the default wallet.

This does essentially the same thing as #10653, except it adds support for new wallet parameters explicitly in individual RPC methods instead of implicitly at the RPC dispatching layer.

Example usage:

bitcoind -regtest -wallet=w1.dat -wallet=w2.dat
bitcoin-cli -regtest -named getwalletinfo wallet=w1.dat
bitcoin-cli -regtest -named getwalletinfo wallet=w2.dat
bitcoin-cli -regtest -named getbalance wallet=w2.dat

@ryanofsky ryanofsky changed the title Add multiwallet support to wallet RPCs Add optional wallet=filename arguments to wallet RPCs Jun 23, 2017
@laanwj
Copy link
Member

laanwj commented Jun 24, 2017

Same comment as on #10653 (comment)****

This change adds an optional wallet=filename argument to RPCs that access the
wallet so they can operate on wallets other than the default wallet.

Example usage:

    bitcoind -regtest -wallet=w1.dat -wallet=w2.dat
    bitcoin-cli -regtest -named getwalletinfo wallet=w1.dat
    bitcoin-cli -regtest -named getwalletinfo wallet=w2.dat
    bitcoin-cli -regtest -named getbalance wallet=w2.dat
@ryanofsky
Copy link
Contributor Author

ryanofsky commented Jul 6, 2017

Closing in favor of #10650 (https://botbot.me/freenode/bitcoin-core-dev/msg/88240462/)

There's also a newer version of this PR which adds a test & makes it an error not to specify a wallet when more than one is loaded: master...ryanofsky:pr/multiopt

@ryanofsky ryanofsky closed this Jul 6, 2017
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants