19:00:14 <meshcollider> #startmeeting
19:00:14 <lightningbot> Meeting started Fri Jan 11 19:00:14 2019 UTC.  The chair is meshcollider. Information about MeetBot at http://wiki.debian.org/MeetBot.
19:00:14 <lightningbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
19:00:17 <meshcollider> #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
19:00:30 <jnewbery> hi
19:00:30 <instagibbs> hi
19:00:34 <phantomcircuit> hi
19:00:45 <meshcollider> First wallet meeting of 2019
19:00:49 <meshcollider> Any topics?
19:00:54 <achow101> hi
19:01:15 <achow101> what's the status on #14491
19:01:17 <gribble> https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider · Pull Request #14491 · bitcoin/bitcoin · GitHub
19:01:47 <instagibbs> I've been mostly paying attention to HWI sorry :X
19:01:53 <achow101> same
19:02:37 <meshcollider> I've got to address a few comments there, and split up one of the commits
19:02:42 <jnewbery> I think might still be some rebase issues with 14491. I commented in the PR
19:03:24 <meshcollider> I'm travelling for the next few days unfortunately so I'm not sure how soon I'll have time
19:05:45 <jnewbery> I might have some time next week to try to rearrange the commits
19:06:31 <instagibbs> open Q I brought up in HWI: You cannot actually import two addresses if the wall already "understands the script" and then it complains about private keys. Is that an intentional limitation?
19:06:44 <instagibbs> say first one is native segwit
19:06:53 <instagibbs> second is p2sh wrapped, it doesn't let you import the second
19:07:06 <instagibbs> is this just a Core thingy we'll wait until "new wallet" to fix?
19:07:20 * instagibbs finding the exact test case I ran into and had to work around
19:08:07 <meshcollider> Is it a case of the wallet saying it already contains the script
19:08:45 <instagibbs> https://github.com/achow101/HWI/commit/68a4935c8803fa04801257ace34c6f07b03d57ac#diff-0d9a21f6363f36a267f2576c8d3d3863R232
19:08:58 <instagibbs> i reordered the keys to generate a new script here, but to test you can unscramble
19:09:02 <instagibbs> we can discuss offline
19:09:11 <achow101> i think it's the wallet saying it already has the script
19:09:52 <instagibbs> im sure that was the intended message
19:10:10 <meshcollider> Because of the "learn relayed scripts" hack
19:10:12 <meshcollider> Related*
19:10:26 <instagibbs> right, so might be "core problems" until revamp
19:10:50 <meshcollider> I can look into it further if you want, if you're not convinced its that
19:11:29 <instagibbs> if you think it's that it's probably that
19:11:45 <instagibbs> realistically you shouldn't be reusing same core script anyways
19:14:00 <instagibbs> other topics?
19:14:01 <meshcollider> But you're right the revamp will fix that
19:14:19 <instagibbs> indeed
19:16:20 <meshcollider> Looks like no other topics for now then
19:16:46 <meshcollider> #endmeeting