{ "founder": "wumpus", "channel": "#bitcoin-core-dev", "network": "freenode", "id": "3ca0dc3506fd47b0ae7a59bc5863b5b2", "name": "#bitcoin-core-dev", "chair": "wumpus", "chairs": [ "wumpus" ], "nicks": { "wumpus": 83, "lightningbot": 2, "jonasschnelli": 7, "sipa": 51, "achow101": 28, "amiti": 1, "meshcollider": 9, "jonatack": 5, "cfields": 14, "kanzure": 1, "dongcarl": 1, "elichai2": 1, "jb55": 6, "MarcoFalke": 23, "luke-jr": 78, "phantomcircuit": 1, "jkczyz": 1, "fjahr": 1, "hebasto": 3, "gribble": 14, "midnight": 1, "ryanofsky": 58, "tryphe": 2, "bitcoin-git": 1 }, "start_time": "2020-04-16T19:03:58+00:00", "end_time": "2020-04-16T20:12:09+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": "experimental libmultiprocess, next steps for multiprocess in general (MarcoFalke)", "messages": [ { "id": "da82f27edddf49368a972cd2d8704c95", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2020-04-16T19:03:58+00:00" }, { "id": "553376baa556468385e874383f4e494b", "sender": "lightningbot", "payload": "Meeting started Thu Apr 16 19:03:58 2020 UTC. The chair is wumpus. Information about MeetBot at http://wiki.debian.org/MeetBot.", "action": false, "timestamp": "2020-04-16T19:03:58+00:00" }, { "id": "d0a2355a503f4794b741b2a98ccb3864", "sender": "lightningbot", "payload": "Useful Commands: #action #agreed #help #info #idea #link #topic.", "action": false, "timestamp": "2020-04-16T19:03:58+00:00" }, { "id": "d987db9086154ed3b4b9b98de6b5ca3a", "sender": "jonasschnelli", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:01+00:00" }, { "id": "0cbc0896b0074089968a54f8d00fb820", "sender": "sipa", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:03+00:00" }, { "id": "c99d68a65b7f4170b8199850869a7b2f", "sender": "achow101", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:06+00:00" }, { "id": "c6dff870f58645148df05b8d6d6a0f4b", "sender": "amiti", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:08+00:00" }, { "id": "7233dc20144e4ee6a5f06ba7f1a0553f", "sender": "meshcollider", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:10+00:00" }, { "id": "07941792639544348ae3023750708563", "sender": "jonatack", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:13+00:00" }, { "id": "4aa7a02628e6409d9309a0134debb665", "sender": "cfields", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:22+00:00" }, { "id": "7cca030dde424336899f874fbf904af6", "sender": "kanzure", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:28+00:00" }, { "id": "5d60dd7be55b42f6aab72b8fd94ea2ad", "sender": "wumpus", "payload": "#bitcoin-core-dev 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 kvaciral ariard digi_james amiti fjahr", "action": false, "timestamp": "2020-04-16T19:04:34+00:00" }, { "id": "27d97442d45146629943be88eb7df62a", "sender": "wumpus", "payload": "jeremyrubin lightlike emilengler jonatack hebasto jb55", "action": false, "timestamp": "2020-04-16T19:04:36+00:00" }, { "id": "031c14cd5ada47528aaa4112e2c1a2b4", "sender": "dongcarl", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:39+00:00" }, { "id": "0c55e7eadab745ee95078af192a082bd", "sender": "elichai2", "payload": "Hi", "action": false, "timestamp": "2020-04-16T19:04:51+00:00" }, { "id": "1840db2b064c4e5d87e6ecc0972e0bbc", "sender": "jb55", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:04:53+00:00" }, { "id": "82f6c11cf77b488fbf6a8fbd7cd50dce", "sender": "wumpus", "payload": "one proposed topic: experimental libmultiprocess, next steps for multiprocess in general (MarcoFalke)", "action": false, "timestamp": "2020-04-16T19:05:14+00:00" }, { "id": "89383c5fa36d4b2890508b32954f324d", "sender": "MarcoFalke", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:05:22+00:00" }, { "id": "3dd7291f4fb440ee84dc78cbff47976a", "sender": "luke-jr", "payload": "wumpus: I proposed one last week.. which really should get addressed before 0.20", "action": false, "timestamp": "2020-04-16T19:05:30+00:00" }, { "id": "eb5780595d6c43c2b859a1e309b16fc0", "sender": "phantomcircuit", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:05:40+00:00" }, { "id": "85ab45bd02b14fd88edf0f4c0434a69f", "sender": "jkczyz", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:05:44+00:00" }, { "id": "feb9aa02f7574be5b0fb27be7cbe32b2", "sender": "fjahr", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:05:51+00:00" }, { "id": "f3d614c2f60b4861b3443101f3abc58c", "sender": "cfields", "payload": "I know fanquake really wants to be part of the multiprocess conversation but he can't make the meeting today.", "action": false, "timestamp": "2020-04-16T19:05:56+00:00" }, { "id": "eceefbec8afa41d8a3f2734791e3ace3", "sender": "hebasto", "payload": "hi", "action": false, "timestamp": "2020-04-16T19:05:57+00:00" }, { "id": "fed90b32ad4443a58ef1ad0b606d1e93", "sender": "cfields", "payload": "maybe we can introduce the issue here and pick it up on the PR?", "action": false, "timestamp": "2020-04-16T19:06:19+00:00" }, { "id": "c9b4ebea252f43d6bf82c6c4a001989b", "sender": "wumpus", "payload": "luke-jr: can you be more specific? the last topic suggestion from you that I see is the PPA URL and we've spent half a meeting on that", "action": false, "timestamp": "2020-04-16T19:07:31+00:00" }, { "id": "ffdfe3728c6a49e4bc4f819006075212", "sender": "wumpus", "payload": "#topic High priority for review", "action": false, "timestamp": "2020-04-16T19:07:43+00:00" }, { "id": "cc09706b11f747e98fe5fe609d5de454", "sender": "luke-jr", "payload": "wumpus: right now, the wallet implementation is still broken, and if we don't address it before 0.20, it complicates backward compatibility", "action": false, "timestamp": "2020-04-16T19:07:54+00:00" }, { "id": "7e47c59616a84fad9aa6508cc1ff98a2", "sender": "sipa", "payload": "can i haz #185512", "action": false, "timestamp": "2020-04-16T19:08:05+00:00" }, { "id": "30f8ddbec7b941f0b11669003d8c4cfa", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/185512 | HTTP Error 404: Not Found", "action": false, "timestamp": "2020-04-16T19:08:06+00:00" }, { "id": "6c1827f8afb24db1a7238040b3866472", "sender": "jonasschnelli", "payload": "may I add #18242 to the hp list?", "action": false, "timestamp": "2020-04-16T19:08:07+00:00" }, { "id": "2b42abe0067d4bb0a2ef3d1599aa0dc2", "sender": "sipa", "payload": "can i haz #18512", "action": false, "timestamp": "2020-04-16T19:08:08+00:00" }, { "id": "f207a1a3a7004bb5bd8880535cf0679f", "sender": "midnight", "payload": "hi!", "action": false, "timestamp": "2020-04-16T19:08:08+00:00" }, { "id": "a64f0efcedde447a8199d38cf5bca01c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18242 | Add BIP324 encrypted p2p transport de-/serializer (only used in tests) by jonasschnelli \u00c3\u0082\u00c2\u00b7 Pull Request #18242 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:11+00:00" }, { "id": "6b5a92f1351f4f148cbb2a9ecc4600dd", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18512 | Improve asmap checks and add sanity check by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #18512 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:14+00:00" }, { "id": "379475a85f3b49058476cbcc4758788b", "sender": "luke-jr", "payload": "re #18192, #18550, #18572, and #18608", "action": false, "timestamp": "2020-04-16T19:08:21+00:00" }, { "id": "fe2c07e52393489c8e58b68b7bd80610", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:24+00:00" }, { "id": "7e5ce8445943498db14819b769aa5365", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18550 | Store destdata for change in separate key for backward compatibility by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18550 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:25+00:00" }, { "id": "5fb1abf59ac547acbd3bcce7101e57b1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18572 | Wallet: Accept \"changedata\" db key as an alias to \"destdata\" by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18572 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:26+00:00" }, { "id": "4b31f8d605454193badd4ed15684a139", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18608 | refactor: Remove CAddressBookData::destdata by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #18608 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:27+00:00" }, { "id": "5074e520621d491a9c1a95b8a9893e06", "sender": "wumpus", "payload": "uhmm slow down a bit please", "action": false, "timestamp": "2020-04-16T19:08:37+00:00" }, { "id": "3a69c012a3174ef6be401245414534a8", "sender": "jonasschnelli", "payload": ";-)", "action": false, "timestamp": "2020-04-16T19:08:44+00:00" }, { "id": "90da39d3791342ba8f0524803cd7cba0", "sender": "luke-jr", "payload": "(my list is not for high-prio-for-review topic)", "action": false, "timestamp": "2020-04-16T19:09:11+00:00" }, { "id": "fdc05765d2754facb392f4502f234baf", "sender": "achow101", "payload": "#18598 for 0.20 tag and backport (or merge now)", "action": false, "timestamp": "2020-04-16T19:10:53+00:00" }, { "id": "a98da469715d49f29e85824fb590b4f0", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18598 | gitian: Add missing automake package to gitian-win-signer.yml by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #18598 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:10:55+00:00" }, { "id": "89b1789c8a034dd8a6f03e0fa0f15a7c", "sender": "wumpus", "payload": "jonasschnelli: sipa added", "action": false, "timestamp": "2020-04-16T19:10:56+00:00" }, { "id": "175b395cf1e1453493ce58466fac3047", "sender": "jonasschnelli", "payload": "thanks", "action": false, "timestamp": "2020-04-16T19:11:03+00:00" }, { "id": "bced7de5e17b4a0ba2d024dcbd567f7d", "sender": "MarcoFalke", "payload": "achow101: Assigned milestone. I think this will be dealt with in rc2", "action": false, "timestamp": "2020-04-16T19:11:46+00:00" }, { "id": "7d96eb1301c14df69379212871436ec6", "sender": "wumpus", "payload": "yes, if it's required for windows signing on some environments it definitely needs to go into the next rc", "action": false, "timestamp": "2020-04-16T19:12:13+00:00" }, { "id": "25eaa84ce6954185a0b3f8333fbd7088", "sender": "wumpus", "payload": "I didn't need it for LXC fwiw", "action": false, "timestamp": "2020-04-16T19:12:24+00:00" }, { "id": "26b44ae1157f455f8d0c707cbda0ed71", "sender": "jonasschnelli", "payload": "me2", "action": false, "timestamp": "2020-04-16T19:12:34+00:00" }, { "id": "a0121debfdc940b9a3cd27172b41d4f2", "sender": "MarcoFalke", "payload": "happens only on docker/podman", "action": false, "timestamp": "2020-04-16T19:12:37+00:00" }, { "id": "831bc0ff270c4f05b75f6e14ea2aec35", "sender": "hebasto", "payload": "yes, lxc does need it", "action": false, "timestamp": "2020-04-16T19:12:42+00:00" }, { "id": "c2bfcc1531294442b1de0ba6fd7fc6dd", "sender": "achow101", "payload": "yeah, seems to be a virutaliation specific issue", "action": false, "timestamp": "2020-04-16T19:12:58+00:00" }, { "id": "7e8fde03dd0547cdba01e2d813f103bd", "sender": "achow101", "payload": "just wanted to make sure no one forgot about it", "action": false, "timestamp": "2020-04-16T19:13:12+00:00" }, { "id": "3cf7d5432acb4b8e89fef5441d0a48d8", "sender": "hebasto", "payload": "achow101: mind specify it in pr description?", "action": false, "timestamp": "2020-04-16T19:13:29+00:00" }, { "id": "e11da00806074e68a49f3d2b3db213cb", "sender": "wumpus", "payload": "yes, good point", "action": false, "timestamp": "2020-04-16T19:13:32+00:00" }, { "id": "070631ba878b4a05a7565d015aa473ab", "sender": "jonatack", "payload": "i think i ran into it when win gitian signing with docker", "action": false, "timestamp": "2020-04-16T19:13:51+00:00" }, { "id": "81145007227941c7b82e36cf03420f68", "sender": "achow101", "payload": "hebasto: done", "action": false, "timestamp": "2020-04-16T19:14:17+00:00" }, { "id": "063f25ae7f954650905dcb0cab1d4ae8", "sender": "wumpus", "payload": "makes sense", "action": false, "timestamp": "2020-04-16T19:14:19+00:00" }, { "id": "6c90e9cbd57445d39082c6b4b4722182", "sender": "wumpus", "payload": "luke-jr: I think there have been some disagreements about your PRs, meshcollider was concerned about #18550 for example", "action": false, "timestamp": "2020-04-16T19:15:46+00:00" }, { "id": "f5fea06f668d49779bcf433081e0609c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18550 | Store destdata for change in separate key for backward compatibility by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18550 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:15:47+00:00" }, { "id": "2bdd3e4fd192452c8c2f5f7a3f907b12", "sender": "wumpus", "payload": "it's probably too late in the release cycle to do changes like that", "action": false, "timestamp": "2020-04-16T19:16:11+00:00" }, { "id": "ea109748cc9d43e7846acde4c81fa761", "sender": "luke-jr", "payload": "wumpus: are we changing to that topic now? I can wait a bit", "action": false, "timestamp": "2020-04-16T19:16:14+00:00" }, { "id": "faa36d0a1dad4f6d9a2943dc74817a4f", "sender": "luke-jr", "payload": "it's a bugfix", "action": false, "timestamp": "2020-04-16T19:16:32+00:00" }, { "id": "b54169875a2e4092ad01b44cd0f9a4e8", "sender": "MarcoFalke", "payload": "What bug is it fixing? You didn't include any regression tests", "action": false, "timestamp": "2020-04-16T19:17:24+00:00" }, { "id": "dae5bbbcfd3b4e839659fdeae588a584", "sender": "meshcollider", "payload": "ryanofsky should really be here for this discussion too", "action": false, "timestamp": "2020-04-16T19:17:33+00:00" }, { "id": "6aa1e36449ee4e9c8664f8f526d95b32", "sender": "ryanofsky", "payload": "here", "action": false, "timestamp": "2020-04-16T19:17:41+00:00" }, { "id": "31a74680675b4ad5868ba05f99bf8e80", "sender": "luke-jr", "payload": "MarcoFalke: I don't see how to make tests for this kind of thing", "action": false, "timestamp": "2020-04-16T19:17:46+00:00" }, { "id": "eeb8881b5a9b46858a0cd74539edc0d3", "sender": "ryanofsky", "payload": "My summary of the destdata issue is https://github.com/bitcoin/bitcoin/pull/18550#issuecomment-612672886", "action": false, "timestamp": "2020-04-16T19:18:14+00:00" }, { "id": "155fc968a32549329e924f0698055d29", "sender": "wumpus", "payload": "so is it a regression in 0.20?", "action": false, "timestamp": "2020-04-16T19:18:16+00:00" }, { "id": "c86f702108334580bc12466e5432c81d", "sender": "MarcoFalke", "payload": "Is this a bug that users can observe or is it only a developer issue?", "action": false, "timestamp": "2020-04-16T19:18:41+00:00" }, { "id": "ba742b7cbc564489ac480a5c768619d1", "sender": "luke-jr", "payload": "it's a wallet format issue", "action": false, "timestamp": "2020-04-16T19:19:09+00:00" }, { "id": "4c42f7747202461e9eb2f7de0b048b22", "sender": "luke-jr", "payload": "if we don't fix it now, it complicates backward compatibility when we do fix it later", "action": false, "timestamp": "2020-04-16T19:19:21+00:00" }, { "id": "f50cf841654a4ee9a357cd4be74b3716", "sender": "wumpus", "payload": "yes, if it's hard to write a test for and it's not a GUI issue that usually means it's very hard to observe", "action": false, "timestamp": "2020-04-16T19:19:26+00:00" }, { "id": "7b02d97c3ff943c6bc9cb7878568fc5c", "sender": "luke-jr", "payload": "it's hard to write a test, because the breaking is backward compatibility", "action": false, "timestamp": "2020-04-16T19:19:40+00:00" }, { "id": "2bbc36b59fb04d348d0f552f2eada7d6", "sender": "luke-jr", "payload": "so we'd need a way to run an old version", "action": false, "timestamp": "2020-04-16T19:19:49+00:00" }, { "id": "017458a29f714720bf574e1da729c059", "sender": "MarcoFalke", "payload": "the python tests can do that", "action": false, "timestamp": "2020-04-16T19:20:06+00:00" }, { "id": "9b248dcec49a483487ddadf4c313d726", "sender": "meshcollider", "payload": "It's not a new issue, the actual issue itself was fixed in #18192", "action": false, "timestamp": "2020-04-16T19:20:08+00:00" }, { "id": "ac1a8a5488bc4bc3ba309847feefb638", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:20:10+00:00" }, { "id": "84189115f9e842c7bc5d473c205a91e8", "sender": "luke-jr", "payload": "\"destdata\" was added in a manner that it only supported non-change", "action": false, "timestamp": "2020-04-16T19:20:11+00:00" }, { "id": "3c0b014ea96f43c2ae8321d0d5dc6817", "sender": "ryanofsky", "payload": "Luke, I'm not sure what the exact issue you are concerned about, your PR changes behavior in lots of ways, most of them bad", "action": false, "timestamp": "2020-04-16T19:20:20+00:00" }, { "id": "6c267172b20b4af0a38f67f14489e721", "sender": "luke-jr", "payload": "but now we suddenly need to support it for change too", "action": false, "timestamp": "2020-04-16T19:20:26+00:00" }, { "id": "78896777292440c8a752079e2cf84014", "sender": "wumpus", "payload": "destdata has been in there since 2012 or so isn't it", "action": false, "timestamp": "2020-04-16T19:20:28+00:00" }, { "id": "5b2dcfdfd45e481fa2fd338127ee3f2d", "sender": "luke-jr", "payload": "the problem remaining is that the fix in #18192 breaks compatibility", "action": false, "timestamp": "2020-04-16T19:20:37+00:00" }, { "id": "f520548edd2b4a4f86fac68f3a710930", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:20:40+00:00" }, { "id": "2b99f76cc997462088d2e81024ffbc0e", "sender": "sipa", "payload": "luke-jr: can you give an exact scenario for reproducing the issue?", "action": false, "timestamp": "2020-04-16T19:20:53+00:00" }, { "id": "35246c8acc3e40d0b944154496db728a", "sender": "luke-jr", "payload": "wumpus: yes, 0.9", "action": false, "timestamp": "2020-04-16T19:20:55+00:00" }, { "id": "a6978d8ebb7c4886a17c17d89f21b3c8", "sender": "luke-jr", "payload": "sipa: set a destdata on change, then use the wallet in an older version", "action": false, "timestamp": "2020-04-16T19:21:14+00:00" }, { "id": "c19596ffa573427cbd57dba7f0043618", "sender": "sipa", "payload": "assuming no further changes related to the topic go into 0.20", "action": false, "timestamp": "2020-04-16T19:21:18+00:00" }, { "id": "050b5890236c4b97ac2b3fdecf12d791", "sender": "wumpus", "payload": "why does it now suddenly need to be changed before 0.20", "action": false, "timestamp": "2020-04-16T19:21:25+00:00" }, { "id": "a5c571e048cf4c68a5f85422cd787811", "sender": "sipa", "payload": "what does a user do to cause \"set a destdata on change\" ?", "action": false, "timestamp": "2020-04-16T19:21:31+00:00" }, { "id": "bce726688768458990f33e17d8909448", "sender": "ryanofsky", "payload": "luke-jr, that is a pre-existing bug in 0.19", "action": false, "timestamp": "2020-04-16T19:21:33+00:00" }, { "id": "e0c7e7e5f8f242ba8dd21dd9483ad03f", "sender": "luke-jr", "payload": "sipa: currently, it is possible only be using an avoid-reuse wallet", "action": false, "timestamp": "2020-04-16T19:21:54+00:00" }, { "id": "f31bc65da7274d258fb708966f273549", "sender": "ryanofsky", "payload": "it already sets destdata on change and then starts treating it as nonchange", "action": false, "timestamp": "2020-04-16T19:21:55+00:00" }, { "id": "bef7249e01fc4786b9297ed505ace34c", "sender": "luke-jr", "payload": "only by*", "action": false, "timestamp": "2020-04-16T19:21:58+00:00" }, { "id": "96fe0139b6a640efaca77e84a26c6dd1", "sender": "achow101", "payload": "luke-jr: IMO it isn't worth trying to fix this display bug with users who upgrade then downgrade", "action": false, "timestamp": "2020-04-16T19:22:07+00:00" }, { "id": "21717772f81444a7ac97a5d6ab2927ab", "sender": "MarcoFalke", "payload": "I'd say that anything that is not a regression can wait for 0.20.1 or 0.21.0", "action": false, "timestamp": "2020-04-16T19:22:22+00:00" }, { "id": "ec3f5ea206ef43158164ced491ab24f5", "sender": "luke-jr", "payload": "achow101: it will come back when downgrading 0.21 to 0.20", "action": false, "timestamp": "2020-04-16T19:22:24+00:00" }, { "id": "4a138fa728fd4afaa783770a4765591c", "sender": "ryanofsky", "payload": "it'd be easy to backport a trivial fix for that issue in the 0.19 branch, but this PR only makes the bug and introduces other bugs", "action": false, "timestamp": "2020-04-16T19:22:27+00:00" }, { "id": "a05e370256c64316a76ae391687652a5", "sender": "achow101", "payload": "because it's already an issue for those users using the old software, this isn't a regression", "action": false, "timestamp": "2020-04-16T19:22:27+00:00" }, { "id": "8e10376911ab4d67821d23b1a2cf3465", "sender": "luke-jr", "payload": "unless we specifically exclude \"used\" from the fix in 0.21", "action": false, "timestamp": "2020-04-16T19:22:34+00:00" }, { "id": "8cead84e269c4f0aa8d1bfe575f0c36e", "sender": "ryanofsky", "payload": "*masks the bug", "action": false, "timestamp": "2020-04-16T19:22:38+00:00" }, { "id": "2166a99d98b4439298ab46a35248e0d9", "sender": "sipa", "payload": "what is the impact? is it just the change address showing up in the address book, or more?", "action": false, "timestamp": "2020-04-16T19:22:49+00:00" }, { "id": "79de3ef00b05410f8ca42ad6f25f5535", "sender": "luke-jr", "payload": "sipa: just that, I think", "action": false, "timestamp": "2020-04-16T19:23:06+00:00" }, { "id": "0216e793d861482fbfb6181149e85485", "sender": "luke-jr", "payload": "(which can be serious IMO)\\", "action": false, "timestamp": "2020-04-16T19:23:18+00:00" }, { "id": "338b2c179fdf42eaa951768f9a629d0f", "sender": "achow101", "payload": "sipa: it may have an effect on coin selection", "action": false, "timestamp": "2020-04-16T19:23:23+00:00" }, { "id": "9b1e526820484fbfbd465ce0b699bd1f", "sender": "achow101", "payload": "in particular trusted change vs untrusted non-change", "action": false, "timestamp": "2020-04-16T19:23:33+00:00" }, { "id": "ef4311496a8f4b2bb3eccb7ac0355f9c", "sender": "luke-jr", "payload": "achow101: well, if it's spent\u00c3\u00a2\u00c2\u0080\u00c2\u00a6", "action": false, "timestamp": "2020-04-16T19:23:38+00:00" }, { "id": "0abca7394ace4c72a49f82904400258f", "sender": "sipa", "payload": "my understanding is that it does not affect IsChange, unless a user also goes to set an explicit label on one of those (now shown) change addresses?", "action": false, "timestamp": "2020-04-16T19:24:01+00:00" }, { "id": "ae62151aca1b4bfaae2f715279c03247", "sender": "wumpus", "payload": "who uses the address book for receiving addresses anyhow", "action": false, "timestamp": "2020-04-16T19:24:09+00:00" }, { "id": "3dd2db0c85074e0e91ea9e4e05a9a3c4", "sender": "luke-jr", "payload": "sipa: it does", "action": false, "timestamp": "2020-04-16T19:24:10+00:00" }, { "id": "fab6054ba12b4bd6878aa531b687d21b", "sender": "luke-jr", "payload": "sipa: when the change is used, it gets a \"\" label", "action": false, "timestamp": "2020-04-16T19:24:20+00:00" }, { "id": "313ca1cbfc57410b9de7512eac05a09b", "sender": "sipa", "payload": "i see", "action": false, "timestamp": "2020-04-16T19:24:27+00:00" }, { "id": "dedcb572f57441d8af9f0a27d3eff620", "sender": "luke-jr", "payload": "wumpus: uh, everyone?", "action": false, "timestamp": "2020-04-16T19:24:29+00:00" }, { "id": "17cc7a24489d48c3a4224fb8ce06716b", "sender": "sipa", "payload": "i certainly don't, but i don't think that's an important question; if we have an address book, it should work", "action": false, "timestamp": "2020-04-16T19:25:00+00:00" }, { "id": "926b1a8eb88f449b9b967d7ac400dc2b", "sender": "wumpus", "payload": "luke-jr: I must admit I haven't used the address book *at all* for years, but last time I did it was to check an address I sent to", "action": false, "timestamp": "2020-04-16T19:25:02+00:00" }, { "id": "858d82c6576443999837d4319dc46c48", "sender": "luke-jr", "payload": "sipa: this gets worse if we add any new features using destdata - for example, marking which destination are actually used to provide warnings on reuse", "action": false, "timestamp": "2020-04-16T19:25:09+00:00" }, { "id": "ddd75ade1f274a3bb4fa1a790a375029", "sender": "ryanofsky", "payload": "this is right but it's a preexisting bug, 0.19 already marks addresses used and has the same bug", "action": false, "timestamp": "2020-04-16T19:25:10+00:00" }, { "id": "43427a3e865a4b179fcbdbc49d0efa2f", "sender": "achow101", "payload": "CAddressBook effects more than just the address book IIRC", "action": false, "timestamp": "2020-04-16T19:25:17+00:00" }, { "id": "c01fbcd178644baa8816b86403aed12f", "sender": "luke-jr", "payload": "wumpus: how do you even receive without using it now?", "action": false, "timestamp": "2020-04-16T19:25:27+00:00" }, { "id": "b1b4c4cadaa24494b5e0fc64ace1118c", "sender": "sipa", "payload": "it affecting IsChange is more concerning to me", "action": false, "timestamp": "2020-04-16T19:25:35+00:00" }, { "id": "714013beac1848a0a003cad12cd5f648", "sender": "wumpus", "payload": "luke-jr: create new receiving address in the receive tab, give it out", "action": false, "timestamp": "2020-04-16T19:25:47+00:00" }, { "id": "868e4b27d3bd428798171106cddcd3d2", "sender": "luke-jr", "payload": "wumpus: that uses the address book..", "action": false, "timestamp": "2020-04-16T19:25:54+00:00" }, { "id": "91cfc9b9da0f4273a47669e3eb66f5f7", "sender": "wumpus", "payload": "no, it doesn't, the 'payment request' table is separate", "action": false, "timestamp": "2020-04-16T19:26:09+00:00" }, { "id": "e7980ab2c5d44ad89ceb38081aeb7a82", "sender": "sipa", "payload": "ok, this is a semantics discussion; this is not productive", "action": false, "timestamp": "2020-04-16T19:26:18+00:00" }, { "id": "a69b8bf4f851482896e60b4933160eed", "sender": "MarcoFalke", "payload": "we should just remove the wallet and gui. Doesn't that solve the problem?", "action": false, "timestamp": "2020-04-16T19:26:28+00:00" }, { "id": "0b20f11f0a9c43c8baad373d51f36eb5", "sender": "luke-jr", "payload": "-.-", "action": false, "timestamp": "2020-04-16T19:26:34+00:00" }, { "id": "ff8346b6a47d4a27808c2a520462cd71", "sender": "luke-jr", "payload": "ryanofsky was proposing removing destdata entirely, but that breaks compatibility in new ways, and loses a big feature IMO", "action": false, "timestamp": "2020-04-16T19:26:54+00:00" }, { "id": "6f6dea484bb44d838497e33f6c71dc0b", "sender": "ryanofsky", "payload": "no, that is not my proposal", "action": false, "timestamp": "2020-04-16T19:27:08+00:00" }, { "id": "31223bde46f840c2aa95ff49172922d9", "sender": "ryanofsky", "payload": "my proposal is a refactoring that does not change behavior in any way", "action": false, "timestamp": "2020-04-16T19:27:21+00:00" }, { "id": "571d2388e7ae49828be0775e09ca58c7", "sender": "ryanofsky", "payload": "and is meant to prevent bugs like this in the future, but it's basically orthogonal to what we are talking about", "action": false, "timestamp": "2020-04-16T19:27:36+00:00" }, { "id": "48bbb76484824df6b84c8888e438d7a5", "sender": "achow101", "payload": "I think the question is whether we will try to fix this in the future in a way that allows downgrading to 0.19", "action": false, "timestamp": "2020-04-16T19:27:39+00:00" }, { "id": "3c860d2e12804326a825674d155eab6a", "sender": "ryanofsky", "payload": "i described the bug https://github.com/bitcoin/bitcoin/pull/18550#issuecomment-612672886", "action": false, "timestamp": "2020-04-16T19:27:51+00:00" }, { "id": "6c75b7867e0e4ab582f5dafa6644d214", "sender": "luke-jr", "payload": "achow101: to 0.20*", "action": false, "timestamp": "2020-04-16T19:27:51+00:00" }, { "id": "c37cd455a0ef4120a54b3b1ff0fabbbc", "sender": "sipa", "payload": "so am i correct that this triggers only if a user on 0.19 has avoid-reuse on, and then uses 0.19 or 0.20?", "action": false, "timestamp": "2020-04-16T19:27:54+00:00" }, { "id": "26b1285bedc4423788d0b8327f1b406c", "sender": "ryanofsky", "payload": "and have 3 solutions for dealing with it", "action": false, "timestamp": "2020-04-16T19:27:57+00:00" }, { "id": "764732aa5519437995767888de0f8cc4", "sender": "wumpus", "payload": "destdata was mainly introduced to store payment request data and such, if it's no longer needed it should be removed", "action": false, "timestamp": "2020-04-16T19:27:58+00:00" }, { "id": "4117231f6d994953b8927dc5855d266d", "sender": "wumpus", "payload": "but not for 0.20", "action": false, "timestamp": "2020-04-16T19:27:59+00:00" }, { "id": "2179af21aef549ecb3391fbfbfd67b27", "sender": "ryanofsky", "payload": "my PR is separate and compatible with all 3 approaches", "action": false, "timestamp": "2020-04-16T19:28:07+00:00" }, { "id": "64e66937c5544431996eb7726809a793", "sender": "sipa", "payload": "or also when they use 0.20 and then downgrades in certain cases?", "action": false, "timestamp": "2020-04-16T19:28:21+00:00" }, { "id": "fb06ca20c0d14a26884fb63f3441b7c2", "sender": "luke-jr", "payload": "sipa: it's unfixable for already-released 0.19; my hope is to fix it so downgrading to 0.20 works", "action": false, "timestamp": "2020-04-16T19:28:27+00:00" }, { "id": "df1813d6048e4015821028cea0c88d3f", "sender": "luke-jr", "payload": "wumpus: it's still needed", "action": false, "timestamp": "2020-04-16T19:28:40+00:00" }, { "id": "75f20c4ac3724a49b64e4052765b62ed", "sender": "luke-jr", "payload": "destdata is nice because you don't need to upgrade wallets to get features using new metadata", "action": false, "timestamp": "2020-04-16T19:28:59+00:00" }, { "id": "0400b51a6204407b956c4ae527d72b34", "sender": "achow101", "payload": "to be clear, is the issue *only* with downgrading?", "action": false, "timestamp": "2020-04-16T19:29:07+00:00" }, { "id": "08b48eb57644408baed1d2624c57a60f", "sender": "luke-jr", "payload": "achow101: yes, which is something we've always tried to support for wallets", "action": false, "timestamp": "2020-04-16T19:29:22+00:00" }, { "id": "67a0269154224bbca2296c4f9d38cc2c", "sender": "tryphe", "payload": "address book should be removed outright imo, metadeta that can be modified in a locked wallet is kind of silly", "action": false, "timestamp": "2020-04-16T19:29:23+00:00" }, { "id": "54353849ba464fb0a770acb83fd407cd", "sender": "achow101", "payload": "If we didn't care about dowgrading at all, this would not be an issue?", "action": false, "timestamp": "2020-04-16T19:29:30+00:00" }, { "id": "3e716ab5aeb843649b9a1f6036ea0cdf", "sender": "ryanofsky", "payload": "the issue is not only with downgrading, but the fix only deals with downgrading, and is only partial", "action": false, "timestamp": "2020-04-16T19:29:34+00:00" }, { "id": "86593059850c457fa9367988ff991e76", "sender": "ryanofsky", "payload": "and it introduces other bug of avoding-reuse feature in 0.19 being broken", "action": false, "timestamp": "2020-04-16T19:29:58+00:00" }, { "id": "9b79cc7fd47847e4901d038d29cc37e7", "sender": "bitcoin-git", "payload": "[bitcoin] jnewbery opened pull request #18675: tests: Don't initialize PrecomputedTransactionData in txvalidationcache tests (master...2020-04-precomputedtransactiondata-txvalidationcache) https://github.com/bitcoin/bitcoin/pull/18675", "action": false, "timestamp": "2020-04-16T19:30:13+00:00" }, { "id": "76aab445aa1d4e4583afdea0dd7be006", "sender": "achow101", "payload": "ryanofsky: what's your pr?", "action": false, "timestamp": "2020-04-16T19:30:27+00:00" }, { "id": "8315569b6e2e46f99660b59850aa78dd", "sender": "sipa", "payload": "can someone give me an exact user scenario that results in a user observing something incorrect, where they start with 0.20.0rc1 as we have now (and then do thing, or downgrade, or ...)", "action": false, "timestamp": "2020-04-16T19:30:39+00:00" }, { "id": "5f7ce4ee6fbc40818f45a1a61a62ebb3", "sender": "ryanofsky", "payload": "again my pr is orthogonal, refactoring only #18608, comptaible with any dataformat we chose now or in the future", "action": false, "timestamp": "2020-04-16T19:31:23+00:00" }, { "id": "e4087c5b2f6c4045ae86ea0d4e53dae6", "sender": "wumpus", "payload": "tryphe: I think that's unrelated; wallet encryption in bitcoin core only protects private keys, nothing more. Labels and such are not part of that either and should definitely be kept.", "action": false, "timestamp": "2020-04-16T19:31:23+00:00" }, { "id": "5ad9b961ab524429a1bd518e7c8b8419", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18608 | refactor: Remove CAddressBookData::destdata by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #18608 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:31:24+00:00" }, { "id": "50977c64dd6c404d80a1fc53ab0a1a95", "sender": "luke-jr", "payload": "sipa: create avoid-reuse wallet; send a transaction that has change; spend that change; suddenly that change is no longer change", "action": false, "timestamp": "2020-04-16T19:32:07+00:00" }, { "id": "3c0f18e900c34523a9beb4ef89fae05a", "sender": "ryanofsky", "payload": "the scenario is just a change address gets used, and 0.19 software treats all addresses marked as used as nonchange", "action": false, "timestamp": "2020-04-16T19:32:08+00:00" }, { "id": "3ff3d54f2ecb45aa8bca9b0505a2dd63", "sender": "ryanofsky", "payload": "this is a pre-existing scenario that has been around since 0.19", "action": false, "timestamp": "2020-04-16T19:32:21+00:00" }, { "id": "e17e497bd80744a6a1c266606aaa5f77", "sender": "ryanofsky", "payload": "and the only way to fix it reliably is with backports", "action": false, "timestamp": "2020-04-16T19:32:29+00:00" }, { "id": "6b111fdce04849579d34edce2fc3d7ba", "sender": "luke-jr", "payload": "sipa: the last step is only in 0.19", "action": false, "timestamp": "2020-04-16T19:32:30+00:00" }, { "id": "1b7a34f54b514129a6b68b4d3deccb1d", "sender": "luke-jr", "payload": "(the result)", "action": false, "timestamp": "2020-04-16T19:32:34+00:00" }, { "id": "f11d4fe6be5a4e5d87d6dda472ace577", "sender": "luke-jr", "payload": "sipa: but if we fix this in 0.21, 0.20 would no longer see the \"used\" flag unless we special-case it", "action": false, "timestamp": "2020-04-16T19:32:53+00:00" }, { "id": "bc87a9bb4d41425f9f30df36401e273e", "sender": "sipa", "payload": "luke-jr: that very much depends on how it is fixed", "action": false, "timestamp": "2020-04-16T19:33:06+00:00" }, { "id": "5678915b0c544935824e10c857e4295f", "sender": "ryanofsky", "payload": "luke's pr is a partial fix for the the issue because it stops marking addresses as used in a way 0.19 understands, and marks them used in a different way it is blind to", "action": false, "timestamp": "2020-04-16T19:33:14+00:00" }, { "id": "78eb379580b0483ba7966780af2586cb", "sender": "sipa", "payload": "my concern right now is behavior that is observable with 0.20", "action": false, "timestamp": "2020-04-16T19:33:17+00:00" }, { "id": "dbd742ac2eda420393fbff6b4aae5c8a", "sender": "ryanofsky", "payload": "there are 0 bugs observable with 0.20", "action": false, "timestamp": "2020-04-16T19:33:28+00:00" }, { "id": "7a8ef7dbc736493b8ab3e82a2635eb99", "sender": "luke-jr", "payload": "I don't see a way to fix it later, without special-casing 0.20 support", "action": false, "timestamp": "2020-04-16T19:33:29+00:00" }, { "id": "03f33c28195143c48d14ee515321ed80", "sender": "wumpus", "payload": "tbh if it's only a minor backwards compatibility issue that is only apparent in downgrading, I wouldn't want to do things substantially different because of it", "action": false, "timestamp": "2020-04-16T19:34:03+00:00" }, { "id": "29dfccb67e7d4962876c5ea8b1b2f2cd", "sender": "tryphe", "payload": "wumpus, mostly agree, although allowing modification of metadata of someone's wallet while in cold storage that they might assume stays constant opens up a lot of doors for bad actors", "action": false, "timestamp": "2020-04-16T19:34:05+00:00" }, { "id": "7c412e982d954362b5b7e24bdde096fc", "sender": "achow101", "payload": "How about we revert the \"fix\" in 0.20 and try again for 0.21? It seems like ~no one has complained about this bug in 0.19", "action": false, "timestamp": "2020-04-16T19:34:09+00:00" }, { "id": "8255a078508b46b797622fffecee0663", "sender": "ryanofsky", "payload": "the fix in 0.20 is correct and causes no backwards compatiability issues", "action": false, "timestamp": "2020-04-16T19:34:26+00:00" }, { "id": "b81e609d88be4e03bef5a6afb05ec6b7", "sender": "meshcollider", "payload": "I don't think that's a good idea, the fix is still an improvement", "action": false, "timestamp": "2020-04-16T19:34:29+00:00" }, { "id": "6d97e6763aac44d4bfde3e0c8a2b72e8", "sender": "sipa", "payload": "luke-jr: if after that scenario (create avoid-reuse in 0.20; create change in 0.20; spend change in 0.20; downgrade to 0.19; do things), the user upgrades again to 0.20, is the issue resolved, or might it persist?", "action": false, "timestamp": "2020-04-16T19:34:29+00:00" }, { "id": "fde6841a38ca41f6a800b1144a332637", "sender": "luke-jr", "payload": "achow101: leaving it as-is now is strictly less bad than reverting I think", "action": false, "timestamp": "2020-04-16T19:34:39+00:00" }, { "id": "8ad4752f1b2a4ea4873443d54c47098f", "sender": "sipa", "payload": "tryphe: please stay on topic", "action": false, "timestamp": "2020-04-16T19:34:43+00:00" }, { "id": "2bf7c0ae4fcf4b279ae261913e40bb5b", "sender": "wumpus", "payload": "tryphe: if that's your concern, encrypt the entire wallet file or store it on an encrypted file system, it's not something bitcoin core's wallet encyrption solves", "action": false, "timestamp": "2020-04-16T19:34:44+00:00" }, { "id": "6983f2ef43ac436aa08e5ab97e663170", "sender": "ryanofsky", "payload": "the fix causes no issues whatsoever", "action": false, "timestamp": "2020-04-16T19:34:46+00:00" }, { "id": "05020a0419db497daf773369d8f0fa59", "sender": "ryanofsky", "payload": "i mean the fix that's already been merged causes no issues, the new fix proposed is a different story", "action": false, "timestamp": "2020-04-16T19:35:07+00:00" }, { "id": "e029b8d2d8774323849a89b6ef7ccef1", "sender": "luke-jr", "payload": "sipa: the issue remains resolved in 0.20, provided the user doesn't see it in 0.19 and set a label or something", "action": false, "timestamp": "2020-04-16T19:35:14+00:00" }, { "id": "2fce875f4ef44f56adb5c9f5ff8d9692", "sender": "sipa", "payload": "luke-jr: thank you", "action": false, "timestamp": "2020-04-16T19:35:29+00:00" }, { "id": "2c5eaa85d1f548d7b9e5fede90742346", "sender": "luke-jr", "payload": "(in fact, I think 0.19 making the broken state itself, would also appear correct in 0.20)", "action": false, "timestamp": "2020-04-16T19:35:57+00:00" }, { "id": "e32fd724c9af43f99e93b4ee36c20492", "sender": "wumpus", "payload": "okay", "action": false, "timestamp": "2020-04-16T19:36:07+00:00" }, { "id": "d0b9b4de73444d2e83a2e49f6b632731", "sender": "sipa", "payload": "that's good to hear as well", "action": false, "timestamp": "2020-04-16T19:36:08+00:00" }, { "id": "d3a3f752ef65433783666d1e84df0f0b", "sender": "wumpus", "payload": "yes, that's good to hear", "action": false, "timestamp": "2020-04-16T19:36:16+00:00" }, { "id": "69048bf947194b2480f7ffa43faa2df1", "sender": "sipa", "payload": "i think we should document the issue in the release notes, with the point that if the issue appears, upgrading (again) to 0.20 will fix things unless a user manually set a label on an errorneously-shown address in 0.19", "action": false, "timestamp": "2020-04-16T19:36:54+00:00" }, { "id": "6237a4d5a63f4b698238921b54aea2fa", "sender": "luke-jr", "payload": "my biggest \"end result\" concern, is that I don't think users should need to -upgradewallet to get address reuse warnings when we finally merge that ; but that's a few steps into the future", "action": false, "timestamp": "2020-04-16T19:37:03+00:00" }, { "id": "8b828bcee31a4a828f6a472d0e3e864c", "sender": "sipa", "payload": "for 0.21, we can discuss solutions - whether those involved -upgradewallet or special-casing the 0.20 behavior", "action": false, "timestamp": "2020-04-16T19:37:29+00:00" }, { "id": "caaae7e24a8d4e94aa7b959347293989", "sender": "achow101", "payload": "luke-jr: with what is currently merged, why is changing the fix necessary?", "action": false, "timestamp": "2020-04-16T19:37:30+00:00" }, { "id": "5c2c9e84a98b4a409172d5d92e1c0d5c", "sender": "achow101", "payload": "as in why is your proposed 0.21 fix necessary", "action": false, "timestamp": "2020-04-16T19:37:55+00:00" }, { "id": "3b0b0b0169bf4466832bafedf2ec475a", "sender": "luke-jr", "payload": "achow101: it's one thing to break compatibility with <0.19 only for avoid-reuse wallets>, another entirely to break compatibiltiy with ", "action": false, "timestamp": "2020-04-16T19:38:04+00:00" }, { "id": "2957210c789745598e0c6dade3a45e5d", "sender": "luke-jr", "payload": "achow101: since \"used\" destdata is only in avoid-reuse, we have the former situation right now", "action": false, "timestamp": "2020-04-16T19:38:25+00:00" }, { "id": "c034d53564804ffdaf83c109a28f15d6", "sender": "luke-jr", "payload": "but as soon as we add destdata for change on normal wallets, we get the latter", "action": false, "timestamp": "2020-04-16T19:38:35+00:00" }, { "id": "d63b2f708d95406f9f964649b6b3eea2", "sender": "achow101", "payload": "are we adding destdata for change on normal wallets?", "action": false, "timestamp": "2020-04-16T19:39:02+00:00" }, { "id": "ec0f135e922c4e359e325e8c75f2991c", "sender": "luke-jr", "payload": "achow101: that's my plan for address reuse warnings", "action": false, "timestamp": "2020-04-16T19:39:12+00:00" }, { "id": "f5b2f31a82b649b797899913a3011a4f", "sender": "ryanofsky", "payload": "achow101, yes, we've been doing that since kallewoof implemented avoidreuse", "action": false, "timestamp": "2020-04-16T19:39:23+00:00" }, { "id": "a83c2d16618841db87391f97708e77fe", "sender": "wumpus", "payload": "just a reminder that we have to reserve some time for MarcoFalke's topic as well", "action": false, "timestamp": "2020-04-16T19:39:32+00:00" }, { "id": "8e1f7d51eca4424ab20827f28fca07b8", "sender": "jonatack", "payload": "i agree with sipa's proposals (and with meshcollider and ryanofsky that it's better to keep fix in luke-jr's merged pr)", "action": false, "timestamp": "2020-04-16T19:39:39+00:00" }, { "id": "887004db374e4959abaf334d03b72bcd", "sender": "luke-jr", "payload": "that's how it can avoid needing a -walletupgrade", "action": false, "timestamp": "2020-04-16T19:39:41+00:00" }, { "id": "e95dc64dfea24cbdbbbb202f6ec8ddf1", "sender": "achow101", "payload": "ryanofsky: that's only for avoid reuse", "action": false, "timestamp": "2020-04-16T19:39:53+00:00" }, { "id": "d5bc04bf3b034f62addac0fc7bc12128", "sender": "achow101", "payload": "luke-jr: i see, so what if we don't do that?", "action": false, "timestamp": "2020-04-16T19:40:00+00:00" }, { "id": "a71144be2d2540f1843643abe0563089", "sender": "achow101", "payload": "and just don't change it", "action": false, "timestamp": "2020-04-16T19:40:06+00:00" }, { "id": "7d8775649d664171a2668f55507b501b", "sender": "luke-jr", "payload": "achow101: I don't understand what you mean", "action": false, "timestamp": "2020-04-16T19:40:18+00:00" }, { "id": "518099d7441c44bb80a1b042e3d52723", "sender": "luke-jr", "payload": "if we don't add address reuse warnigns, we don't have address reuse warnings..", "action": false, "timestamp": "2020-04-16T19:40:32+00:00" }, { "id": "25539efc5dfc432093ddb376278795f9", "sender": "achow101", "payload": "why do we have to add destdata on change for normal wallets", "action": false, "timestamp": "2020-04-16T19:40:46+00:00" }, { "id": "8828fc29024f48c6bdc0587a31b58065", "sender": "luke-jr", "payload": "to mark the change address as used", "action": false, "timestamp": "2020-04-16T19:40:55+00:00" }, { "id": "e367fe60242645d1bdb6244a36ed1e8b", "sender": "achow101", "payload": "if the wallet doesn't signal avoidreuse, then there's no need?", "action": false, "timestamp": "2020-04-16T19:41:01+00:00" }, { "id": "444f14dd461549628c9d90ff26c5a24d", "sender": "luke-jr", "payload": "address reuse warnings are for all wallets", "action": false, "timestamp": "2020-04-16T19:41:16+00:00" }, { "id": "48847bd3c6b44f72b98e97313cd336c1", "sender": "sipa", "payload": "yeah, address reuse warnings != avoidreuse behavior", "action": false, "timestamp": "2020-04-16T19:41:36+00:00" }, { "id": "4d79d21247284943ba78ef7a4c29e61d", "sender": "achow101", "payload": "ok i see", "action": false, "timestamp": "2020-04-16T19:41:46+00:00" }, { "id": "b94c90ba1e0f466fb908232d612e02ec", "sender": "sipa", "payload": "but can't that be done with a different db record instead?", "action": false, "timestamp": "2020-04-16T19:41:48+00:00" }, { "id": "8b4083bd9f204dfa9d1ccfc413d762e5", "sender": "achow101", "payload": "there's the context I'm missing :)", "action": false, "timestamp": "2020-04-16T19:41:50+00:00" }, { "id": "3e42938d799d4b19ab7921e6b6379c32", "sender": "luke-jr", "payload": "sipa: that's a wallet format change, and needs -upgradewallet", "action": false, "timestamp": "2020-04-16T19:41:58+00:00" }, { "id": "e30176caab664bb596dc47eccf1b939b", "sender": "ryanofsky", "payload": "sipa, yes", "action": false, "timestamp": "2020-04-16T19:41:59+00:00" }, { "id": "4f0b68fc8d974f57b26052dfe401b800", "sender": "sipa", "payload": "luke-jr: why? old wallets would just ignore the record", "action": false, "timestamp": "2020-04-16T19:42:11+00:00" }, { "id": "317b7b780c654e8b9ca1f5c6a0603d2a", "sender": "ryanofsky", "payload": "from https://github.com/bitcoin/bitcoin/pull/18550#issuecomment-612672886", "action": false, "timestamp": "2020-04-16T19:42:24+00:00" }, { "id": "5424adf343b848ca95d791d1379047ed", "sender": "ryanofsky", "payload": "option 1 is keep using \"destdata\" record", "action": false, "timestamp": "2020-04-16T19:42:31+00:00" }, { "id": "1f0e151f1a894c57b39528700deb1d65", "sender": "ryanofsky", "payload": "option 2 is switch to \"changedata\" record", "action": false, "timestamp": "2020-04-16T19:42:40+00:00" }, { "id": "a3292945c41847ef983612a7ad27b286", "sender": "sipa", "payload": "also, can't the usage data be computed at runtime from other wallet data?", "action": false, "timestamp": "2020-04-16T19:42:45+00:00" }, { "id": "bc23a654233f4ea0b1e2b27a524809c7", "sender": "ryanofsky", "payload": "option 3 is \"useddata\" record", "action": false, "timestamp": "2020-04-16T19:42:45+00:00" }, { "id": "9aac19c996c34c9eaeef966a70270ca7", "sender": "luke-jr", "payload": "sipa: historical best practice?", "action": false, "timestamp": "2020-04-16T19:43:00+00:00" }, { "id": "ada9b7621d3a4c14ad6ac9fd72f3a738", "sender": "sipa", "payload": "?", "action": false, "timestamp": "2020-04-16T19:43:07+00:00" }, { "id": "ada86b2305ec4a9bb4aecdc3c002d8c7", "sender": "luke-jr", "payload": "sipa: maybe in this case (I haven't looked into that option yet, but I plan to), but this will probably come up again either way", "action": false, "timestamp": "2020-04-16T19:43:16+00:00" }, { "id": "a9c661d1f40244588de6e4e81427bbf8", "sender": "meshcollider", "payload": "Using a different record would be fine, only a minor code complication", "action": false, "timestamp": "2020-04-16T19:43:23+00:00" }, { "id": "cb4f600e4e254c3db18c88face32c88e", "sender": "luke-jr", "payload": "sipa: we've always tried to not change wallet format outside of an explicit -upgradewallet", "action": false, "timestamp": "2020-04-16T19:43:33+00:00" }, { "id": "2f7825f45c894812a656b0d7bd833c6f", "sender": "meshcollider", "payload": "I dont think it would come up again luke-jr", "action": false, "timestamp": "2020-04-16T19:43:34+00:00" }, { "id": "a7c81787ad394a47876987da7a56ce38", "sender": "luke-jr", "payload": "meshcollider: tax metadata for example", "action": false, "timestamp": "2020-04-16T19:43:49+00:00" }, { "id": "78d6a48aebd04b27b5ce533525d82e9e", "sender": "sipa", "payload": "luke-jr: it sounds to me like no wallet format change is needed *at all* for this functionality", "action": false, "timestamp": "2020-04-16T19:43:53+00:00" }, { "id": "2c46b78afbcf4bdc9eedc7de9b8bf3b2", "sender": "meshcollider", "payload": "Again use a different record, not destdata", "action": false, "timestamp": "2020-04-16T19:44:01+00:00" }, { "id": "94c8624d9f3e4b6fa000a4b95be963d3", "sender": "luke-jr", "payload": "meshcollider: what different record?", "action": false, "timestamp": "2020-04-16T19:44:12+00:00" }, { "id": "56798b23744642139c26904082bbf5ad", "sender": "sipa", "payload": "i think this is taking us too far", "action": false, "timestamp": "2020-04-16T19:44:43+00:00" }, { "id": "f0a4ad589df24f5ebc835db4957782f4", "sender": "luke-jr", "payload": "meshcollider: adding a new one is a wallet format change..", "action": false, "timestamp": "2020-04-16T19:44:51+00:00" }, { "id": "d0b329a54a004fb593c37b0ed1e0a5d2", "sender": "sipa", "payload": "so be it?", "action": false, "timestamp": "2020-04-16T19:45:06+00:00" }, { "id": "1e5da29e7ffd4fcfa213e8796f76c63f", "sender": "achow101", "payload": "luke-jr: we've added/changed records before without needing upgradewallet", "action": false, "timestamp": "2020-04-16T19:45:09+00:00" }, { "id": "ab6192869e0e4b9e8fe2584f0fbb6206", "sender": "wumpus", "payload": "please wrap up this topic", "action": false, "timestamp": "2020-04-16T19:45:16+00:00" }, { "id": "d0c4c3fb4ac7449c94e67816b16b4357", "sender": "meshcollider", "payload": "In a backwards compatible way if it's new", "action": false, "timestamp": "2020-04-16T19:45:19+00:00" }, { "id": "c37c18c38f3f4721aac96aecabba3386", "sender": "sipa", "payload": "if it's a record old wallets can just ignore, no need for upgradewallet", "action": false, "timestamp": "2020-04-16T19:45:26+00:00" }, { "id": "f0bec806d2ab456b8fa414440f1fb26e", "sender": "luke-jr", "payload": "achow101: we have?", "action": false, "timestamp": "2020-04-16T19:45:28+00:00" }, { "id": "b828b0560d42424bba66a85b911a7b4c", "sender": "sipa", "payload": "otherwise, use it", "action": false, "timestamp": "2020-04-16T19:45:29+00:00" }, { "id": "4d787801ceff46a5888544f1d624647b", "sender": "luke-jr", "payload": "ok then", "action": false, "timestamp": "2020-04-16T19:45:34+00:00" }, { "id": "bd6ddb02523b4ea9a3b9adf3e0f069f0", "sender": "meshcollider", "payload": "Yes the topic is done, the PRs are closed as noone else concept ACKs the change", "action": false, "timestamp": "2020-04-16T19:45:42+00:00" }, { "id": "4b771303ad1141d4a1c72ab429af74d9", "sender": "achow101", "payload": "luke-jr: see UpgradeKeyMetadata", "action": false, "timestamp": "2020-04-16T19:45:48+00:00" }, { "id": "5f50892570af45f98b5c1563a1b03146", "sender": "wumpus", "payload": "#topic experimental libmultiprocess, next steps for multiprocess in general (MarcoFalke)", "action": false, "timestamp": "2020-04-16T19:45:53+00:00" }, { "id": "6361724859484e87b588b3dc77d2775d", "sender": "MarcoFalke", "payload": "Background is that libmultiprocess has been added to ./depends , and libmultiprocess compile and link flags have been added to our makefile. Everything was opt-in but after merge discussion concluded that it was too early to do this step, so the change has been reverted. I (and ryanofsky) would like to hear and discuss everyone's concerns about libmultiprocess as part of the meeting for brainstorming.", "action": false, "timestamp": "2020-04-16T19:45:55+00:00" }, { "id": "34911929f06f4bb594f174945ae864f3", "sender": "MarcoFalke", "payload": "cc cfields", "action": false, "timestamp": "2020-04-16T19:46:09+00:00" }, { "id": "6b8f7cdc15e741b08c09c4e861ebc0cf", "sender": "cfields", "payload": "^^", "action": false, "timestamp": "2020-04-16T19:46:16+00:00" }, { "id": "e1942dd948694abea260e97b572fd1fe", "sender": "MarcoFalke", "payload": "cc fanquake (probably sleeping)", "action": false, "timestamp": "2020-04-16T19:46:22+00:00" }, { "id": "afbfcdcfd53a49029d81fce23c1f8151", "sender": "sipa", "payload": "my impression is that we should only merge changes for this if the plan is that eventually this will end up in release binaries", "action": false, "timestamp": "2020-04-16T19:46:34+00:00" }, { "id": "1684d53c487b474983926bfd68cc08f3", "sender": "sipa", "payload": "it's not clear to me if that's the case", "action": false, "timestamp": "2020-04-16T19:46:52+00:00" }, { "id": "2c05d0c969f84d4fbd2feaa7804b1ed1", "sender": "cfields", "payload": "fanquake listed a bunch of really good questions, and ryanofsky has given good answers to on #18588.", "action": false, "timestamp": "2020-04-16T19:46:57+00:00" }, { "id": "73bab8c1305d40948b65f1dfa8a4e897", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18588 | Revert \"Merge #16367: Multiprocess build support\" by MarcoFalke \u00c3\u0082\u00c2\u00b7 Pull Request #18588 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:46:58+00:00" }, { "id": "17c197fc680e4ad48a1b24a30bd01d76", "sender": "MarcoFalke", "payload": "sipa: Eventually that was the goal (at least as I understood it)", "action": false, "timestamp": "2020-04-16T19:46:59+00:00" }, { "id": "eae747d7805e4ae896cee30f5f8aa4fd", "sender": "ryanofsky", "payload": "that is the goal as far as i know", "action": false, "timestamp": "2020-04-16T19:47:12+00:00" }, { "id": "2e2d11f5cfef4530872da4da0c50866b", "sender": "MarcoFalke", "payload": "sipa: Though, there was no timeline for it", "action": false, "timestamp": "2020-04-16T19:47:14+00:00" }, { "id": "be284007b88b40338ee6f15eb69faa2e", "sender": "cfields", "payload": "sipa: yes, that was exactly my concern as well. The path forward isn't clear enough to be merging it in in-parts, imo.", "action": false, "timestamp": "2020-04-16T19:47:22+00:00" }, { "id": "4661e30823904ce2bc05b4019c45adfc", "sender": "wumpus", "payload": "sipa: yes, I think that's obvious", "action": false, "timestamp": "2020-04-16T19:47:23+00:00" }, { "id": "fef8869a701b4c1a9a7554d613a1e234", "sender": "sipa", "payload": "i haven't paid that much attention to the multiprocess project as i don't really care about it (and kind of dread pulling in extra dependencies like capnproto), but i assumed that it was something lots of people wanted - which is fine", "action": false, "timestamp": "2020-04-16T19:48:05+00:00" }, { "id": "e19a18c6cd3c4bd39dac0bd803936142", "sender": "wumpus", "payload": "though it's probably ok to have it experimental for a while before it ends up in release binaries", "action": false, "timestamp": "2020-04-16T19:48:15+00:00" }, { "id": "761856fa1fff4a7aa31102414682d537", "sender": "ryanofsky", "payload": "maybe relevant: it builds new binaries", "action": false, "timestamp": "2020-04-16T19:48:33+00:00" }, { "id": "50b4bc4d1b5c4a7789e15491c1f418df", "sender": "sipa", "payload": "wumpus: sure", "action": false, "timestamp": "2020-04-16T19:48:34+00:00" }, { "id": "4ea6397b4bd84fb888fdc7843fadf7d0", "sender": "wumpus", "payload": "I'm not really sure I like importing capnproto either, just now we got rid of google serialization thing", "action": false, "timestamp": "2020-04-16T19:49:06+00:00" }, { "id": "5df7872b0f49401188f287b548217d4b", "sender": "ryanofsky", "payload": "so a theoretical release could include existing binaries, alongside new multiprocess binaries that let you start and stop node/gui/wallet separately and connect each other", "action": false, "timestamp": "2020-04-16T19:49:14+00:00" }, { "id": "99034c23e4be4dffbf927b4a2f12cf59", "sender": "wumpus", "payload": "then again it's probably better than hand-rolling everything", "action": false, "timestamp": "2020-04-16T19:49:35+00:00" }, { "id": "5962ee4e89414dddb79b88530e620090", "sender": "MarcoFalke", "payload": "The multiprocess and capnproto will stay opt-in unless it is decided to change it that", "action": false, "timestamp": "2020-04-16T19:49:50+00:00" }, { "id": "cbdcbb7a7b7f453db74aa4b585919920", "sender": "luke-jr", "payload": "ryanofsky: can the same binary do single-process and multiprocess?", "action": false, "timestamp": "2020-04-16T19:50:06+00:00" }, { "id": "e43060a20f27417286ad80285f39f7f1", "sender": "MarcoFalke", "payload": "luke-jr: No, they are two binaries", "action": false, "timestamp": "2020-04-16T19:50:23+00:00" }, { "id": "9d87d75bda1d4e76a3710635fb7ec4f4", "sender": "luke-jr", "payload": "I mean hypothetically", "action": false, "timestamp": "2020-04-16T19:50:30+00:00" }, { "id": "d948591bd5e144299558ea31ad567012", "sender": "wumpus", "payload": "ryanofsky: I'm not sure I like that solution, couldn't we emulate the old way using the new binaries without shipping everything twice?", "action": false, "timestamp": "2020-04-16T19:50:42+00:00" }, { "id": "d64d931b172a47ebac039d733396fc90", "sender": "ryanofsky", "payload": "yes, I just didn't add the option", "action": false, "timestamp": "2020-04-16T19:50:47+00:00" }, { "id": "516a549ea145490eafa8be180e996026", "sender": "ryanofsky", "payload": "there is lots of flexibility here", "action": false, "timestamp": "2020-04-16T19:50:56+00:00" }, { "id": "85fcfcc27a284ac9af73062f07503a8b", "sender": "wumpus", "payload": "with the static builds we do that's expensive", "action": false, "timestamp": "2020-04-16T19:51:09+00:00" }, { "id": "5ee88e5fb818464687c9117eb57084c0", "sender": "ryanofsky", "payload": "I just built separate binaries because i meant I had to run ./configure less often", "action": false, "timestamp": "2020-04-16T19:51:15+00:00" }, { "id": "f64745360b604b5b9e6c5ced6ce416cb", "sender": "ryanofsky", "payload": "If we want unified binaries with a runtime options, that's easy too", "action": false, "timestamp": "2020-04-16T19:51:38+00:00" }, { "id": "3a7ddbe39001404cb9c2ceec5de245c9", "sender": "wumpus", "payload": "ah yes like busybox", "action": false, "timestamp": "2020-04-16T19:51:52+00:00" }, { "id": "e6ac60e13448438a928b2214919191b6", "sender": "luke-jr", "payload": "eg, bitcoin-qt gets what we have now; bitcoin-qt -process=foo gets just the foo part in MP mode", "action": false, "timestamp": "2020-04-16T19:52:02+00:00" }, { "id": "d961fae39fb54cd69c2056dd9f8fce04", "sender": "wumpus", "payload": "that kinda makes sense", "action": false, "timestamp": "2020-04-16T19:52:34+00:00" }, { "id": "65650ffe3c61473dbff4944f8ed4c46a", "sender": "ryanofsky", "payload": "these are really just packaging questions, my question is how to make progress on getting code reviewed", "action": false, "timestamp": "2020-04-16T19:53:29+00:00" }, { "id": "791c7266c72c437ba2d58b1b830a1b1f", "sender": "wumpus", "payload": "well, 'just packaging questions' are kind of important to do this, I think", "action": false, "timestamp": "2020-04-16T19:53:53+00:00" }, { "id": "8eace517d09148dd9c4f78fe187720ff", "sender": "luke-jr", "payload": "review club? :P", "action": false, "timestamp": "2020-04-16T19:53:55+00:00" }, { "id": "0e0d678139d6485794be11d16005af8f", "sender": "wumpus", "payload": "if you split up things people are going to ask how to re-bundle them :)", "action": false, "timestamp": "2020-04-16T19:54:12+00:00" }, { "id": "93ac705f9f484571955a737142601fe0", "sender": "ryanofsky", "payload": "i mean, packaging questions are the things you would be deciding on last, and maybe changing with trial and error", "action": false, "timestamp": "2020-04-16T19:54:28+00:00" }, { "id": "58c38f808f22438d98bcb44befe6945e", "sender": "cfields", "payload": "ryanofsky: some of those packaging questions come down to language choices though, those things need to be decided on much earlier.", "action": false, "timestamp": "2020-04-16T19:55:00+00:00" }, { "id": "521ec1d888094f02a5aad2d5d3a3b15b", "sender": "ryanofsky", "payload": "99% of the code stays the same regardless", "action": false, "timestamp": "2020-04-16T19:55:02+00:00" }, { "id": "c4f0cfb48791438096aefc29575f83f9", "sender": "wumpus", "payload": "in any case if you want concept ACK on multiprocess, concept ACK, I think it's a good thing in the longer run", "action": false, "timestamp": "2020-04-16T19:55:11+00:00" }, { "id": "641331f57cce4e0ebd4fc63fe51874b8", "sender": "ryanofsky", "payload": "cfields ?", "action": false, "timestamp": "2020-04-16T19:55:27+00:00" }, { "id": "ee2d899de2734dc48057bae30bafb434", "sender": "wumpus", "payload": "security-wise process isloation is a good start as well", "action": false, "timestamp": "2020-04-16T19:55:50+00:00" }, { "id": "d5950ce71fd24812b87b75f3f70d6950", "sender": "cfields", "payload": "ryanofsky: I'm curious to know, for example, what the downside of using the c++11 version libmultiprocess you've mentioned?", "action": false, "timestamp": "2020-04-16T19:56:21+00:00" }, { "id": "1f795ff4b51b49d090e1a699d52a3aa6", "sender": "jonasschnelli", "payload": "looking forward to wireguard-tunnel the GUI to a remote node.", "action": false, "timestamp": "2020-04-16T19:56:26+00:00" }, { "id": "5176cea417b5404e96218d86ad781d43", "sender": "cfields", "payload": "does that mean dragging boost back in?", "action": false, "timestamp": "2020-04-16T19:56:29+00:00" }, { "id": "371fae9c61804a9893e93299d52abfce", "sender": "wumpus", "payload": "so maybe the packaging details is the only thing we can disagree on :)", "action": false, "timestamp": "2020-04-16T19:57:01+00:00" }, { "id": "64dbc46b482f4c489342fb56b2bfb469", "sender": "jonasschnelli", "payload": "(but I guess that needs much more)", "action": false, "timestamp": "2020-04-16T19:57:01+00:00" }, { "id": "bcf94afc18b746d0a248fa1f18cc0fdd", "sender": "ryanofsky", "payload": "cfields, no downside, i can revert the vasild pr and replace std::optional with boost::optional again", "action": false, "timestamp": "2020-04-16T19:57:01+00:00" }, { "id": "db46f80a0e9b4651977b15bdd341e80b", "sender": "wumpus", "payload": "wait, why?", "action": false, "timestamp": "2020-04-16T19:57:33+00:00" }, { "id": "09c895859896476d8accbc6e25334d7c", "sender": "MarcoFalke", "payload": "In about 6 months we'll have C++17", "action": false, "timestamp": "2020-04-16T19:57:58+00:00" }, { "id": "765496ab840544a78841118ebba23c37", "sender": "ryanofsky", "payload": "jonasschnelli, not too much, you can kind of do it with my existing branch if you use socat (existing branch only create UNIX socket)", "action": false, "timestamp": "2020-04-16T19:57:59+00:00" }, { "id": "aa2231df8084493591577f8bf54822fa", "sender": "cfields", "payload": "wumpus: I'm just trying to understand our options.", "action": false, "timestamp": "2020-04-16T19:57:59+00:00" }, { "id": "1f1724291288409ca83ce964532916f0", "sender": "wumpus", "payload": "how can we be using std::optional in the first place we haven''t switched to C++17 yet right?", "action": false, "timestamp": "2020-04-16T19:58:02+00:00" }, { "id": "eeb8e750e7874763867d1e691f8b4e83", "sender": "sipa", "payload": "wumpus: libmultiprocess is using std::optional, but we're not using libmultiprocess yet", "action": false, "timestamp": "2020-04-16T19:58:20+00:00" }, { "id": "2511cac91784428398bfeb313e3f8b50", "sender": "ryanofsky", "payload": "wumpus, we are not, libmultiprocess used it internally because vasild submitted a pr to get rid of boost, and i thought it was nice", "action": false, "timestamp": "2020-04-16T19:58:33+00:00" }, { "id": "d5ec41a6546c49ca88c93c6d7156fcd9", "sender": "wumpus", "payload": "I think the idea was to have 0.21 still c++11 compatible then go full c++17 for 0.22", "action": false, "timestamp": "2020-04-16T19:58:51+00:00" }, { "id": "81c68f4b39674b178b1953ea96fda1c3", "sender": "MarcoFalke", "payload": "wumpus: libmultiprocess is compiled with c++14 flags in depends", "action": false, "timestamp": "2020-04-16T19:58:52+00:00" }, { "id": "dee40f0fb8d540548302324e198273e3", "sender": "sipa", "payload": "if it's just std::optional or boost::optional... that sounds like something we can just decide whenever it's merged", "action": false, "timestamp": "2020-04-16T19:58:55+00:00" }, { "id": "5c72089c002e417093fe35d1874a64ea", "sender": "wumpus", "payload": "c++14??!", "action": false, "timestamp": "2020-04-16T19:58:58+00:00" }, { "id": "d6dee4c20eaf4be0945fd479e16d8379", "sender": "wumpus", "payload": "nooo", "action": false, "timestamp": "2020-04-16T19:59:04+00:00" }, { "id": "7adca32cfa01497e8256fb79c0d25f52", "sender": "MarcoFalke", "payload": "s/is/was/", "action": false, "timestamp": "2020-04-16T19:59:23+00:00" }, { "id": "28a0f72ba1094dcb81fff6da1c8ff008", "sender": "luke-jr", "payload": "do we need to fork upstream to get C++11 support?", "action": false, "timestamp": "2020-04-16T19:59:26+00:00" }, { "id": "9ff3e920a0dc4f42b5d51d53add2a2b2", "sender": "sipa", "payload": "if multiprocess integration only lands with 0.22, we'd be fine", "action": false, "timestamp": "2020-04-16T19:59:27+00:00" }, { "id": "2944532bceb14ec3b16a5a0b893921d5", "sender": "ryanofsky", "payload": "it works fine now...", "action": false, "timestamp": "2020-04-16T19:59:38+00:00" }, { "id": "2666b5d83d0a470480b9a3d5febc245f", "sender": "sipa", "payload": "otherwise, it seems it can easily be turned into c++11 compatible", "action": false, "timestamp": "2020-04-16T19:59:42+00:00" }, { "id": "e364e00b08514281bfbaf2b480f2ef5f", "sender": "MarcoFalke", "payload": "sipa: Yes, that was my thinking", "action": false, "timestamp": "2020-04-16T19:59:43+00:00" }, { "id": "97dab069a47147638344b92c631fc7df", "sender": "wumpus", "payload": "I doubt that has to depend on using boost::optional or std::optional they're kind of the same right?", "action": false, "timestamp": "2020-04-16T19:59:46+00:00" }, { "id": "d9a52e472ab24cafa9cf6d2ca70d6c39", "sender": "luke-jr", "payload": "ryanofsky: with a non-C++14 compiler?", "action": false, "timestamp": "2020-04-16T19:59:51+00:00" }, { "id": "88f61e18c2db4ace9fbae461b7067bfe", "sender": "wumpus", "payload": "could use a wrapper or something like we do for fs.h", "action": false, "timestamp": "2020-04-16T19:59:54+00:00" }, { "id": "87f6a4ef66914f369c3d753a941e9551", "sender": "wumpus", "payload": "but we intentionally skipped over C++14, we're not going to do that now", "action": false, "timestamp": "2020-04-16T20:00:07+00:00" }, { "id": "e5ef793ced544c82989c3f37b1e0fcca", "sender": "achow101", "payload": "don't we have an Optional wrapper already?", "action": false, "timestamp": "2020-04-16T20:00:14+00:00" }, { "id": "8b17f8aa3d2e4b6f981fb545c91b4a9e", "sender": "luke-jr", "payload": "yes", "action": false, "timestamp": "2020-04-16T20:00:18+00:00" }, { "id": "745fa6e17d2544e18f120a8cb2a72599", "sender": "wumpus", "payload": "achow101: lol yes we do", "action": false, "timestamp": "2020-04-16T20:00:23+00:00" }, { "id": "37f7b953a4d447cfa39cb5afacad09a2", "sender": "ryanofsky", "payload": "i'm not sure what's not supposed to work here", "action": false, "timestamp": "2020-04-16T20:00:30+00:00" }, { "id": "58221b9077664510b52adf5245fb1698", "sender": "wumpus", "payload": "in any case I agree these are all small details?", "action": false, "timestamp": "2020-04-16T20:01:19+00:00" }, { "id": "994edfab2ac2442ba75a2e83e92eb788", "sender": "MarcoFalke", "payload": "I think boost::optional vs std::optional should be the least of our concerns in regard to multiprocess. This is just a sylistic issue", "action": false, "timestamp": "2020-04-16T20:01:22+00:00" }, { "id": "01273d1a19c9457ca7f8b31e6afe4ef5", "sender": "MarcoFalke", "payload": "wumpus: jup", "action": false, "timestamp": "2020-04-16T20:01:28+00:00" }, { "id": "c036945f1995421ba7705134c69c2145", "sender": "jonatack", "payload": "ryanofsky: why not add the PR to blockers?", "action": false, "timestamp": "2020-04-16T20:01:30+00:00" }, { "id": "d704583b8b844192bc39ec7946189969", "sender": "wumpus", "payload": "let's go forward with multiprocess imo", "action": false, "timestamp": "2020-04-16T20:01:32+00:00" }, { "id": "29eb90718abb4983b50514e550042b9f", "sender": "wumpus", "payload": "my biggest gripe is really the serialization lib dependency not boost", "action": false, "timestamp": "2020-04-16T20:01:50+00:00" }, { "id": "671abaddf41e49bdb4e6c5c48a36f9cc", "sender": "wumpus", "payload": "everyone wants to get rid of boost but also not get rid of boost it's not going to happen any time soon", "action": false, "timestamp": "2020-04-16T20:02:13+00:00" }, { "id": "92e518d5e2c24e848689e475ac31fef7", "sender": "ryanofsky", "payload": "jonatack, #16367 has been in high priority list, and was even merged (then got reverted)", "action": false, "timestamp": "2020-04-16T20:02:16+00:00" }, { "id": "42df8c0b51bd4582acc08e0080748528", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/16367 | Multiprocess build support by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #16367 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T20:02:18+00:00" }, { "id": "cd2f7db0ad184a9784cc8b11ab2f2dbf", "sender": "MarcoFalke", "payload": "So I think people don't want a half-merged multiprocess? In which case we should focus on the main pr", "action": false, "timestamp": "2020-04-16T20:02:49+00:00" }, { "id": "45f70ef3757e4f9e938107ef92cb602d", "sender": "wumpus", "payload": "we should wrap up the meeting btw", "action": false, "timestamp": "2020-04-16T20:03:04+00:00" }, { "id": "544d59d5f8a045a1b728c7a70a6d14eb", "sender": "wumpus", "payload": "sorry for only leaving so little time for this", "action": false, "timestamp": "2020-04-16T20:03:11+00:00" }, { "id": "85388482ff3e48f692b1ce4152d4518e", "sender": "cfields", "payload": "MarcoFalke: I didn't want a half-decided multiprocess. If we've decided to move forward on defaulting it, I have no issue with the merge.", "action": false, "timestamp": "2020-04-16T20:03:12+00:00" }, { "id": "6128abff476c45919190d377413b1049", "sender": "luke-jr", "payload": "it would be nice if there was a standard interface involved here, but lacking that, why not just use serialization.h?", "action": false, "timestamp": "2020-04-16T20:03:23+00:00" }, { "id": "305ae767c70e4cf0ae271ae527c74475", "sender": "ryanofsky", "payload": "wumpus, it's a valid concern. as much as possible I tried to make framework agnostic to serializtion and allow substituting in other implementations later", "action": false, "timestamp": "2020-04-16T20:03:24+00:00" }, { "id": "d47f9bb5689d45cb9162239e626b820d", "sender": "jb55", "payload": "wumpus: I've also ran into build issues with changing capnproto versions. is there a plan to rework that branch without capnproto?", "action": false, "timestamp": "2020-04-16T20:03:41+00:00" }, { "id": "133d88f305644869aacd839511cbce8b", "sender": "luke-jr", "payload": "cfields: why do we need to default it?", "action": false, "timestamp": "2020-04-16T20:03:46+00:00" }, { "id": "e1ad2a53c2da45fe8a7ea4225f0d85a5", "sender": "cfields", "payload": "luke-jr: ship the binaries by default, sorry.", "action": false, "timestamp": "2020-04-16T20:04:00+00:00" }, { "id": "b6e38c11ff534dd894cf14a35f078c30", "sender": "luke-jr", "payload": "ah", "action": false, "timestamp": "2020-04-16T20:04:06+00:00" }, { "id": "cc7d15bdef284c6aa4aa60949841fa58", "sender": "wumpus", "payload": "jb55: so the thing is, I think, that our own serialization framework is not powerful enough", "action": false, "timestamp": "2020-04-16T20:04:12+00:00" }, { "id": "39be0b5952234174aa10a80ff130f4f1", "sender": "sipa", "payload": "for the wallet<->node communication my big hope was that the protocol would be Bitcoin P2P, so that it can be substituted with whatever on either side... but it doesn't seem anyone's working on that", "action": false, "timestamp": "2020-04-16T20:04:17+00:00" }, { "id": "2fc3188e5e2b48f2a5c14155284b38ba", "sender": "jonatack", "payload": "ryanofsky: yes; a re-opened PR when it's ready (since you mentioned review)", "action": false, "timestamp": "2020-04-16T20:04:28+00:00" }, { "id": "f59963944c334a0293e2a437187714b8", "sender": "wumpus", "payload": "jb55: also using consensus serialization for other things might not be the best idea, so people have used something else", "action": false, "timestamp": "2020-04-16T20:04:58+00:00" }, { "id": "5e338b1f047f42758bd522cdc67a8b8d", "sender": "ryanofsky", "payload": "sipa, we're maybe getting closer that that with ariard's prs", "action": false, "timestamp": "2020-04-16T20:05:02+00:00" }, { "id": "8047ae5e2b9d4418944a8a1fbfa3185c", "sender": "sipa", "payload": "ryanofsky: okay", "action": false, "timestamp": "2020-04-16T20:05:08+00:00" }, { "id": "b17d9735495a47a6aa4c337bd52eb70b", "sender": "wumpus", "payload": "but yes what particular library to use, no idea", "action": false, "timestamp": "2020-04-16T20:05:11+00:00" }, { "id": "9cc3ec8ba3514aac9a15b12abcc96786", "sender": "ryanofsky", "payload": "he's stripping down the node/wallet interface so it's something more akin to p2p", "action": false, "timestamp": "2020-04-16T20:05:17+00:00" }, { "id": "595839d074384d11ae3ee96dd80dfed4", "sender": "MarcoFalke", "payload": "sipa: Wouldn't that be dangerous to accidentally connect to a malicious remote p2p node?", "action": false, "timestamp": "2020-04-16T20:05:27+00:00" }, { "id": "5e65f2b76fec473897da89edaa376429", "sender": "wumpus", "payload": "ryanofsky: oh that's nice!", "action": false, "timestamp": "2020-04-16T20:05:28+00:00" }, { "id": "d72ad553b6774c3e85ac251fb36774c4", "sender": "jb55", "payload": "what pr is that", "action": false, "timestamp": "2020-04-16T20:05:28+00:00" }, { "id": "40f4b0f634d942d6b9ec03bf5a5b355f", "sender": "luke-jr", "payload": "just to be clear: we're not planning on making the interfaces backward/forward compatible, right?", "action": false, "timestamp": "2020-04-16T20:05:48+00:00" }, { "id": "adfb6251f72c4c1382fdbad9c2e4d731", "sender": "sipa", "payload": "MarcoFalke: well you'd have SPV security - which may be even desirable", "action": false, "timestamp": "2020-04-16T20:05:49+00:00" }, { "id": "e1643054cc49464da17179ebdb20fe89", "sender": "sipa", "payload": "ryanofsky: (i didn't mean that as a \"you shouldn't do what you're doing\" - they're orthogonal ways of separating things; a P2P-protocol based one is just what i'm more interested in)", "action": false, "timestamp": "2020-04-16T20:06:08+00:00" }, { "id": "bb484a518c834bdd98a623dbce45efbd", "sender": "ryanofsky", "payload": "jb55, i think he only has locking PRs open now that make node/wallet more asynhrnous, but he has branches to do things like store block info in wallet i believe", "action": false, "timestamp": "2020-04-16T20:06:13+00:00" }, { "id": "8084ce701a7b41bfa12500490bbd166f", "sender": "wumpus", "payload": "but for an experimental thing it might be OK to rely on capnproto", "action": false, "timestamp": "2020-04-16T20:06:41+00:00" }, { "id": "a4cda4e7900d4210b7592201df83001e", "sender": "wumpus", "payload": "I don't want to stand in the way of that", "action": false, "timestamp": "2020-04-16T20:06:46+00:00" }, { "id": "1682ad2fd5ca4b1fabc9b9f16d17a2f4", "sender": "ryanofsky", "payload": "sipa, yes, I know that, that's the way I look at it too", "action": false, "timestamp": "2020-04-16T20:06:52+00:00" }, { "id": "bfdce57793824b3b8c66d05f0d9f3dda", "sender": "wumpus", "payload": "we've often enough delayed things before some 'perfect solutoin' would appear which then would never happen", "action": false, "timestamp": "2020-04-16T20:07:11+00:00" }, { "id": "d775fe1a9e6443d6a8a363642d78239a", "sender": "sipa", "payload": "agreed", "action": false, "timestamp": "2020-04-16T20:07:16+00:00" }, { "id": "21341aa357e84fe1b548ece283f3555d", "sender": "jb55", "payload": "so experimental with plans to replace it eventually?", "action": false, "timestamp": "2020-04-16T20:07:21+00:00" }, { "id": "bfd19640adcb47608b327930dee15b2d", "sender": "wumpus", "payload": "yess", "action": false, "timestamp": "2020-04-16T20:07:28+00:00" }, { "id": "5210a20b01594998967e9eebcac40546", "sender": "jb55", "payload": "I'm ok with that", "action": false, "timestamp": "2020-04-16T20:07:34+00:00" }, { "id": "842b2601a79348cfbc2b22c161ae8bf8", "sender": "sipa", "payload": "ryanofsky: what specifically does capnproto offer that's hard to do in our own serialization.h?", "action": false, "timestamp": "2020-04-16T20:07:45+00:00" }, { "id": "9db9fc6d9f3f46a28bc25361f922186e", "sender": "cfields", "payload": "sgtm", "action": false, "timestamp": "2020-04-16T20:07:48+00:00" }, { "id": "306bf6d44881477dbd4183b48266e5c0", "sender": "sipa", "payload": "or is it just avoiding lots of boilerplate code", "action": false, "timestamp": "2020-04-16T20:07:56+00:00" }, { "id": "a4203ccc6c604855a7cdc8ab5f6dc8ca", "sender": "sipa", "payload": "(which may be a valid reason too)", "action": false, "timestamp": "2020-04-16T20:08:05+00:00" }, { "id": "d9a7ef6acd414cd9ac0aaf07f14e2f74", "sender": "wumpus", "payload": "boilerplate I think", "action": false, "timestamp": "2020-04-16T20:08:11+00:00" }, { "id": "041a9c6fe7da4b8caf9659261b7db032", "sender": "wumpus", "payload": "capnproto autogenerates a lot", "action": false, "timestamp": "2020-04-16T20:08:19+00:00" }, { "id": "43357968e901412d9b0ab722c192db15", "sender": "ryanofsky", "payload": "sipa, it's a whole io framework, more than just serialization", "action": false, "timestamp": "2020-04-16T20:08:19+00:00" }, { "id": "a2829d3650a04290a5e5c2d1bffbeb54", "sender": "ryanofsky", "payload": "it's higher level even than things like grpc and thrift, it gives you object handles and supports callbacks", "action": false, "timestamp": "2020-04-16T20:08:42+00:00" }, { "id": "b75a0cbc3342432d963e5ff7138f679a", "sender": "wumpus", "payload": "right, it's a real RPC mechanism, not just serialization", "action": false, "timestamp": "2020-04-16T20:09:25+00:00" }, { "id": "c7b5b351c80a4aa09b470d3a12c749b1", "sender": "sipa", "payload": "i see", "action": false, "timestamp": "2020-04-16T20:09:30+00:00" }, { "id": "c386fca9396f4a8ca3e652416ec4d15e", "sender": "wumpus", "payload": "it goes further than protobuf in that regard", "action": false, "timestamp": "2020-04-16T20:09:42+00:00" }, { "id": "809b1a3f8738455c91b7a5a712e9ecdb", "sender": "ryanofsky", "payload": "and i've found it just very nice to work with, but I've tried to make everything around it as flexible as possible so it could be swapped with something else", "action": false, "timestamp": "2020-04-16T20:09:45+00:00" }, { "id": "43dde0cea37343ff949a77ec7ecd20f0", "sender": "luke-jr", "payload": "so might be more apt to compare it to bitcoind's RPC server..", "action": false, "timestamp": "2020-04-16T20:10:00+00:00" }, { "id": "b378a66fd7cd43f5b1854b2bd032fb83", "sender": "wumpus", "payload": "also it doesn't need to be compatible between differnt verisons", "action": false, "timestamp": "2020-04-16T20:10:03+00:00" }, { "id": "456368d0e40b4c65a65495dfec4ed59d", "sender": "wumpus", "payload": "as it's only used for internal communication", "action": false, "timestamp": "2020-04-16T20:10:12+00:00" }, { "id": "9a0c78d212e643abbf5a5dd99479ee8f", "sender": "jb55", "payload": "if its in in-tree as an experimental maybe we can get more people working on it... status has been ambiguous for so long", "action": false, "timestamp": "2020-04-16T20:10:16+00:00" }, { "id": "4f7e66cbb0774821bb4dc5d058a26c20", "sender": "wumpus", "payload": "between components of the same release", "action": false, "timestamp": "2020-04-16T20:10:20+00:00" }, { "id": "dcd6916d103941e88c8de328a3325474", "sender": "wumpus", "payload": "luke-jr: nononono it's *NOT* an official API", "action": false, "timestamp": "2020-04-16T20:10:30+00:00" }, { "id": "d4642200fc3d4151b60b8b8dd0b7a4fa", "sender": "ryanofsky", "payload": "wumpus, yes that is true, though capnproto does have same nice versioning / compatibility features as protobufs", "action": false, "timestamp": "2020-04-16T20:10:40+00:00" }, { "id": "8a018c36ab6c495ca1e8fb80f3f046cf", "sender": "luke-jr", "payload": "wumpus: I mean technically comparable, not supported the same", "action": false, "timestamp": "2020-04-16T20:10:44+00:00" }, { "id": "13ace685052f48c0a54e8b5b15293faa", "sender": "wumpus", "payload": "luke-jr: ok :)", "action": false, "timestamp": "2020-04-16T20:10:51+00:00" }, { "id": "51cb143dbd544fb0801c9aeca7846093", "sender": "wumpus", "payload": "agreed in that case", "action": false, "timestamp": "2020-04-16T20:11:10+00:00" }, { "id": "d47f395a2e454021aac042af9171be0d", "sender": "luke-jr", "payload": "wumpus: I too would be against a stable interface for this right now :P", "action": false, "timestamp": "2020-04-16T20:11:10+00:00" }, { "id": "ba624c1dcd54400d8315248d0f889e93", "sender": "luke-jr", "payload": "(unless we literally made it use bitcoind RPC maybe)", "action": false, "timestamp": "2020-04-16T20:11:20+00:00" }, { "id": "e1f8a3d67c6141ddb322faae2aeecc49", "sender": "wumpus", "payload": "yes, maybe in the future", "action": false, "timestamp": "2020-04-16T20:11:29+00:00" }, { "id": "6102a3ffcf9d425f83cc253fd4a26fc7", "sender": "wumpus", "payload": "ok, let's wrap up the meeting", "action": false, "timestamp": "2020-04-16T20:11:34+00:00" }, { "id": "d2aa7ad5f0804fb5988701b6f484d992", "sender": "cfields", "payload": "Thanks for the discussion :)", "action": false, "timestamp": "2020-04-16T20:11:34+00:00" }, { "id": "53647005389e498faf6e10cb4b443440", "sender": "wumpus", "payload": "thank you too", "action": false, "timestamp": "2020-04-16T20:11:38+00:00" }, { "id": "1c6fc37535d34de6a2f2c30ddcc805d4", "sender": "ryanofsky", "payload": "yes, the interfaces is just the headers in src/interfaces/, changes very often", "action": false, "timestamp": "2020-04-16T20:11:39+00:00" }, { "id": "1b8e669426ae40cca695a723d9084d58", "sender": "sipa", "payload": "it's certainly hard to agree on a stable interface 2 major releases before we plan to have it available :p", "action": false, "timestamp": "2020-04-16T20:11:39+00:00" }, { "id": "fc329e2960c2400ba77b3037caff82f1", "sender": "wumpus", "payload": "sipa: lol exactly", "action": false, "timestamp": "2020-04-16T20:11:49+00:00" }, { "id": "da2d4ab555264acb9a07ecf332bfda0b", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2020-04-16T20:12:09+00:00" } ], "events": [ { "event_type": "START_MEETING", "message": { "id": "da82f27edddf49368a972cd2d8704c95", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2020-04-16T19:03:58+00:00" }, "operand": null, "id": "da82f27edddf49368a972cd2d8704c95", "timestamp": "2020-04-16T19:03:58+00:00" }, { "event_type": "TOPIC", "message": { "id": "ffdfe3728c6a49e4bc4f819006075212", "sender": "wumpus", "payload": "#topic High priority for review", "action": false, "timestamp": "2020-04-16T19:07:43+00:00" }, "operand": "High priority for review", "id": "ffdfe3728c6a49e4bc4f819006075212", "timestamp": "2020-04-16T19:07:43+00:00" }, { "event_type": "LINK", "message": { "id": "30f8ddbec7b941f0b11669003d8c4cfa", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/185512 | HTTP Error 404: Not Found", "action": false, "timestamp": "2020-04-16T19:08:06+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/185512", "id": "30f8ddbec7b941f0b11669003d8c4cfa", "timestamp": "2020-04-16T19:08:06+00:00" }, { "event_type": "LINK", "message": { "id": "a64f0efcedde447a8199d38cf5bca01c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18242 | Add BIP324 encrypted p2p transport de-/serializer (only used in tests) by jonasschnelli \u00c3\u0082\u00c2\u00b7 Pull Request #18242 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:11+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18242", "id": "a64f0efcedde447a8199d38cf5bca01c", "timestamp": "2020-04-16T19:08:11+00:00" }, { "event_type": "LINK", "message": { "id": "6b5a92f1351f4f148cbb2a9ecc4600dd", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18512 | Improve asmap checks and add sanity check by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #18512 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:14+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18512", "id": "6b5a92f1351f4f148cbb2a9ecc4600dd", "timestamp": "2020-04-16T19:08:14+00:00" }, { "event_type": "LINK", "message": { "id": "fe2c07e52393489c8e58b68b7bd80610", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:24+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18192", "id": "fe2c07e52393489c8e58b68b7bd80610", "timestamp": "2020-04-16T19:08:24+00:00" }, { "event_type": "LINK", "message": { "id": "7e5ce8445943498db14819b769aa5365", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18550 | Store destdata for change in separate key for backward compatibility by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18550 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:25+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18550", "id": "7e5ce8445943498db14819b769aa5365", "timestamp": "2020-04-16T19:08:25+00:00" }, { "event_type": "LINK", "message": { "id": "5fb1abf59ac547acbd3bcce7101e57b1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18572 | Wallet: Accept \"changedata\" db key as an alias to \"destdata\" by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18572 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:26+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18572", "id": "5fb1abf59ac547acbd3bcce7101e57b1", "timestamp": "2020-04-16T19:08:26+00:00" }, { "event_type": "LINK", "message": { "id": "4b31f8d605454193badd4ed15684a139", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18608 | refactor: Remove CAddressBookData::destdata by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #18608 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:08:27+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18608", "id": "4b31f8d605454193badd4ed15684a139", "timestamp": "2020-04-16T19:08:27+00:00" }, { "event_type": "LINK", "message": { "id": "a98da469715d49f29e85824fb590b4f0", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18598 | gitian: Add missing automake package to gitian-win-signer.yml by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #18598 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:10:55+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18598", "id": "a98da469715d49f29e85824fb590b4f0", "timestamp": "2020-04-16T19:10:55+00:00" }, { "event_type": "LINK", "message": { "id": "f5fea06f668d49779bcf433081e0609c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18550 | Store destdata for change in separate key for backward compatibility by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18550 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:15:47+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18550", "id": "f5fea06f668d49779bcf433081e0609c", "timestamp": "2020-04-16T19:15:47+00:00" }, { "event_type": "LINK", "message": { "id": "ac1a8a5488bc4bc3ba309847feefb638", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:20:10+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18192", "id": "ac1a8a5488bc4bc3ba309847feefb638", "timestamp": "2020-04-16T19:20:10+00:00" }, { "event_type": "LINK", "message": { "id": "f520548edd2b4a4f86fac68f3a710930", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18192 | Bugfix: Wallet: Safely deal with change in the address book by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #18192 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:20:40+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18192", "id": "f520548edd2b4a4f86fac68f3a710930", "timestamp": "2020-04-16T19:20:40+00:00" }, { "event_type": "LINK", "message": { "id": "5ad9b961ab524429a1bd518e7c8b8419", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18608 | refactor: Remove CAddressBookData::destdata by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #18608 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:31:24+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18608", "id": "5ad9b961ab524429a1bd518e7c8b8419", "timestamp": "2020-04-16T19:31:24+00:00" }, { "event_type": "TOPIC", "message": { "id": "5f50892570af45f98b5c1563a1b03146", "sender": "wumpus", "payload": "#topic experimental libmultiprocess, next steps for multiprocess in general (MarcoFalke)", "action": false, "timestamp": "2020-04-16T19:45:53+00:00" }, "operand": "experimental libmultiprocess, next steps for multiprocess in general (MarcoFalke)", "id": "5f50892570af45f98b5c1563a1b03146", "timestamp": "2020-04-16T19:45:53+00:00" }, { "event_type": "LINK", "message": { "id": "73bab8c1305d40948b65f1dfa8a4e897", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18588 | Revert \"Merge #16367: Multiprocess build support\" by MarcoFalke \u00c3\u0082\u00c2\u00b7 Pull Request #18588 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T19:46:58+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18588", "id": "73bab8c1305d40948b65f1dfa8a4e897", "timestamp": "2020-04-16T19:46:58+00:00" }, { "event_type": "LINK", "message": { "id": "42df8c0b51bd4582acc08e0080748528", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/16367 | Multiprocess build support by ryanofsky \u00c3\u0082\u00c2\u00b7 Pull Request #16367 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2020-04-16T20:02:18+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/16367", "id": "42df8c0b51bd4582acc08e0080748528", "timestamp": "2020-04-16T20:02:18+00:00" }, { "event_type": "END_MEETING", "message": { "id": "da2d4ab555264acb9a07ecf332bfda0b", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2020-04-16T20:12:09+00:00" }, "operand": null, "id": "da2d4ab555264acb9a07ecf332bfda0b", "timestamp": "2020-04-16T20:12:09+00:00" } ], "aliases": {}, "vote_in_progress": false, "motion_index": null }