19:01:14 <achow101> #startmeeting
19:01:14 <lightningbot> Meeting started Fri Jul 31 19:01:14 2020 UTC.  The chair is achow101. Information about MeetBot at http://wiki.debian.org/MeetBot.
19:01:14 <lightningbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
19:01:18 <provoostenator> hi
19:01:57 <achow101> #bitcoin-core-dev Wallet Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator aj Chris_Stewart_5 dongcarl gwillen jamesob ken281221 ryanofsky gleb moneyball ariard digi_james amiti fjahr
19:02:08 <provoostenator> Review nag for #15382 which is hopefully pretty close
19:02:10 <gribble> https://github.com/bitcoin/bitcoin/issues/15382 | util: add RunCommandParseJSON by Sjors · Pull Request #15382 · bitcoin/bitcoin · GitHub
19:02:24 <achow101> that's on my review queue
19:02:54 <achow101> the main sqlite wallet pr is now ready for review
19:03:13 <provoostenator> Nice, will move on to that after I'm done with a Taproot excursion
19:03:26 <achow101> I was also wondering whether we wanted to do #19619 before it though
19:03:28 <gribble> https://github.com/bitcoin/bitcoin/issues/19619 | Remove wallet.dat path handling from wallet.cpp, rpcwallet.cpp by ryanofsky · Pull Request #19619 · bitcoin/bitcoin · GitHub
19:05:13 <achow101> no topics today?
19:05:36 <provoostenator> Not much news from my end
19:06:12 <provoostenator> 0.20.1 has the PSBT change, that's all we needed there?
19:06:21 <achow101> yes
19:06:36 <provoostenator> sipa: when miniscript?
19:06:50 <achow101> also #19602 is legacy to descriptor wallet migration
19:06:52 <gribble> https://github.com/bitcoin/bitcoin/issues/19602 | wallet: Migrate legacy wallets to descriptor wallets by achow101 · Pull Request #19602 · bitcoin/bitcoin · GitHub
19:07:19 <provoostenator> Shouldn't that go in the wallet tool?
19:07:26 <provoostenator> At least until we really insist on an upgrade?
19:07:47 <achow101> it could
19:08:23 <achow101> i made it a separate RPC, but it could also be rolled into upgradewallet
19:08:28 <provoostenator> And even if we insist on an upgrade, only the GUI would need access to this code, client users can use wallettool
19:09:27 <achow101> sure
19:10:25 <achow101> however it is useful to not have to unload and reload a wallet in order to migrate it
19:12:31 <achow101> #endmeeting