{
  "founder": "meshcollider",
  "channel": "#bitcoin-core-dev",
  "network": "freenode",
  "id": "85a557c37cbd40ec8930e39d923bf7e0",
  "name": "#bitcoin-core-dev",
  "chair": "meshcollider",
  "chairs": [
    "meshcollider"
  ],
  "nicks": {
    "meshcollider": 30,
    "lightningbot": 2,
    "sipa": 26,
    "jnewbery": 8,
    "kanzure": 1,
    "instagibbs": 2,
    "provoostenator": 42,
    "gribble": 8,
    "achow101": 1,
    "bitcoin-git": 9,
    "phantomcircuit": 3,
    "luke-jr": 2,
    "MarcoFalke": 2
  },
  "start_time": "2018-12-14T19:01:42+00:00",
  "end_time": "2018-12-14T19:41:51+00:00",
  "active": false,
  "original_topic": "Bitcoin Core development discussion and commit log | This is the channel for developing Bitcoin Core. Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Meeting topics https://gist.github.com/moneyball/071d608fdae217c2a6d7c35955881d8a",
  "current_topic": "progress towards hardware wallets (provoostenator)",
  "messages": [
    {
      "id": "56143019f823418ea312f3be748015ed",
      "sender": "meshcollider",
      "payload": "#startmeeting",
      "action": false,
      "timestamp": "2018-12-14T19:01:42+00:00"
    },
    {
      "id": "d49b284829ba4254a4af35ea20b476ac",
      "sender": "lightningbot",
      "payload": "Meeting started Fri Dec 14 19:01:42 2018 UTC.  The chair is meshcollider. Information about MeetBot at http://wiki.debian.org/MeetBot.",
      "action": false,
      "timestamp": "2018-12-14T19:01:42+00:00"
    },
    {
      "id": "a5df2a55914247e7b4e5778eb5a63ce2",
      "sender": "lightningbot",
      "payload": "Useful Commands: #action #agreed #help #info #idea #link #topic.",
      "action": false,
      "timestamp": "2018-12-14T19:01:42+00:00"
    },
    {
      "id": "2c389bfa6489451a844a2ea58e609994",
      "sender": "sipa",
      "payload": "gwillen: i always assumed that it did",
      "action": false,
      "timestamp": "2018-12-14T19:01:49+00:00"
    },
    {
      "id": "4a37664ced7143f4a2d45d61b9879adb",
      "sender": "jnewbery",
      "payload": "feowertyne niht",
      "action": false,
      "timestamp": "2018-12-14T19:02:02+00:00"
    },
    {
      "id": "b992bc0bda5c499e84ee786add0b784f",
      "sender": "meshcollider",
      "payload": "#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",
      "action": false,
      "timestamp": "2018-12-14T19:02:04+00:00"
    },
    {
      "id": "2a50da0705344dab8551a174ecfcf706",
      "sender": "meshcollider",
      "payload": "Ok topics?",
      "action": false,
      "timestamp": "2018-12-14T19:02:29+00:00"
    },
    {
      "id": "bcd7d3000e9249c6b732a306e73f1d3d",
      "sender": "kanzure",
      "payload": "hi.",
      "action": false,
      "timestamp": "2018-12-14T19:02:49+00:00"
    },
    {
      "id": "e1b0cbc6d7f741359d33f5376e6ee1ac",
      "sender": "jnewbery",
      "payload": "high priority for review?",
      "action": false,
      "timestamp": "2018-12-14T19:02:58+00:00"
    },
    {
      "id": "63af2ce67c6846edb8197209ce1e51c2",
      "sender": "sipa",
      "payload": "yay, some stuff got merged",
      "action": false,
      "timestamp": "2018-12-14T19:02:59+00:00"
    },
    {
      "id": "9f0298dad36448d2801f99fc3b42c3b9",
      "sender": "instagibbs",
      "payload": "yes yay",
      "action": false,
      "timestamp": "2018-12-14T19:03:07+00:00"
    },
    {
      "id": "19d97fc52e144f0eb1b365992cb07190",
      "sender": "provoostenator",
      "payload": "Probably topics:",
      "action": false,
      "timestamp": "2018-12-14T19:03:13+00:00"
    },
    {
      "id": "3d485c66ff0541c5b07a431dc51c1b40",
      "sender": "jnewbery",
      "payload": "#14565 is wallety",
      "action": false,
      "timestamp": "2018-12-14T19:03:18+00:00"
    },
    {
      "id": "9c55f73cf8ad4fd69dd41eff7178b605",
      "sender": "provoostenator",
      "payload": "1. progress towards descriptor wallets",
      "action": false,
      "timestamp": "2018-12-14T19:03:19+00:00"
    },
    {
      "id": "8ec85c2c08b44fa8ac12487be0e3c5fe",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14565 | Overhaul importmulti logic by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14565 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:03:23+00:00"
    },
    {
      "id": "6aaafd200447457ba0944a75049f3d63",
      "sender": "provoostenator",
      "payload": "2. progrress towards hardware wallets",
      "action": false,
      "timestamp": "2018-12-14T19:03:26+00:00"
    },
    {
      "id": "9b816535149445d08843a329ca26edce",
      "sender": "jnewbery",
      "payload": "#11082 isn't wallety, but is blocking provoostenator's wallety PRs",
      "action": false,
      "timestamp": "2018-12-14T19:03:46+00:00"
    },
    {
      "id": "619e1c38d5af4750954394564a4e98d0",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/11082 | Add new bitcoin_rw.conf file that is used for settings modified by this software itself by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #11082 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:03:49+00:00"
    },
    {
      "id": "3b4095bddd1643b0bf9c5032b32147d6",
      "sender": "achow101",
      "payload": "hi",
      "action": false,
      "timestamp": "2018-12-14T19:04:07+00:00"
    },
    {
      "id": "0b1adb149fca4f28b9b3a5de02d5e537",
      "sender": "meshcollider",
      "payload": "#topic high priority",
      "action": false,
      "timestamp": "2018-12-14T19:04:30+00:00"
    },
    {
      "id": "ee57a3e5c3254489a3911651a6cfa08a",
      "sender": "provoostenator",
      "payload": "jnewbery: it's blocking my future walety PR's (once I start on the GUI side of hardware wallets)",
      "action": false,
      "timestamp": "2018-12-14T19:04:30+00:00"
    },
    {
      "id": "42a55416dd074ab4ab1b75e6cf505151",
      "sender": "meshcollider",
      "payload": "jnewbery: they are both already on the list right",
      "action": false,
      "timestamp": "2018-12-14T19:05:18+00:00"
    },
    {
      "id": "dd023347c10a4973b0851efa8dd1aad9",
      "sender": "provoostenator",
      "payload": "Correct",
      "action": false,
      "timestamp": "2018-12-14T19:05:36+00:00"
    },
    {
      "id": "aa4d86457bc644fab8d278fb36223a81",
      "sender": "jnewbery",
      "payload": "14565 looks good. It was missing tests, but now looks pretty well covered (thanks sipa!)",
      "action": false,
      "timestamp": "2018-12-14T19:05:37+00:00"
    },
    {
      "id": "cf981c3ad7e447d2a31e167c01951c9e",
      "sender": "jnewbery",
      "payload": "yes, both there already: https://github.com/bitcoin/bitcoin/projects/8",
      "action": false,
      "timestamp": "2018-12-14T19:05:47+00:00"
    },
    {
      "id": "0e3024691df64d3b979ce4fae9741ac4",
      "sender": "provoostenator",
      "payload": "Once that's in, I would say #14491 become priority.",
      "action": false,
      "timestamp": "2018-12-14T19:06:10+00:00"
    },
    {
      "id": "8289ef95854c475e9a886a5c8adc8509",
      "sender": "meshcollider",
      "payload": "Yes 14565 is very nearly ready I think",
      "action": false,
      "timestamp": "2018-12-14T19:06:13+00:00"
    },
    {
      "id": "7570b1b1cf594d5fb02d23e9b6a00fb9",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider \u00c3\u0082\u00c2\u00b7 Pull Request #14491 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:06:13+00:00"
    },
    {
      "id": "c37c7a65dcde40d19dd87b432b0b071b",
      "sender": "jnewbery",
      "payload": "reminder that 14565 blocks PRs from meshcollider and achow101, so it'd be nice to move it towards merge",
      "action": false,
      "timestamp": "2018-12-14T19:06:36+00:00"
    },
    {
      "id": "530131fb4b8a4e228ff0605801b87e35",
      "sender": "meshcollider",
      "payload": "Yes",
      "action": false,
      "timestamp": "2018-12-14T19:07:59+00:00"
    },
    {
      "id": "d4e1dfc30ed748088512b413946bde18",
      "sender": "meshcollider",
      "payload": "I had one last little question on there",
      "action": false,
      "timestamp": "2018-12-14T19:08:08+00:00"
    },
    {
      "id": "10f7d923735c44e3954d77cba25f2c66",
      "sender": "meshcollider",
      "payload": "Is there anything else wallet related that should go onto high priority list now?",
      "action": false,
      "timestamp": "2018-12-14T19:09:23+00:00"
    },
    {
      "id": "4430451ff80c44a99b9a09de5bd5b749",
      "sender": "meshcollider",
      "payload": "#topic progress towards descriptor wallets (provoostenator)",
      "action": false,
      "timestamp": "2018-12-14T19:10:08+00:00"
    },
    {
      "id": "be88f640008f44e698857f70a319a362",
      "sender": "provoostenator",
      "payload": "looks as sipa",
      "action": true,
      "timestamp": "2018-12-14T19:10:33+00:00"
    },
    {
      "id": "398f4c24a7c843e6abdb09dc0fbbbfa5",
      "sender": "provoostenator",
      "payload": "*at",
      "action": false,
      "timestamp": "2018-12-14T19:10:39+00:00"
    },
    {
      "id": "63d735bc0b7d4cce82414d9d40b1ad60",
      "sender": "sipa",
      "payload": "stares back",
      "action": true,
      "timestamp": "2018-12-14T19:10:53+00:00"
    },
    {
      "id": "bed23d7535764c85a2a270ff965e9ef6",
      "sender": "instagibbs",
      "payload": "various descriptor support, like in listunspent?",
      "action": false,
      "timestamp": "2018-12-14T19:11:21+00:00"
    },
    {
      "id": "0ec4d3b480464af3b81e216a140c965e",
      "sender": "meshcollider",
      "payload": "sipa: have you thought more about it recently or been mostly focused on the PRNG stuff",
      "action": false,
      "timestamp": "2018-12-14T19:11:22+00:00"
    },
    {
      "id": "e9df4e788def46d4a90fe535e27fb067",
      "sender": "sipa",
      "payload": "meshcollider: no, sorry - i've been busy with a few other projects",
      "action": false,
      "timestamp": "2018-12-14T19:11:53+00:00"
    },
    {
      "id": "3fa695b2571942e5993e1e6923edda11",
      "sender": "meshcollider",
      "payload": "that's fine of course :) I guess that is your update provoostenator",
      "action": false,
      "timestamp": "2018-12-14T19:12:36+00:00"
    },
    {
      "id": "e0402e40916346838e15e38fed10a250",
      "sender": "meshcollider",
      "payload": "There was an issue tracking which RPCs to add descriptor support to iirc",
      "action": false,
      "timestamp": "2018-12-14T19:12:51+00:00"
    },
    {
      "id": "b21264bc9a3c4afca50fcc650ed91692",
      "sender": "provoostenator",
      "payload": "Well, I'd like to know what's next, but we do have enough review work already I guess.",
      "action": false,
      "timestamp": "2018-12-14T19:12:53+00:00"
    },
    {
      "id": "e349f76df37248f3ac7619f46ffcac17",
      "sender": "sipa",
      "payload": "next step is moving IsMine and related functions to be part of the wallet or some other abstraction, rather than free functions",
      "action": false,
      "timestamp": "2018-12-14T19:13:00+00:00"
    },
    {
      "id": "d91d15bb8aee421585d99c225eaf0518",
      "sender": "sipa",
      "payload": "so they can later be extended to be descriptor based",
      "action": false,
      "timestamp": "2018-12-14T19:13:14+00:00"
    },
    {
      "id": "97c37d85020a4d0086e4445756b1e2ba",
      "sender": "provoostenator",
      "payload": "Ok",
      "action": false,
      "timestamp": "2018-12-14T19:13:20+00:00"
    },
    {
      "id": "43a5d8febb8f4e769351878278ab9f18",
      "sender": "provoostenator",
      "payload": "Anything about the Keypool we can improve?",
      "action": false,
      "timestamp": "2018-12-14T19:13:27+00:00"
    },
    {
      "id": "9b40069045eb40d98551bcffe11b13b5",
      "sender": "meshcollider",
      "payload": "I can take a stab at the ismine stuff",
      "action": false,
      "timestamp": "2018-12-14T19:13:58+00:00"
    },
    {
      "id": "6480cda556ad4ae3b7c4308f987dbf09",
      "sender": "sipa",
      "payload": "that needs to be part of the same interface, i expect",
      "action": false,
      "timestamp": "2018-12-14T19:14:01+00:00"
    },
    {
      "id": "83374432ed074e73a86aa8279d6fedaa",
      "sender": "provoostenator",
      "payload": "For example I'm a bit worried about #14075 interacting with the keypool from RPC code.",
      "action": false,
      "timestamp": "2018-12-14T19:14:05+00:00"
    },
    {
      "id": "07d5c8bb3d9447f6bc8e7697fdd02436",
      "sender": "sipa",
      "payload": "but maybe a later step",
      "action": false,
      "timestamp": "2018-12-14T19:14:08+00:00"
    },
    {
      "id": "7f04292cbbd54c96929e6a39f6d89c09",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14075 | Import watch only pubkeys to the keypool if private keys are disabled by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #14075 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:14:08+00:00"
    },
    {
      "id": "37fd398c565f4163885f870b23bfa68a",
      "sender": "sipa",
      "payload": "that looks like it may interact, indeed",
      "action": false,
      "timestamp": "2018-12-14T19:14:39+00:00"
    },
    {
      "id": "1f9eb87910d24794981ed307d73cff31",
      "sender": "provoostenator",
      "payload": "Though it can be refactored after that, it's nice if we at least have an idea of what the final thing needs to look like.",
      "action": false,
      "timestamp": "2018-12-14T19:14:57+00:00"
    },
    {
      "id": "0860867a7b484a0c87940d250f693b63",
      "sender": "meshcollider",
      "payload": "So instead of the keypool being generated by a single descriptor, it will have imports in it too",
      "action": false,
      "timestamp": "2018-12-14T19:15:11+00:00"
    },
    {
      "id": "6953c1d487c24d80b011b9ded3df923a",
      "sender": "provoostenator",
      "payload": "Right now the keypool, when it expands itself, makes strong assumptions about the wallet and just uses the HD structure.",
      "action": false,
      "timestamp": "2018-12-14T19:15:48+00:00"
    },
    {
      "id": "34d04b5626d44f4db4e959a1bdc39826",
      "sender": "provoostenator",
      "payload": "Whereas what we want probably is for the keypool (or something like it) to expand a specific descriptor.",
      "action": false,
      "timestamp": "2018-12-14T19:16:14+00:00"
    },
    {
      "id": "c38c0121f2dc403bb7aea39d99de5039",
      "sender": "sipa",
      "payload": "well there won't be a keypool anymore",
      "action": false,
      "timestamp": "2018-12-14T19:16:27+00:00"
    },
    {
      "id": "e96ae052a38849ed9c83ba13ed9a2999",
      "sender": "sipa",
      "payload": "it's just a descriptor, which some entries cached, and some not (yet)",
      "action": false,
      "timestamp": "2018-12-14T19:16:43+00:00"
    },
    {
      "id": "57ff153812f942d6ac4c536583009b94",
      "sender": "sipa",
      "payload": "the hard part is integrating the existing logic into such a structure",
      "action": false,
      "timestamp": "2018-12-14T19:17:04+00:00"
    },
    {
      "id": "d6d649aeab22424a9bdbdad50af83218",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b53573e5c6c7...6723d8e3a694",
      "action": false,
      "timestamp": "2018-12-14T19:17:16+00:00"
    },
    {
      "id": "4d44bc2da9d7408d8afd3b6a15414b2d",
      "sender": "bitcoin-git",
      "payload": "bitcoin/master fa30a0e MarcoFalke: test: mempool_persist: Verify prioritization is dumped correctly",
      "action": false,
      "timestamp": "2018-12-14T19:17:16+00:00"
    },
    {
      "id": "a79e2ec25afc437293aaa36a39f6e63e",
      "sender": "bitcoin-git",
      "payload": "bitcoin/master 6723d8e MarcoFalke: Merge #14931: test: mempool_persist: Verify prioritization is dumped correctly...",
      "action": false,
      "timestamp": "2018-12-14T19:17:17+00:00"
    },
    {
      "id": "94c84b3db68e4cefb3735eb096e3fae9",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] MarcoFalke closed pull request #14931: test: mempool_persist: Verify prioritization is dumped correctly (master...Mf1812-testMempoolPrio) https://github.com/bitcoin/bitcoin/pull/14931",
      "action": false,
      "timestamp": "2018-12-14T19:17:52+00:00"
    },
    {
      "id": "a7518e0674584375a62f4bbd9a2bbf69",
      "sender": "provoostenator",
      "payload": "Any prose or pseudo-code describing how such integration would work is most welcome (in a Github issue).",
      "action": false,
      "timestamp": "2018-12-14T19:18:12+00:00"
    },
    {
      "id": "d0f4b17064f04f379b437481e65aa3fa",
      "sender": "sipa",
      "payload": "i think we'll want to see the existing keypool/keys/... as a special \"legacy\" descriptor that doesn't really have a text representation",
      "action": false,
      "timestamp": "2018-12-14T19:18:36+00:00"
    },
    {
      "id": "77a30de94ec7403fb106c88ceec701a5",
      "sender": "meshcollider",
      "payload": "sipa: to cache them, would you expand() them with the cache function during the existin topup function",
      "action": false,
      "timestamp": "2018-12-14T19:19:00+00:00"
    },
    {
      "id": "7f7e5466389f4851b18dcf11f745ffda",
      "sender": "sipa",
      "payload": "meshcollider: right, exactly",
      "action": false,
      "timestamp": "2018-12-14T19:19:20+00:00"
    },
    {
      "id": "3a9663ddb3e14bd58cb12eadb4247819",
      "sender": "provoostenator",
      "payload": "Why wouldn't it have a text representation?",
      "action": false,
      "timestamp": "2018-12-14T19:19:43+00:00"
    },
    {
      "id": "7091f8cdc87249d68f7983d216714f23",
      "sender": "sipa",
      "payload": "provoostenator: well the text representation is the entire set of keys, pubkeys, scripts, hdpaths, ... that are currently stored in the wallet :)",
      "action": false,
      "timestamp": "2018-12-14T19:20:10+00:00"
    },
    {
      "id": "414315758d7c445cbc089dc949028bde",
      "sender": "sipa",
      "payload": "i guess you could dump it in hex or something",
      "action": false,
      "timestamp": "2018-12-14T19:20:26+00:00"
    },
    {
      "id": "558b8621d2f74667bbffc993df521bde",
      "sender": "provoostenator",
      "payload": "A migration wizard should be able to, at least for standard wallets, turn that into a series of regular descriptors no?",
      "action": false,
      "timestamp": "2018-12-14T19:20:58+00:00"
    },
    {
      "id": "10cde4472d5241eabc77681d45529e4e",
      "sender": "sipa",
      "payload": "that may be possible, but i don't think that's the priority now",
      "action": false,
      "timestamp": "2018-12-14T19:21:15+00:00"
    },
    {
      "id": "6f14f34b545043fdbe8afd7b105c8314",
      "sender": "meshcollider",
      "payload": "It'd be quicker and safer to just move-only the code type of thing into legacy functions",
      "action": false,
      "timestamp": "2018-12-14T19:21:50+00:00"
    },
    {
      "id": "e20865b19c1643d988aeeb1dbe710fb4",
      "sender": "sipa",
      "payload": "(because then you have to worry about all existing RPCs, and their effect on those descriptors)",
      "action": false,
      "timestamp": "2018-12-14T19:21:56+00:00"
    },
    {
      "id": "6e8e956ad144441fab1a52a3fcd4d1ee",
      "sender": "provoostenator",
      "payload": "Right, just depends on what's easier in practice. Personally I suspect it'll be easier to make the wallet _only_ have descriptors, just from a writing tests point of view.",
      "action": false,
      "timestamp": "2018-12-14T19:22:11+00:00"
    },
    {
      "id": "80b48f7184ce4b5fb2cc59f751f5ed58",
      "sender": "sipa",
      "payload": "i think it's not too much work to just have a legacy subsystem, and a new system - and a wallet can contain just one of them, or both",
      "action": false,
      "timestamp": "2018-12-14T19:22:23+00:00"
    },
    {
      "id": "ba4bfe65f28f4a41bd856d25bd8f8d80",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6723d8e3a694...9133227298ad",
      "action": false,
      "timestamp": "2018-12-14T19:22:23+00:00"
    },
    {
      "id": "fa278526b9544cd48b6dbe18f9399ae4",
      "sender": "bitcoin-git",
      "payload": "bitcoin/master c84c2b8 practicalswift: tests: Test for expected return values when calling functions returning a success code",
      "action": false,
      "timestamp": "2018-12-14T19:22:24+00:00"
    },
    {
      "id": "23fae0d11505441596dc902667ef3e4a",
      "sender": "bitcoin-git",
      "payload": "bitcoin/master 9133227 MarcoFalke: Merge #14935: tests: Test for expected return values when calling functions returning a success code...",
      "action": false,
      "timestamp": "2018-12-14T19:22:24+00:00"
    },
    {
      "id": "6d4bd6d08b674f7abedac668a91458c1",
      "sender": "provoostenator",
      "payload": "But the RPC thing could be a pain yes.",
      "action": false,
      "timestamp": "2018-12-14T19:22:33+00:00"
    },
    {
      "id": "94a9265b70bb4413a98c83bb4b184072",
      "sender": "sipa",
      "payload": "maybe we even want to forbid mixing them in one wallet",
      "action": false,
      "timestamp": "2018-12-14T19:22:34+00:00"
    },
    {
      "id": "d28532691ede4b46beacc7ce45ade3f8",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] MarcoFalke closed pull request #14935: tests: Test for expected return values when calling functions returning a success code (master...test-return-values) https://github.com/bitcoin/bitcoin/pull/14935",
      "action": false,
      "timestamp": "2018-12-14T19:23:06+00:00"
    },
    {
      "id": "a085325ba3f2438ea28ba0bfea58de12",
      "sender": "sipa",
      "payload": "but forced migration may getting it accepted much harder",
      "action": false,
      "timestamp": "2018-12-14T19:23:07+00:00"
    },
    {
      "id": "b47bb8ba1b2f47c1ac63f5acf9bca1aa",
      "sender": "sipa",
      "payload": "so my idea is that every record is a descriptor + some metadata (like gap limit, whether it's change or no), plus some cached keys... and there can be a special \"legacy\" record that's just all the existing keypool/ismine logic",
      "action": false,
      "timestamp": "2018-12-14T19:23:58+00:00"
    },
    {
      "id": "e780e9c0a7e147029eff2c61c93c196a",
      "sender": "phantomcircuit",
      "payload": "the rw config is mostly because the qt stuff writes to random places for settings right?",
      "action": false,
      "timestamp": "2018-12-14T19:24:00+00:00"
    },
    {
      "id": "44c251dceb5a433886f5770c257d7d9b",
      "sender": "provoostenator",
      "payload": "So then you might end up with two seperate wallet systems and a migration tool, where the old system only gets maintenance updates to be able to read from it.",
      "action": false,
      "timestamp": "2018-12-14T19:24:14+00:00"
    },
    {
      "id": "e0741dab33e64fcf9df6b189d84b2977",
      "sender": "luke-jr",
      "payload": "phantomcircuit: random? not really; just different from bitcoind",
      "action": false,
      "timestamp": "2018-12-14T19:24:24+00:00"
    },
    {
      "id": "4323f721a06f4dfd9a44f124ae92d436",
      "sender": "sipa",
      "payload": "provoostenator: maybe",
      "action": false,
      "timestamp": "2018-12-14T19:24:27+00:00"
    },
    {
      "id": "6eb28ce1efdc430db0569d5d8f691267",
      "sender": "phantomcircuit",
      "payload": "luke-jr, i mean it writes to reg on windows",
      "action": false,
      "timestamp": "2018-12-14T19:24:40+00:00"
    },
    {
      "id": "0f9580976f69404096e92952c4c2bb4b",
      "sender": "phantomcircuit",
      "payload": "which is super annoying to change",
      "action": false,
      "timestamp": "2018-12-14T19:24:45+00:00"
    },
    {
      "id": "6e29bbb899a7467ca5e01f6f94fd1994",
      "sender": "sipa",
      "payload": "phantomcircuit: please stick to topic",
      "action": false,
      "timestamp": "2018-12-14T19:24:52+00:00"
    },
    {
      "id": "de2afc6d114f41cd9699374261a2a5d2",
      "sender": "meshcollider",
      "payload": "phantomcircuit: we are in a wallet meeting btw :)",
      "action": false,
      "timestamp": "2018-12-14T19:24:55+00:00"
    },
    {
      "id": "8789b9d82d77404f9b09deb999a5648e",
      "sender": "meshcollider",
      "payload": "We could open an issue to discuss the alternative approaches, or just discuss which is easier as we start actually writing the code",
      "action": false,
      "timestamp": "2018-12-14T19:26:04+00:00"
    },
    {
      "id": "495979c3c434486289beb98d0923f346",
      "sender": "provoostenator",
      "payload": "A new wallet subsystem might also let us cleanly long-term deprecate some wallet RPC methods and replace them with clean ones, that happen to support descriptors?",
      "action": false,
      "timestamp": "2018-12-14T19:26:09+00:00"
    },
    {
      "id": "5e64c798e5d94395a25df31080002d49",
      "sender": "sipa",
      "payload": "provoostenator: yup",
      "action": false,
      "timestamp": "2018-12-14T19:26:18+00:00"
    },
    {
      "id": "dd6d4e8623824c42a3e9e18dfef49791",
      "sender": "provoostenator",
      "payload": "And maybe move to Sqlite3 at the same time.",
      "action": false,
      "timestamp": "2018-12-14T19:26:19+00:00"
    },
    {
      "id": "c31ec032ed2a4b36b9dd727a944f92aa",
      "sender": "sipa",
      "payload": "i think that's completely orthogonal",
      "action": false,
      "timestamp": "2018-12-14T19:26:36+00:00"
    },
    {
      "id": "5e321a77eee249aa86b2f9dd66e6adc0",
      "sender": "provoostenator",
      "payload": "Could be, yes.",
      "action": false,
      "timestamp": "2018-12-14T19:26:49+00:00"
    },
    {
      "id": "0b50a3f18e6c41dbb63dbb84b4a5a324",
      "sender": "luke-jr",
      "payload": "phantomcircuit: I answered in #bitcoin fyi",
      "action": false,
      "timestamp": "2018-12-14T19:27:11+00:00"
    },
    {
      "id": "e8547e04089c426b9d277562a39ea355",
      "sender": "provoostenator",
      "payload": "Anyway, we have some next actions now to continue progress, maybe next topic?",
      "action": false,
      "timestamp": "2018-12-14T19:28:05+00:00"
    },
    {
      "id": "f63df59767ab4dd3bd2f4f1d3c52462f",
      "sender": "meshcollider",
      "payload": "#topic progress towards hardware wallets (provoostenator)",
      "action": false,
      "timestamp": "2018-12-14T19:28:37+00:00"
    },
    {
      "id": "0eb34b698e414be496d7ba702fbd70d5",
      "sender": "jnewbery",
      "payload": "IsMine moved from wallet to common here: https://github.com/bitcoin/bitcoin/commit/a25a4f5b04c3e045557e9e7e807b2af74ad75128 . Was that just because of the way the multisig and P2SH tests are constructed (ie could those tests just be rewritten)?",
      "action": false,
      "timestamp": "2018-12-14T19:28:46+00:00"
    },
    {
      "id": "d24e8cac5db045b2a1913dd770869cba",
      "sender": "provoostenator",
      "payload": "Now that #14491 has been rebased, the `hww` branch I'm building off should also soon be rebased.",
      "action": false,
      "timestamp": "2018-12-14T19:28:54+00:00"
    },
    {
      "id": "1db211cd1fad4dccbc505b1aa532238e",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider \u00c3\u0082\u00c2\u00b7 Pull Request #14491 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:28:57+00:00"
    },
    {
      "id": "c0f1323d66db4f858628e3e5b287949d",
      "sender": "provoostenator",
      "payload": "#14912",
      "action": false,
      "timestamp": "2018-12-14T19:29:02+00:00"
    },
    {
      "id": "d6ee227dc7cd4b6a8b9c32dfb719e7f0",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14912 | [WIP] External signer support (e.g. hardware wallet) by Sjors \u00c3\u0082\u00c2\u00b7 Pull Request #14912 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:29:04+00:00"
    },
    {
      "id": "ecfaeac8c53042c289d98ce092d286c1",
      "sender": "provoostenator",
      "payload": "I'll do some cleaning up of my ugly string concatenation descriptor code after rebase.",
      "action": false,
      "timestamp": "2018-12-14T19:29:27+00:00"
    },
    {
      "id": "6166d37786cc4647884b074552b935ea",
      "sender": "provoostenator",
      "payload": "People can already test it though. It compiles and actually works (at your own risk, so use testnet).",
      "action": false,
      "timestamp": "2018-12-14T19:29:54+00:00"
    },
    {
      "id": "20493977d6744aa5ba21ea7a8a34247e",
      "sender": "meshcollider",
      "payload": "jnewbery: looks like it?",
      "action": false,
      "timestamp": "2018-12-14T19:30:14+00:00"
    },
    {
      "id": "97103c629ded4372abe9eace66dba432",
      "sender": "provoostenator",
      "payload": "As in, you can create a new wallet, put read-only keys in it, show address on the device and spend coins. Using achow101's HWI library to talk to the device.",
      "action": false,
      "timestamp": "2018-12-14T19:30:47+00:00"
    },
    {
      "id": "8ed8f607dac84ad983ca5294b183aa1f",
      "sender": "provoostenator",
      "payload": "So the idea is that users would download that library seperately and just launch bitcoind -signer=../HWI/hwi.py (or some other tool). That way we don't ahve to review individual hardware wallet code.",
      "action": false,
      "timestamp": "2018-12-14T19:31:28+00:00"
    },
    {
      "id": "7df9c93055b64e68899384ccc02208bb",
      "sender": "meshcollider",
      "payload": "provoostenator: cool \\o/",
      "action": false,
      "timestamp": "2018-12-14T19:31:49+00:00"
    },
    {
      "id": "c1b71fb7d48d4e09ab7f3d91fd631966",
      "sender": "provoostenator",
      "payload": "With very little code changes this could also work against a gRPC server, but there's some security trade-offs compared to calling commands. We talked about that a few weeks ago.",
      "action": false,
      "timestamp": "2018-12-14T19:32:30+00:00"
    },
    {
      "id": "06bbdbbdf0f144fab51b98160365642f",
      "sender": "provoostenator",
      "payload": "I am trying to keep it generic enough to keep that possible, so e.g. the -signer= command could later also be a URL. But for now, it just executes a command and parses the JSON that command spits out to stdout.",
      "action": false,
      "timestamp": "2018-12-14T19:33:11+00:00"
    },
    {
      "id": "8bde5172cf6e4b6390c39facfa1aaa95",
      "sender": "provoostenator",
      "payload": "Next step for me is to work on GUI support for this. But there's already a pile of prerequisite stuff to review, so don't worry too much about that :-)",
      "action": false,
      "timestamp": "2018-12-14T19:34:01+00:00"
    },
    {
      "id": "5cf26c4a596640d384ff75ad61f9bc74",
      "sender": "provoostenator",
      "payload": "I personally just like to see the big picture in action.",
      "action": false,
      "timestamp": "2018-12-14T19:34:21+00:00"
    },
    {
      "id": "2c8d7fdd0ce94afdabaf3f37dea87295",
      "sender": "meshcollider",
      "payload": "Yes let's not stack too many PRs at once again ;)",
      "action": false,
      "timestamp": "2018-12-14T19:34:27+00:00"
    },
    {
      "id": "44352c8307c142598b79783d047a85fa",
      "sender": "provoostenator",
      "payload": "For GUI proof of concept I'd just like to nag promag about #13100, which we want anyway.",
      "action": false,
      "timestamp": "2018-12-14T19:35:47+00:00"
    },
    {
      "id": "60a73c9da3604cbd82ad6ea2c04219f0",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/13100 | gui: Add dynamic wallets support by promag \u00c3\u0082\u00c2\u00b7 Pull Request #13100 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:35:50+00:00"
    },
    {
      "id": "35d4ef65b747462da0e7b6fafcd06ccf",
      "sender": "provoostenator",
      "payload": "(that's all I have)",
      "action": false,
      "timestamp": "2018-12-14T19:36:24+00:00"
    },
    {
      "id": "0c87ece66b1b4ae18995cde8157b270c",
      "sender": "MarcoFalke",
      "payload": "wget https://bitcointools.jonasschnelli.ch/data/builds/914/    ... failed: Connection refused",
      "action": false,
      "timestamp": "2018-12-14T19:37:52+00:00"
    },
    {
      "id": "b7ac9971ed1b443683a8de19412f8d7e",
      "sender": "meshcollider",
      "payload": "that PR hasn't been rebased for 2 months, perhaps you'd like to take it up and rebased it yourself?",
      "action": false,
      "timestamp": "2018-12-14T19:37:53+00:00"
    },
    {
      "id": "088ad1cc220643ebab1d3b523dc39694",
      "sender": "MarcoFalke",
      "payload": "^ jonasschnelli",
      "action": false,
      "timestamp": "2018-12-14T19:37:55+00:00"
    },
    {
      "id": "1d55a5da4515484287ff20cf40f4c180",
      "sender": "provoostenator",
      "payload": "He actually said he's working on it soon.",
      "action": false,
      "timestamp": "2018-12-14T19:38:06+00:00"
    },
    {
      "id": "7dbbbb42f856496ca3a63c9c66306d61",
      "sender": "meshcollider",
      "payload": "Oh ok",
      "action": false,
      "timestamp": "2018-12-14T19:38:14+00:00"
    },
    {
      "id": "e72a466d3bbd48aaa0813b2859199f8c",
      "sender": "provoostenator",
      "payload": "I think he needs this to go in first #14573, that's almost mergeable.",
      "action": false,
      "timestamp": "2018-12-14T19:38:48+00:00"
    },
    {
      "id": "db1c266a9a2d4d4f94636d830c8abfbb",
      "sender": "meshcollider",
      "payload": "I'm happy to review it as soon as its ready, its already tagged for 0.18",
      "action": false,
      "timestamp": "2018-12-14T19:38:49+00:00"
    },
    {
      "id": "ee4b453350004e5f858b95205f9e74c8",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14573 | qt: Add Window menu by promag \u00c3\u0082\u00c2\u00b7 Pull Request #14573 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2018-12-14T19:38:50+00:00"
    },
    {
      "id": "4596a2aea6f445ee9010536b10b2a703",
      "sender": "meshcollider",
      "payload": "Are there any other topics?",
      "action": false,
      "timestamp": "2018-12-14T19:39:44+00:00"
    },
    {
      "id": "7e24cd65acb74e36a595d907a94d672f",
      "sender": "provoostenator",
      "payload": "luke-jr phantomcircuit did you want to discuss rw_config stuff?",
      "action": false,
      "timestamp": "2018-12-14T19:40:04+00:00"
    },
    {
      "id": "194686b5d88241539cfe0e095bfc3b92",
      "sender": "provoostenator",
      "payload": "I noticed the rabase, so I'll rebase my Settings migration stuff as well.",
      "action": false,
      "timestamp": "2018-12-14T19:40:31+00:00"
    },
    {
      "id": "ac844280d79549f78ccefe93fcecbe5d",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] ch4ot1c opened pull request #14961: [docs] Root readme improvements (master...improvements/readme) https://github.com/bitcoin/bitcoin/pull/14961",
      "action": false,
      "timestamp": "2018-12-14T19:40:47+00:00"
    },
    {
      "id": "594e0641d52a4901ac4ac84a1269b2a9",
      "sender": "provoostenator",
      "payload": "But it's not very wallety.",
      "action": false,
      "timestamp": "2018-12-14T19:40:56+00:00"
    },
    {
      "id": "cbb7260b3afd4e8f9bbf94b6639263d4",
      "sender": "meshcollider",
      "payload": "Looks like that might be it for today",
      "action": false,
      "timestamp": "2018-12-14T19:41:36+00:00"
    },
    {
      "id": "89b6724561ce494bb81ac79cc859fbae",
      "sender": "meshcollider",
      "payload": "Thanks provoostenator :)",
      "action": false,
      "timestamp": "2018-12-14T19:41:43+00:00"
    },
    {
      "id": "0a61d670087b4b709264c5971fc62cc6",
      "sender": "meshcollider",
      "payload": "#endmeeting",
      "action": false,
      "timestamp": "2018-12-14T19:41:51+00:00"
    }
  ],
  "events": [
    {
      "event_type": "START_MEETING",
      "message": {
        "id": "56143019f823418ea312f3be748015ed",
        "sender": "meshcollider",
        "payload": "#startmeeting",
        "action": false,
        "timestamp": "2018-12-14T19:01:42+00:00"
      },
      "operand": null,
      "id": "56143019f823418ea312f3be748015ed",
      "timestamp": "2018-12-14T19:01:42+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "8ec85c2c08b44fa8ac12487be0e3c5fe",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14565 | Overhaul importmulti logic by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14565 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:03:23+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14565",
      "id": "8ec85c2c08b44fa8ac12487be0e3c5fe",
      "timestamp": "2018-12-14T19:03:23+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "619e1c38d5af4750954394564a4e98d0",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/11082 | Add new bitcoin_rw.conf file that is used for settings modified by this software itself by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #11082 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:03:49+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/11082",
      "id": "619e1c38d5af4750954394564a4e98d0",
      "timestamp": "2018-12-14T19:03:49+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "0b1adb149fca4f28b9b3a5de02d5e537",
        "sender": "meshcollider",
        "payload": "#topic high priority",
        "action": false,
        "timestamp": "2018-12-14T19:04:30+00:00"
      },
      "operand": "high priority",
      "id": "0b1adb149fca4f28b9b3a5de02d5e537",
      "timestamp": "2018-12-14T19:04:30+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "7570b1b1cf594d5fb02d23e9b6a00fb9",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider \u00c3\u0082\u00c2\u00b7 Pull Request #14491 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:06:13+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14491",
      "id": "7570b1b1cf594d5fb02d23e9b6a00fb9",
      "timestamp": "2018-12-14T19:06:13+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "4430451ff80c44a99b9a09de5bd5b749",
        "sender": "meshcollider",
        "payload": "#topic progress towards descriptor wallets (provoostenator)",
        "action": false,
        "timestamp": "2018-12-14T19:10:08+00:00"
      },
      "operand": "progress towards descriptor wallets (provoostenator)",
      "id": "4430451ff80c44a99b9a09de5bd5b749",
      "timestamp": "2018-12-14T19:10:08+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "7f04292cbbd54c96929e6a39f6d89c09",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14075 | Import watch only pubkeys to the keypool if private keys are disabled by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #14075 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:14:08+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14075",
      "id": "7f04292cbbd54c96929e6a39f6d89c09",
      "timestamp": "2018-12-14T19:14:08+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "f63df59767ab4dd3bd2f4f1d3c52462f",
        "sender": "meshcollider",
        "payload": "#topic progress towards hardware wallets (provoostenator)",
        "action": false,
        "timestamp": "2018-12-14T19:28:37+00:00"
      },
      "operand": "progress towards hardware wallets (provoostenator)",
      "id": "f63df59767ab4dd3bd2f4f1d3c52462f",
      "timestamp": "2018-12-14T19:28:37+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "1db211cd1fad4dccbc505b1aa532238e",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider \u00c3\u0082\u00c2\u00b7 Pull Request #14491 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:28:57+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14491",
      "id": "1db211cd1fad4dccbc505b1aa532238e",
      "timestamp": "2018-12-14T19:28:57+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "d6ee227dc7cd4b6a8b9c32dfb719e7f0",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14912 | [WIP] External signer support (e.g. hardware wallet) by Sjors \u00c3\u0082\u00c2\u00b7 Pull Request #14912 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:29:04+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14912",
      "id": "d6ee227dc7cd4b6a8b9c32dfb719e7f0",
      "timestamp": "2018-12-14T19:29:04+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "60a73c9da3604cbd82ad6ea2c04219f0",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/13100 | gui: Add dynamic wallets support by promag \u00c3\u0082\u00c2\u00b7 Pull Request #13100 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:35:50+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/13100",
      "id": "60a73c9da3604cbd82ad6ea2c04219f0",
      "timestamp": "2018-12-14T19:35:50+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "ee4b453350004e5f858b95205f9e74c8",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14573 | qt: Add Window menu by promag \u00c3\u0082\u00c2\u00b7 Pull Request #14573 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2018-12-14T19:38:50+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14573",
      "id": "ee4b453350004e5f858b95205f9e74c8",
      "timestamp": "2018-12-14T19:38:50+00:00"
    },
    {
      "event_type": "END_MEETING",
      "message": {
        "id": "0a61d670087b4b709264c5971fc62cc6",
        "sender": "meshcollider",
        "payload": "#endmeeting",
        "action": false,
        "timestamp": "2018-12-14T19:41:51+00:00"
      },
      "operand": null,
      "id": "0a61d670087b4b709264c5971fc62cc6",
      "timestamp": "2018-12-14T19:41:51+00:00"
    }
  ],
  "aliases": {},
  "vote_in_progress": false,
  "motion_index": null
}