< fanquake>
wumpus are you able to merge #7 on the univalue repo. The same change had ACKs from sipa and gmaxwell in #6 if your waiting for more reviewers.
< GAit>
wumpus: just noticed an error in the release notes for 0.15.0/0.15.0.1 not sure if already reported: "maximumCount - a number specifying the minimum number of UTXOs" - minimum vs maximum
< promag>
GAit: IIRC that was already detected
< instagibbs>
yes there's already a PR for a fix
< promag>
jnewbery: regarding #7061, at the moment the RPC can block a lot of time
< bitcoin-git>
[bitcoin] mess110 closed pull request #11455: CTxMemPool::GetMinFee should not return CFeeRate(0) (master...fix_mempool_GetMinFee_bug_returning_below_minRelayTxFee) https://github.com/bitcoin/bitcoin/pull/11455
< bitcoin-git>
[bitcoin] mess110 opened pull request #11475: [rpc] mempoolinfo should take ::minRelayTxFee into account (master...include_minRelayTxFee_in_getmempoolinfo_rpc) https://github.com/bitcoin/bitcoin/pull/11475
< jnewbery>
promag: I don't think that's a showstopper, but others may feel differently. Perhaps it can be addressed in a follow-up PR
< jonasschnelli>
promag: Yes. 7061 is blocking...Its not directly related to this PR. See #11281 for an possible improvement of the rescan locking
< meshcollider>
jnewbery: do you have time to discuss some of your points in #11466? Re: moving GetWalletDir() out of utils.cpp, is there already a wallet util file you'd suggest moving it too, or I should create one?
< meshcollider>
I still haven't fully thought through where to put the initial directory creation stuff, to move it out of the GetDataDir function and allow the path caching to be removed
< meshcollider>
So if I made a InitDataDir() function to call just once on initialization, which would check for datadir existence and make default one if it doesn't exist, that should store the data directory path and just return it on GetDataDir()
< meshcollider>
But we want to create the wallet dir on first run, so should a wallets/ directory always be created when the datadir is? Or is that still too much mixing of server + wallet stuff?
< esotericnonsense>
meshcollider: I haven't looked at the code; when/where is wallet.dat opened and written for the first time? that seems like a natural point?
< esotericnonsense>
(you don't need a wallets folder until you actually have something to put in it)
< esotericnonsense>
argh I missed a line above gribble's. ignore
< meshcollider>
esotericnonsense: the problem is that you only want to create a wallets/ folder if this is the first run, because otherwise you'll already have a wallet.dat in the datadir root which would get ignored from that point on
< meshcollider>
And from what I can think, the best way to know if this is a first-run is if we are creating the datadir too
< meshcollider>
Hmm perhaps it should go in wallet/init.cpp?
< meshcollider>
Also wanted to discuss how best to list `available` wallets without relying on .dat extension