{ "founder": "wumpus", "channel": "#bitcoin-core-dev", "network": "freenode", "id": "8f5c1e47d08c49849fda44523127f5dc", "name": "#bitcoin-core-dev", "chair": "wumpus", "chairs": [ "wumpus" ], "nicks": { "wumpus": 55, "lightningbot": 2, "sipa": 54, "dongcarl": 1, "moneyball": 6, "meshcollider": 27, "BlueMatt": 64, "gribble": 12, "kanzure": 1, "gmaxwell": 35, "chenpo": 1, "jnewbery": 18, "provoostenator": 33, "achow101": 2, "ryanofsky": 2, "morcos": 3, "instagibbs": 6, "Chris_Stewart_5": 1, "sdaftuar": 3, "bitcoin-git": 1 }, "start_time": "2018-11-29T19:00:25+00:00", "end_time": "2018-11-29T20:00:46+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/", "current_topic": "wallet maintainer", "messages": [ { "id": "70559c785b874bfa8173dfd607d75ef5", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2018-11-29T19:00:25+00:00" }, { "id": "8f2157ca9b714ed2af218207eefd127b", "sender": "lightningbot", "payload": "Meeting started Thu Nov 29 19:00:25 2018 UTC. The chair is wumpus. Information about MeetBot at http://wiki.debian.org/MeetBot.", "action": false, "timestamp": "2018-11-29T19:00:25+00:00" }, { "id": "d6fa21584efa4326ad6210a32f644f9d", "sender": "lightningbot", "payload": "Useful Commands: #action #agreed #help #info #idea #link #topic.", "action": false, "timestamp": "2018-11-29T19:00:25+00:00" }, { "id": "1593fbb068cd449189d4eafa02bdf4ef", "sender": "sipa", "payload": "oh, meeting!", "action": false, "timestamp": "2018-11-29T19:00:38+00:00" }, { "id": "857cafbdf6b244d4adc3332c62552ab4", "sender": "dongcarl", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:00:42+00:00" }, { "id": "3526f51dacc94aeb9850afc2631ed383", "sender": "moneyball", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:00:59+00:00" }, { "id": "2720d54a3d044c619dcc61ceaa5ac799", "sender": "wumpus", "payload": "#bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator", "action": false, "timestamp": "2018-11-29T19:01:04+00:00" }, { "id": "6ecd9d83d9cf4a3dbe5a68a819f91147", "sender": "moneyball", "payload": "suggested topic: proposed meeting topics ahead of time", "action": false, "timestamp": "2018-11-29T19:01:14+00:00" }, { "id": "6e6c38874de04459a5d70db5043c8e0b", "sender": "meshcollider", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:01:23+00:00" }, { "id": "466c95265e18433a82cfddc602bf98a2", "sender": "BlueMatt", "payload": "topics: 0.17.1", "action": false, "timestamp": "2018-11-29T19:01:25+00:00" }, { "id": "eaf293183a3f4233b9191f2e0e16a4e2", "sender": "wumpus", "payload": "#topic high priority for review", "action": false, "timestamp": "2018-11-29T19:01:52+00:00" }, { "id": "90bcbe9f5691443bab12a89bbecb3018", "sender": "wumpus", "payload": "current PRs: #14670 #13932 #14336 #14646", "action": false, "timestamp": "2018-11-29T19:02:30+00:00" }, { "id": "92699bcf2af04a0d9a8647ea9d4725b2", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14670 | http: Fix HTTP server shutdown by promag \u00c3\u0082\u00c2\u00b7 Pull Request #14670 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:32+00:00" }, { "id": "0580adb0b65d442b89f3888b973d09b1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/13932 | Additional utility RPCs for PSBT by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #13932 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:36+00:00" }, { "id": "179cbe03b6e345c2a695e0ded6ae30a8", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14646 | Add expansion cache functions to descriptors (unused for now) by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14646 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:38+00:00" }, { "id": "ebfd3a9d649043bf8a3737d1ea120e2c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14336 | net: implement poll by pstratem \u00c3\u0082\u00c2\u00b7 Pull Request #14336 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:42+00:00" }, { "id": "9354a72978a34cdbadfe632102506364", "sender": "kanzure", "payload": "hi.", "action": false, "timestamp": "2018-11-29T19:02:53+00:00" }, { "id": "8ac3a502b38244339274d6f27a5c3545", "sender": "wumpus", "payload": "anyone want to add or remove anything?", "action": false, "timestamp": "2018-11-29T19:03:48+00:00" }, { "id": "f11aa8a4ce9545c8a16fa64556d82796", "sender": "sipa", "payload": "lgtm", "action": false, "timestamp": "2018-11-29T19:04:07+00:00" }, { "id": "c438c1eb15294ef3a090a382324bc12d", "sender": "meshcollider", "payload": "Pieter already has one but it'd be great to have #14565", "action": false, "timestamp": "2018-11-29T19:04:25+00:00" }, { "id": "ce33e752cbfc4a0eacb31a9dea369756", "sender": "gmaxwell", "payload": "whats the overlap with 0.17.1 tag?", "action": false, "timestamp": "2018-11-29T19:04:28+00:00" }, { "id": "adbf148e05f1484a86986782481d3281", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14565 | Overhaul importmulti logic by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14565 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:04:29+00:00" }, { "id": "80f3291d84aa4766b9e2b2ea7533a1eb", "sender": "chenpo", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:04:43+00:00" }, { "id": "17d91d94f58e46bf9feb3f44af7f8532", "sender": "wumpus", "payload": "does the 0.17.1 tag have any non-backports left?", "action": false, "timestamp": "2018-11-29T19:05:05+00:00" }, { "id": "b2cde5556cca4ead9919b1f2665d636c", "sender": "gmaxwell", "payload": "#14380", "action": false, "timestamp": "2018-11-29T19:05:08+00:00" }, { "id": "f7a3bdaefde84d22aa1e7b031257b066", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14380 | fix assert crash when specified change output spend size is unknown by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #14380 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:05:12+00:00" }, { "id": "cc3363e1382749238b407b57aa1cb710", "sender": "gmaxwell", "payload": "is what I was thinking of.", "action": false, "timestamp": "2018-11-29T19:05:25+00:00" }, { "id": "b936edc0cb2c430e9c90bb68a79e777f", "sender": "wumpus", "payload": "https://github.com/bitcoin/bitcoin/milestone/39", "action": false, "timestamp": "2018-11-29T19:05:30+00:00" }, { "id": "d593285d1dc6484a8905c4ff4ac3dfd3", "sender": "wumpus", "payload": "let's add that one then", "action": false, "timestamp": "2018-11-29T19:06:09+00:00" }, { "id": "b03fd826b81449f08cb5c3a4aaa5aff8", "sender": "meshcollider", "payload": "14782 is sort-of not a backport", "action": false, "timestamp": "2018-11-29T19:06:32+00:00" }, { "id": "bceac9f145c74b18aa1d78c92c1ad9ae", "sender": "jnewbery", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:06:41+00:00" }, { "id": "9b50c19955e645bd91e40929366163ec", "sender": "wumpus", "payload": "meshcollider: which one is the most-blocking?", "action": false, "timestamp": "2018-11-29T19:06:49+00:00" }, { "id": "11646f27c1f94eac97385c73672933c4", "sender": "wumpus", "payload": "#14646 seems like it needs to go in before something else so looks like a typical candidate at least", "action": false, "timestamp": "2018-11-29T19:07:14+00:00" }, { "id": "979a2e342ddc49588844d1d6ce451f77", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14646 | Add expansion cache functions to descriptors (unused for now) by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14646 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:07:16+00:00" }, { "id": "397769407bf64a4f86ebc7311c8ead0e", "sender": "meshcollider", "payload": "Yeah don't remove that one", "action": false, "timestamp": "2018-11-29T19:07:38+00:00" }, { "id": "1e665570de614dbbac12a69f1807638f", "sender": "gmaxwell", "payload": "The broken getbalance behavior should get fixed. It's an actual bug.", "action": false, "timestamp": "2018-11-29T19:08:19+00:00" }, { "id": "9cbff94d4db14514bdc69516cb163a5a", "sender": "wumpus", "payload": "that's only broken on 0.17 and not master?", "action": false, "timestamp": "2018-11-29T19:08:40+00:00" }, { "id": "c02be43dc092413c845449924aa46191", "sender": "provoostenator", "payload": "#14602", "action": false, "timestamp": "2018-11-29T19:08:56+00:00" }, { "id": "11176d9503654a9fb096f5abdeccc03c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14602 | Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14602 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:08:58+00:00" }, { "id": "8f8a1095baaf49c280345b7ea88d522c", "sender": "achow101", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:09:15+00:00" }, { "id": "d2dd49c990564563a6e102bea2a778fa", "sender": "wumpus", "payload": "wait, I added #14782 which is the 0.17 backport", "action": false, "timestamp": "2018-11-29T19:09:17+00:00" }, { "id": "40557a7f5c324a63847cc6b474f92169", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14782 | [0.17] Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14782 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:09:19+00:00" }, { "id": "e2dee01d0a064ef8af9799b605491098", "sender": "jnewbery", "payload": "I'm not clear what the behaviour is 'supposed' to be. There's no documentation or test coverage", "action": false, "timestamp": "2018-11-29T19:09:21+00:00" }, { "id": "bafe3a64b38249feb3c0dd2be707390b", "sender": "gmaxwell", "payload": "wumpus: not yet, there is another PR", "action": false, "timestamp": "2018-11-29T19:09:26+00:00" }, { "id": "085289ed5bb8432286e1c63966dcb574", "sender": "wumpus", "payload": "why is a backport open before something is merged into master? is it signifiantly different?", "action": false, "timestamp": "2018-11-29T19:09:41+00:00" }, { "id": "7819edab729c4e19a5bfe970380ae73b", "sender": "provoostenator", "payload": "Ah yes, it's already marked for backport.", "action": false, "timestamp": "2018-11-29T19:09:42+00:00" }, { "id": "4be4234ef1554d2688d43d10a7113b93", "sender": "meshcollider", "payload": "The 0.17 backport of it is different yes", "action": false, "timestamp": "2018-11-29T19:09:49+00:00" }, { "id": "c5771f42b9b04aa1a561b08a47eadf13", "sender": "meshcollider", "payload": "Because of the accounts removal", "action": false, "timestamp": "2018-11-29T19:09:53+00:00" }, { "id": "6effcbed16d24fb8965d8db43813d61f", "sender": "sipa", "payload": "maybe the getbalance behaviour needs to be topic on its own>?", "action": false, "timestamp": "2018-11-29T19:09:57+00:00" }, { "id": "cd3d5f9fb9c340c9b7b89ce6d42b2d86", "sender": "wumpus", "payload": "right", "action": false, "timestamp": "2018-11-29T19:09:57+00:00" }, { "id": "5d764294b8e446d0b26a8289a0b280a2", "sender": "gmaxwell", "payload": "jnewbery: thats a fine concern, but it's not cool to break functionality just because there wasn't enough tests to stop you from doing so", "action": false, "timestamp": "2018-11-29T19:10:00+00:00" }, { "id": "447fc7837fdb4aef90ec667335dcd853", "sender": "meshcollider", "payload": "ping luke-jr", "action": false, "timestamp": "2018-11-29T19:10:10+00:00" }, { "id": "e6f8faa943964e33978dcaf28566ae28", "sender": "wumpus", "payload": "if there's any doubt it probably needs to stay as it was", "action": false, "timestamp": "2018-11-29T19:10:18+00:00" }, { "id": "4751a74121134b68b14f9f5393aa46a9", "sender": "gmaxwell", "payload": "The alternative of going and reverting the breaking change is unfortunately too disruptive (I already tried).", "action": false, "timestamp": "2018-11-29T19:10:33+00:00" }, { "id": "6f57f936fea34908be21ec96ca4e5cc2", "sender": "meshcollider", "payload": "Re-implementing the old behaviour without reintroducing account parameters is the issue", "action": false, "timestamp": "2018-11-29T19:10:47+00:00" }, { "id": "b5ec807cb7bc41debcfec03d471a4b53", "sender": "jnewbery", "payload": "the new behaviour in the fix is different from pre-0.17", "action": false, "timestamp": "2018-11-29T19:10:54+00:00" }, { "id": "d1585a7606df40bb9b93ce2526328a5d", "sender": "meshcollider", "payload": "And it has some weird parameter type polymorphism", "action": false, "timestamp": "2018-11-29T19:11:15+00:00" }, { "id": "9b14c92205c54f1db4c936387f7f78a8", "sender": "wumpus", "payload": "well for 0.18, changing the behavior can be discussed if there's agood reason", "action": false, "timestamp": "2018-11-29T19:11:18+00:00" }, { "id": "ff71e4a6dd1347a8a430892a65fa5045", "sender": "sipa", "payload": "gmaxwell: well given the removal of accounts, i think it's fair to say that anyone passing not-nothing as the account parameter to getbalance needs to change, and you can get the 0-conf behavior using getunconfirmedbalance", "action": false, "timestamp": "2018-11-29T19:11:23+00:00" }, { "id": "3452971625c24bc197b1bfd8f682e188", "sender": "jnewbery", "payload": "so before we decide whether it should be merged, we should figure out what the behavious should be, document it and cover it with tests so it's not regressed", "action": false, "timestamp": "2018-11-29T19:11:25+00:00" }, { "id": "ba7e354f67fb4de88ef235f234079fc0", "sender": "provoostenator", "payload": "getunconfirmedbalance _only_ includes unconfirmed balance afaik", "action": false, "timestamp": "2018-11-29T19:11:47+00:00" }, { "id": "9aa451a69d574ef8b011835a7bfd696e", "sender": "sipa", "payload": "oh", "action": false, "timestamp": "2018-11-29T19:11:51+00:00" }, { "id": "f2fd0265778a4205bda566c775ece868", "sender": "wumpus", "payload": "for 0.17 it'd be good to go back to the old behavior because it was not announced in the release notes that it would change", "action": false, "timestamp": "2018-11-29T19:12:00+00:00" }, { "id": "326af9ebe43649ee900c4a3c647e2e1b", "sender": "sipa", "payload": "ignore my suggestion, again", "action": false, "timestamp": "2018-11-29T19:12:01+00:00" }, { "id": "d32dadd853034a5e9ff4112946df53a3", "sender": "meshcollider", "payload": "Add them together then :)", "action": false, "timestamp": "2018-11-29T19:12:02+00:00" }, { "id": "808e50e066984cc5a8906002f70b5dd2", "sender": "gmaxwell", "payload": "if you add them you'll double count", "action": false, "timestamp": "2018-11-29T19:12:18+00:00" }, { "id": "fb56c7f81b574e25a25a8d2d0005b13a", "sender": "meshcollider", "payload": "wumpus: that's why the 0.17 backport was opened already", "action": false, "timestamp": "2018-11-29T19:12:32+00:00" }, { "id": "f1a9313914914cda905de3c21ca223c8", "sender": "provoostenator", "payload": "It's useful in practice to be able to get the balance including unconfirmed, so you can compose a PSBT while you're waiting for confirmation.", "action": false, "timestamp": "2018-11-29T19:12:35+00:00" }, { "id": "69ab693877334b76822b6f0fc712cb03", "sender": "BlueMatt", "payload": "also for 0.17.1 this seems a bit too late to be figuring out", "action": false, "timestamp": "2018-11-29T19:12:36+00:00" }, { "id": "458720eaf2e14f97a7e6730c2202511c", "sender": "BlueMatt", "payload": "we can revisit for 0.17.2 if there is one", "action": false, "timestamp": "2018-11-29T19:12:43+00:00" }, { "id": "eebec7443a374e1aba759130ed1c426d", "sender": "provoostenator", "payload": "(without doing math)", "action": false, "timestamp": "2018-11-29T19:12:43+00:00" }, { "id": "e569cb0bc36e42489c0b069cbcb023f5", "sender": "BlueMatt", "payload": "but 0.17.0 currently cannot be self-built and bitcoin-qt opened on latest fedora/ubuntu release", "action": false, "timestamp": "2018-11-29T19:13:11+00:00" }, { "id": "4f58f8b47fb04aeb95ce7331fe770b8e", "sender": "BlueMatt", "payload": "both of which have been out for a bit now", "action": false, "timestamp": "2018-11-29T19:13:16+00:00" }, { "id": "323ff0547f454ba69fd6f1070d55aa50", "sender": "gmaxwell", "payload": "It isn't just a question of unconfirmed or not, it's a question of your balance suddenly showing zero when you make a small payment.. because now your change is unconfirmed.", "action": false, "timestamp": "2018-11-29T19:13:17+00:00" }, { "id": "fe60f118ee4a4c03bc7ad9d16c6e5047", "sender": "sipa", "payload": "gmaxwell: getbalance includes change", "action": false, "timestamp": "2018-11-29T19:13:29+00:00" }, { "id": "3c16ff4d6e564ff1bd0fd5d85de1b12a", "sender": "sipa", "payload": "just not unconfirmed incoming payments", "action": false, "timestamp": "2018-11-29T19:13:40+00:00" }, { "id": "6c4f2564cb0d47a6a29c83a07a48e9a8", "sender": "sipa", "payload": "that was the distinction between getbalance and getbalance *, afaik", "action": false, "timestamp": "2018-11-29T19:14:02+00:00" }, { "id": "54fab47b971e46b0979537e20be4a104", "sender": "wumpus", "payload": "#topic getbalance", "action": false, "timestamp": "2018-11-29T19:14:13+00:00" }, { "id": "35737fb3c5ee40b28d583d008dbf77a5", "sender": "gmaxwell", "payload": "Right", "action": false, "timestamp": "2018-11-29T19:14:16+00:00" }, { "id": "19bab2d0f6d24419a01f8108e4f1f56f", "sender": "gmaxwell", "payload": "sipa: and thats also why you can't add getunconfirmedbalance.", "action": false, "timestamp": "2018-11-29T19:14:31+00:00" }, { "id": "be30c6aafcdf4961a619b9b593dbc228", "sender": "jnewbery", "payload": "for those without context, this is (mostly) my fault for removing accounts. The first parameter to getbalance was 'account', it was marked DEPRECATED with a warning saying 'don't use this'. I removed it and it turns out people were using it to sum up untrusted coins", "action": false, "timestamp": "2018-11-29T19:14:39+00:00" }, { "id": "87a40f8c498b44ddbe5e8ef4bed9aa9a", "sender": "sipa", "payload": "you can simulate \"getbalance *\" using \"getbalance minconf=1\" + \"getunconfirmedbalance\" i guess", "action": false, "timestamp": "2018-11-29T19:14:53+00:00" }, { "id": "e24a011244a34d098da99de067a2aee8", "sender": "wumpus", "payload": "BlueMatt: yes, let's discuss what to in include in 0.17.1 in the 0.17.1 topic", "action": false, "timestamp": "2018-11-29T19:14:56+00:00" }, { "id": "9ba5139fe7184265a9a613d542aafa61", "sender": "meshcollider", "payload": "But the actual difference wasn't just unspent, it was untrusted, right?", "action": false, "timestamp": "2018-11-29T19:15:00+00:00" }, { "id": "8a1ba8001eca402281e93c6c369ba957", "sender": "sipa", "payload": "meshcollider: indeed", "action": false, "timestamp": "2018-11-29T19:15:05+00:00" }, { "id": "4926625d31514d05b973d18a88523aa9", "sender": "meshcollider", "payload": "Unconfirmed*", "action": false, "timestamp": "2018-11-29T19:15:08+00:00" }, { "id": "72159341346b48c5b1238763430ce70f", "sender": "jnewbery", "payload": "getbalance -minconf=0 includes unconfirmed but not untrusted coins (eg your change)", "action": false, "timestamp": "2018-11-29T19:15:27+00:00" }, { "id": "1023b0c400cf4cbc9cefe93845022f90", "sender": "jnewbery", "payload": "getbalance * 0 would (prior to 0.17) include untrusted", "action": false, "timestamp": "2018-11-29T19:15:40+00:00" }, { "id": "0d329efba10e459b87a8992c7ba49d48", "sender": "jnewbery", "payload": "so for example if someone sent you an output, then RBF'ed it, you'd still see that in your getbalance * 0", "action": false, "timestamp": "2018-11-29T19:16:00+00:00" }, { "id": "dbeb27e20eac4b8181c926edf3ef9cb0", "sender": "meshcollider", "payload": "as you pointed out on the PR though john, does that also include conflicting transactions and stuff too?", "action": false, "timestamp": "2018-11-29T19:16:04+00:00" }, { "id": "d72a148e8e7e4fafaa6c470356a529f1", "sender": "jnewbery", "payload": "or RBF'ed it to you, you'd double count it", "action": false, "timestamp": "2018-11-29T19:16:22+00:00" }, { "id": "fc3d80b35f914db3981da5dbb08bdd1c", "sender": "jnewbery", "payload": "or you could have a negative balance", "action": false, "timestamp": "2018-11-29T19:16:29+00:00" }, { "id": "c2f91edeb9594c39a12d4d35e458b137", "sender": "meshcollider", "payload": "Thats not sane behaviour to keep then :/", "action": false, "timestamp": "2018-11-29T19:16:57+00:00" }, { "id": "bc581c0284f14b5eb273a5e1e13a8d94", "sender": "wumpus", "payload": "that sounds pretty broken", "action": false, "timestamp": "2018-11-29T19:17:06+00:00" }, { "id": "1b03a6de269648bea18eafcf731241de", "sender": "gmaxwell", "payload": "I didn't think it included conflicteds, but actually it makes sense.", "action": false, "timestamp": "2018-11-29T19:17:08+00:00" }, { "id": "c1514ff286d24cd09f6c00425b1a21db", "sender": "jnewbery", "payload": "I don't know why people were using getbalance * 0, or what they were using it for. I've asked a few times on the PR but no-one can answer", "action": false, "timestamp": "2018-11-29T19:17:11+00:00" }, { "id": "2c723b90148844798bba6681a76e9fcc", "sender": "gmaxwell", "payload": "If you recall when we removed accounts I raised a concern that didn't we still need move to fixed people's negative balances... because we still had those reported from time to time.", "action": false, "timestamp": "2018-11-29T19:17:41+00:00" }, { "id": "7bce51d54519439daf20fce078c48e26", "sender": "jnewbery", "payload": "ok, I've fixed that :)", "action": false, "timestamp": "2018-11-29T19:17:58+00:00" }, { "id": "4962d40dd431462abd579a1316158587", "sender": "gmaxwell", "payload": "jnewbery: to see unconfirmed payments... it's certantly what I always did before I gave up on balances and just switched to looking at the unspent outputs directly", "action": false, "timestamp": "2018-11-29T19:18:08+00:00" }, { "id": "8887d4c84a3c44bdba440fb95070f597", "sender": "ryanofsky", "payload": "It sounded to me like people were just using it to get unconfirmed balance without having two make two calls", "action": false, "timestamp": "2018-11-29T19:18:09+00:00" }, { "id": "85bd5665c37d4838a885c6fa1d8a60ea", "sender": "provoostenator", "payload": "That's exactly how I use it.", "action": false, "timestamp": "2018-11-29T19:18:29+00:00" }, { "id": "4e08f14069a84ba4ac0a44d7bf8f9855", "sender": "sipa", "payload": "how about adding a \"requiretrusted\" parameter to getbalance...?", "action": false, "timestamp": "2018-11-29T19:18:42+00:00" }, { "id": "b7f4af8903a14e93b30418ce6041fae0", "sender": "gmaxwell", "payload": "ryanofsky: not just without having to make two calls, because getbalance included 0 conf trusted coins.", "action": false, "timestamp": "2018-11-29T19:18:43+00:00" }, { "id": "c395e86ea23e4a81a595b8c206f071a2", "sender": "provoostenator", "payload": "If you want to prepare a transaction to spend all, you need to enter the amount to send.", "action": false, "timestamp": "2018-11-29T19:18:52+00:00" }, { "id": "53125c44e08c4647a00c3136de77b96f", "sender": "gmaxwell", "payload": "ryanofsky: and the minconf parameter was added later", "action": false, "timestamp": "2018-11-29T19:18:53+00:00" }, { "id": "d92d71313a164a52b180299b4c0387a9", "sender": "jnewbery", "payload": "I think it's fine to have a balance including unconfirmeds if they're trusted (ie all inputs were yours). Having a balance containing unconfirmeds that aren't trusted and so can be double-accounted or worse doesn't make sense to me", "action": false, "timestamp": "2018-11-29T19:19:01+00:00" }, { "id": "8a6a8d7f64534061907ac4ea87a0633c", "sender": "meshcollider", "payload": "sipa: the PR introduces a bool for trustedness", "action": false, "timestamp": "2018-11-29T19:19:12+00:00" }, { "id": "d6feb591afdc45baa7692a7695207165", "sender": "sipa", "payload": "meshcollider: oh!", "action": false, "timestamp": "2018-11-29T19:19:18+00:00" }, { "id": "0ceb90025e0941a2bf849056b146c032", "sender": "jnewbery", "payload": "I strongly discourage any 'balance' that contains untrusted coins", "action": false, "timestamp": "2018-11-29T19:19:36+00:00" }, { "id": "924707793125451ab56ff29eff5152f1", "sender": "sipa", "payload": "does it allow counting untrusted, while excluding conflicting etc?", "action": false, "timestamp": "2018-11-29T19:19:38+00:00" }, { "id": "a283adec62444a7c928cebb33a05e49a", "sender": "meshcollider", "payload": "No", "action": false, "timestamp": "2018-11-29T19:19:44+00:00" }, { "id": "f716e9d15edb416a9253077f1f9d6fb0", "sender": "gmaxwell", "payload": "jnewbery: I don't understand \"trusted and so can be double-accounted\" -- even a trusted payment could be conflicted", "action": false, "timestamp": "2018-11-29T19:19:53+00:00" }, { "id": "a3548ab6a6e04675aecc44b97a5eb04d", "sender": "provoostenator", "payload": "jnewbery: why? Isn't that implied when the user enters 0 instead of 1 confirmations?", "action": false, "timestamp": "2018-11-29T19:20:00+00:00" }, { "id": "a6a604d820c24123bf74f792e76c85e8", "sender": "meshcollider", "payload": "unconfirmed != untrusted though", "action": false, "timestamp": "2018-11-29T19:20:28+00:00" }, { "id": "ec56683187ba4458a7f683b45100b35b", "sender": "gmaxwell", "payload": "jnewbery: we _show_ unconfirmed payments in the wallet, so it's somewhat odd and confusing that we don't have \"pending balance\"", "action": false, "timestamp": "2018-11-29T19:20:41+00:00" }, { "id": "c9453c7b4b014c3dac7256ddd7c9cfcb", "sender": "wumpus", "payload": "the GUI does have a 'unconfirmed balance' IIRC?", "action": false, "timestamp": "2018-11-29T19:21:10+00:00" }, { "id": "dbc708bf9d3c4a349bdfcc3e032f052e", "sender": "sipa", "payload": "showing untrusted balance sounds perfectly reasonable, but it shouldn't include conflicting/... things", "action": false, "timestamp": "2018-11-29T19:21:27+00:00" }, { "id": "21dbb1317944441b994e846201024752", "sender": "meshcollider", "payload": "We could break the getbalance output and return an array of \"confirmed\":, \"unconfirmed\":, \"conflicting:", "action": false, "timestamp": "2018-11-29T19:21:43+00:00" }, { "id": "893303964eae430e9cdaa962aa452bf5", "sender": "jnewbery", "payload": "for me 'pending balance' doesn't make sense if it can double-count conflicting transactions. It's worse that not having a pending balance at all", "action": false, "timestamp": "2018-11-29T19:21:54+00:00" }, { "id": "f13252640d394d1792eb87896e5b6785", "sender": "meshcollider", "payload": "^", "action": false, "timestamp": "2018-11-29T19:22:02+00:00" }, { "id": "47c578493fbc4ac38fd5dac4164bc3ae", "sender": "sipa", "payload": "jnewbery: i agree", "action": false, "timestamp": "2018-11-29T19:22:06+00:00" }, { "id": "de81ae182df1426e82ea4ccfe558d8c0", "sender": "morcos", "payload": "yes, its not clear to me what the problem is with using getbalance and getunconfirmedbalance when using -cli and the GUI already does the right thing?", "action": false, "timestamp": "2018-11-29T19:22:07+00:00" }, { "id": "f9e668e0893e470cbc34b1fceb570e81", "sender": "gmaxwell", "payload": "In any case, you wanted the use case, the reason people use getbalance \"*\" is to (Attempt) to learn their pending balance. What they'll have assuming nothing goes wrong. Now if that was also double counting conflicteds and ending up negative, I think thats a bug. (and probably the origin of the negative balances I was concerned about before)", "action": false, "timestamp": "2018-11-29T19:22:07+00:00" }, { "id": "5d022d495d3845d6bbe0ed0a50c8a11a", "sender": "wumpus", "payload": "users don't know what 'untrusted balance' is supposed to be, I think exposing these kind of implementation details on the interface is a bit confusing", "action": false, "timestamp": "2018-11-29T19:22:14+00:00" }, { "id": "a22b8e18683c4fec9dac33991c0f82a7", "sender": "provoostenator", "payload": "I agree double counting is undesirable.", "action": false, "timestamp": "2018-11-29T19:22:15+00:00" }, { "id": "e94b085ca03140bdb25694f059f2763e", "sender": "jnewbery", "payload": "it will make users think their money has disappeared because once one of those conflicting transactions is confirmed their balance goes down", "action": false, "timestamp": "2018-11-29T19:22:20+00:00" }, { "id": "7c833937e8f646d5be18010ebd7ff918", "sender": "morcos", "payload": "can someone clearly point out exactly what we can not do now (other than we didn't properly explain what was changing)", "action": false, "timestamp": "2018-11-29T19:22:39+00:00" }, { "id": "98e50d0757bf47ca8c136abfb3692f3f", "sender": "meshcollider", "payload": "luke-jr: should be here to argue the other side", "action": false, "timestamp": "2018-11-29T19:22:43+00:00" }, { "id": "a2dfe33e16624ed3be7a7facaf778813", "sender": "sipa", "payload": "jnewbery: i think no getbalance* call should evr include conflicted things", "action": false, "timestamp": "2018-11-29T19:22:45+00:00" }, { "id": "655027a81b4841ea9aedb46383bc11f1", "sender": "provoostenator", "payload": "It should probably use mempool rules to figure out which transaction to ignore.", "action": false, "timestamp": "2018-11-29T19:22:49+00:00" }, { "id": "e975ebfcb27e4187b453d6f89eaaa98a", "sender": "sipa", "payload": "provoostenator: some things do", "action": false, "timestamp": "2018-11-29T19:23:05+00:00" }, { "id": "f29a3153569f4e7f873e9bb88463461d", "sender": "meshcollider", "payload": "IMO setting min_conf to 0 is enough?", "action": false, "timestamp": "2018-11-29T19:23:06+00:00" }, { "id": "5495308f68b6426abe9e17e2c1e39c92", "sender": "gmaxwell", "payload": "sipa: wait. I would think that it would include one side of a conflict...", "action": false, "timestamp": "2018-11-29T19:23:08+00:00" }, { "id": "3ba56f468003421190bcf31802b389f2", "sender": "sipa", "payload": "gmaxwell: oh sure; terminology", "action": false, "timestamp": "2018-11-29T19:23:32+00:00" }, { "id": "22cc52ee39074bd3978cb80dcb6ffe9f", "sender": "wumpus", "payload": "isn't it supposed to make one side of a conflict non-conflicting?", "action": false, "timestamp": "2018-11-29T19:23:33+00:00" }, { "id": "f1061379fd3f4692aff46542dcff6bff", "sender": "sipa", "payload": "i guess by non-conflicting i mean \"in the mempool\" ?", "action": false, "timestamp": "2018-11-29T19:23:46+00:00" }, { "id": "89c36bf17b6a4889b8bce075bcde800b", "sender": "wumpus", "payload": "right", "action": false, "timestamp": "2018-11-29T19:23:57+00:00" }, { "id": "a99f230b601f489997fe01dab3b91724", "sender": "gmaxwell", "payload": "Good! jnewbery's comment about balances going down confused me", "action": false, "timestamp": "2018-11-29T19:24:01+00:00" }, { "id": "b55d6f5f0f3d42aca86fa98e70065d5e", "sender": "sipa", "payload": "gmaxwell: or do you think unconfirmed untrusted not-in-the-mempool unconflicted things should be counted?", "action": false, "timestamp": "2018-11-29T19:24:18+00:00" }, { "id": "563866d7f304428bad41358efc01849c", "sender": "jnewbery", "payload": "sorry, I've got to step away now. My main point is that the expected behaviour should be well documented and covered by tests", "action": false, "timestamp": "2018-11-29T19:24:22+00:00" }, { "id": "d58a9c8b48ad4279aa6b6316b641fb7c", "sender": "gmaxwell", "payload": "sipa: no. I don't.", "action": false, "timestamp": "2018-11-29T19:24:33+00:00" }, { "id": "08cceb0681e94447b429c53df38266d9", "sender": "sipa", "payload": "ok, good", "action": false, "timestamp": "2018-11-29T19:24:37+00:00" }, { "id": "2364dec485c747189c5da92a549b1683", "sender": "jnewbery", "payload": "I've written a test script for wallet balances, but we need to agree what the behaviour *should* be before merging any of this stuff", "action": false, "timestamp": "2018-11-29T19:24:52+00:00" }, { "id": "fdf549a2f0254e01ad1492ea4ffd67d7", "sender": "meshcollider", "payload": "Agreed", "action": false, "timestamp": "2018-11-29T19:25:04+00:00" }, { "id": "ce34510383bd48dd8961f904a1256d69", "sender": "provoostenator", "payload": "If something gets confirmed against what you would expect fromt he mempool then a balance change is fine imo, but not otherwise.", "action": false, "timestamp": "2018-11-29T19:25:08+00:00" }, { "id": "8e5db14a62794defb127116add775bac", "sender": "sipa", "payload": "so how about getbalance requires the minconf as directed, and (if trusted is directed, trusted, otherwise in-the-mempool)", "action": false, "timestamp": "2018-11-29T19:25:23+00:00" }, { "id": "424f836bbc934feda0b0672afc9f40a8", "sender": "provoostenator", "payload": "sipa: the only criteria for \"trusted\" is whether it's change, right?", "action": false, "timestamp": "2018-11-29T19:26:03+00:00" }, { "id": "d4ef07ebcfda4ce386d9b37c2b00ed87", "sender": "gmaxwell", "payload": "I think if sipa is right that the behavior can be emulated by getbalance minconf=1 + getunconfirmed then at least we have a workaround. Though it's always confusing to have a deault behavior which cannot be explained by some setting of the arguments", "action": false, "timestamp": "2018-11-29T19:26:09+00:00" }, { "id": "033c182dae974fa6ab31c3eb1f4c0808", "sender": "sipa", "payload": "provoostenator: or confirmed", "action": false, "timestamp": "2018-11-29T19:26:14+00:00" }, { "id": "5d40f37006a7441bac9e5de3125d4ea4", "sender": "gmaxwell", "payload": "\"What is the default minconf setting of getbalance? Mu. getbalance output shows minconf=0 for trusted outputs, and minconf=1 for others. so trusted defaults to true? No, there is no trusted argument.\"", "action": false, "timestamp": "2018-11-29T19:27:27+00:00" }, { "id": "ac6304935a4d4f2facd8eff6b155f776", "sender": "sipa", "payload": "gmaxwell: #14602 includes a trusted_only argument that is true by default", "action": false, "timestamp": "2018-11-29T19:28:18+00:00" }, { "id": "3a53521646234d9ea5f2d08d02dd1f37", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14602 | Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14602 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:28:22+00:00" }, { "id": "94ef293e01bc455e88985a511aadb034", "sender": "provoostenator", "payload": "I know the wallet makes a distinction between change and non-change when it comes to spending, which is fine, but I don't think we should do that for balance. So by default minconf=0 should show change and non-change, but not replaced-by.", "action": false, "timestamp": "2018-11-29T19:28:23+00:00" }, { "id": "6dee13b087164102814cac1676c679dd", "sender": "ryanofsky", "payload": "just want to note, it looks like we didn't delete the GetLegacyBalance function, so it would be trivial to restore the old getbalance \"*\" behavior, if we can't figure out something better right now", "action": false, "timestamp": "2018-11-29T19:28:39+00:00" }, { "id": "1affd4613c704c08948fadadcb08c0bf", "sender": "gmaxwell", "payload": "I know 14602 does. I'm trying to imagine a world where we keep things as they are and try to release note ourselves out of it", "action": false, "timestamp": "2018-11-29T19:28:44+00:00" }, { "id": "c9335ec502ed4e17ad9d09fbcb77680d", "sender": "provoostenator", "payload": "(as for 0.17 I'm fine with release-noting our way of it)", "action": false, "timestamp": "2018-11-29T19:29:28+00:00" }, { "id": "fd80d9be6b3f428b8db53d2c59ad94b6", "sender": "sipa", "payload": "oh, i was just trying to establish what the desired behavior is before we go discuss what to do about it", "action": false, "timestamp": "2018-11-29T19:29:42+00:00" }, { "id": "5e22ee3d0e8b4a2d9ae3130ff9675520", "sender": "provoostenator", "payload": "sipa: that's a good idea", "action": false, "timestamp": "2018-11-29T19:30:11+00:00" }, { "id": "423c760a807f45bf89a5687b56c840d6", "sender": "sipa", "payload": "and i think there is at least a rough conclusion? allow counting untrusted, but never count not-in-mempool?", "action": false, "timestamp": "2018-11-29T19:30:37+00:00" }, { "id": "49d643924a7b48a6b9d70592521714c4", "sender": "gmaxwell", "payload": "Right, I think the desired behavior is that balances shows confirmed + trusted-in-mempool-0conf. And that you be able to adjust the threshold for confirmed and turn the trusted-in-mempool-zeroconf on and off. And if you make the confirmed=0, it should still only count in-mempool", "action": false, "timestamp": "2018-11-29T19:31:12+00:00" }, { "id": "e84b848cf2634b658e3fa4fd697caa67", "sender": "provoostenator", "payload": "(and maybe add a 4th argument along the lines of unconfirmed_change_only", "action": false, "timestamp": "2018-11-29T19:31:39+00:00" }, { "id": "86e4b7d69c8f49c1a04e5eba3887cea3", "sender": "gmaxwell", "payload": "(and potentially the arguments should just be minconf_for_not_you=1 minconf_for_from_you=0 )", "action": false, "timestamp": "2018-11-29T19:31:53+00:00" }, { "id": "18dba661d2934d11bc5650815841bba5", "sender": "sipa", "payload": "i think the default of minconf=0 but trusted_only=true makes sense; if you choose trusted_only=false it should still only include confirmed or mempool things", "action": false, "timestamp": "2018-11-29T19:32:15+00:00" }, { "id": "6c3395e6ddcb4f398f6bbe64b2efa319", "sender": "morcos", "payload": "sipa: the problem with never counting not-in-mempool is if you spend a 10 BTC input to pay 0.001 BTC, and that tx doesn't go in your mempool then your balance goes down by 10 mysteriously, but thats hard to solve", "action": false, "timestamp": "2018-11-29T19:32:21+00:00" }, { "id": "81d7d12a69a44cbf9225f54b1518d2af", "sender": "sipa", "payload": "morcos: yeah...", "action": false, "timestamp": "2018-11-29T19:32:36+00:00" }, { "id": "b2a4be34dd45474081a3b676628a904a", "sender": "gmaxwell", "payload": "morcos: yes, I think thats hard to solve, also it's kinda good that it goes down in that you're actually screwed for the moment :P", "action": false, "timestamp": "2018-11-29T19:32:53+00:00" }, { "id": "a69799710c06452ebee448751dff4fdf", "sender": "gmaxwell", "payload": "like if that txn doesn't confirm, your funds are actually stuck.", "action": false, "timestamp": "2018-11-29T19:33:07+00:00" }, { "id": "0a97ee1ca574480d84f1b689639a7de9", "sender": "gmaxwell", "payload": "(until you abandon the transaction)", "action": false, "timestamp": "2018-11-29T19:33:38+00:00" }, { "id": "a4c82eb8fc1141d78ad03f92c90201b6", "sender": "wumpus", "payload": "right", "action": false, "timestamp": "2018-11-29T19:33:41+00:00" }, { "id": "2e248a0645764d5892d90b253fd041a9", "sender": "gmaxwell", "payload": "I guess I can argue the consequences either way, I can imagine someone losing a wallet file with lots of bitcoin in it because it showed a low balance due to some easily abandoned transactions.", "action": false, "timestamp": "2018-11-29T19:34:07+00:00" }, { "id": "1ba60ee825084882a6202abbf8354cce", "sender": "sipa", "payload": "so i think we can work out how to implement this for 0.17 and 0.18 on the PRs or issues; no need for further design here?", "action": false, "timestamp": "2018-11-29T19:34:08+00:00" }, { "id": "a81a3fcb4ebd461c81dae5e52da2dc2e", "sender": "gmaxwell", "payload": "I thought luke's PR basically implemented what we came up with here. But I'm not sure.", "action": false, "timestamp": "2018-11-29T19:34:52+00:00" }, { "id": "2193458f1f6e4f3ea5d724f55b0adabe", "sender": "sipa", "payload": "gmaxwell: i'm not sure about the mempool part", "action": false, "timestamp": "2018-11-29T19:35:07+00:00" }, { "id": "cfa5f04339a744cfa16262c6ec7793b3", "sender": "wumpus", "payload": "gmaxwell: people are likely to use 'getbalance' without parameters in that case, to check wallet balance, not the specific combo that this is about", "action": false, "timestamp": "2018-11-29T19:35:21+00:00" }, { "id": "9fe41986762441ba8c368b39d33bc8e7", "sender": "gmaxwell", "payload": "certantly if we're showing all unconfirmeds, including both sides of a conflict, thats bad. :P", "action": false, "timestamp": "2018-11-29T19:35:28+00:00" }, { "id": "ef439c9178354052926a775644150d3e", "sender": "provoostenator", "payload": "Luke's PR adds a trusted_only as a first parameter. I find that a fairly confusing term, so hopefully we can avoid it with the ^", "action": false, "timestamp": "2018-11-29T19:36:24+00:00" }, { "id": "33a6d85cd71d411b9beb2ffa59e121a1", "sender": "sipa", "payload": "with the what?", "action": false, "timestamp": "2018-11-29T19:36:37+00:00" }, { "id": "578449f3ef624df489d9b81227f155a1", "sender": "wumpus", "payload": "trusted_only sounds really confusing to users, if you make a parameter like that you need to carefully explain in the RPC help what 'truste'd means in this setting", "action": false, "timestamp": "2018-11-29T19:37:02+00:00" }, { "id": "57bfe2c9884b4682a13d4eb242fe5f06", "sender": "sipa", "payload": "agree, it needs documentation for sure", "action": false, "timestamp": "2018-11-29T19:37:13+00:00" }, { "id": "4762aece35fa410f8189585de3ea67be", "sender": "wumpus", "payload": "it's far from obvious, I don't know if we picked a good term for it", "action": false, "timestamp": "2018-11-29T19:37:23+00:00" }, { "id": "f07a2e3863c5421094de974318043572", "sender": "provoostenator", "payload": "With what you said: \"allow counting untrusted, but never count not-in-mempool\"", "action": false, "timestamp": "2018-11-29T19:37:36+00:00" }, { "id": "0026029565444f62b845e005d2776657", "sender": "provoostenator", "payload": "It's already a default, so maybe just a matter of renaming and documenting it.", "action": false, "timestamp": "2018-11-29T19:38:07+00:00" }, { "id": "3f2e060583b440dea94a85f84bc26687", "sender": "sipa", "payload": "provoostenator: it can certainly be explained more friendly as \"Whether unconfirmed payments from others are excluded\"", "action": false, "timestamp": "2018-11-29T19:38:47+00:00" }, { "id": "5915d25552de4a96b71894d60ede7fcf", "sender": "provoostenator", "payload": "1. zero_conf_change_only (default: false)", "action": false, "timestamp": "2018-11-29T19:38:58+00:00" }, { "id": "bf2c80cb416c4b359c24fe6020d335f3", "sender": "wumpus", "payload": "sipa: that sounds much better", "action": false, "timestamp": "2018-11-29T19:39:07+00:00" }, { "id": "0e7e67fbb5f64361a4c5140a8b55c142", "sender": "sipa", "payload": "i think we're getting in the weeds here", "action": false, "timestamp": "2018-11-29T19:39:15+00:00" }, { "id": "0e11ca08de594116bbbe9a98f8600eb8", "sender": "wumpus", "payload": "yes, time for next otpic?", "action": false, "timestamp": "2018-11-29T19:39:34+00:00" }, { "id": "cc46c6b2c9d64ef6bb1639caf93b1475", "sender": "sipa", "payload": "agree", "action": false, "timestamp": "2018-11-29T19:39:41+00:00" }, { "id": "38439800ac604c3bbdd87e2c6cafe4ca", "sender": "wumpus", "payload": "#topic 0.17.1", "action": false, "timestamp": "2018-11-29T19:39:43+00:00" }, { "id": "d3587cb675824842bb9ee8c2414c5316", "sender": "provoostenator", "payload": "Next topic sounds good.", "action": false, "timestamp": "2018-11-29T19:39:43+00:00" }, { "id": "8fd4ce1de8d247bca978cef3d0454d2f", "sender": "BlueMatt", "payload": "release time!", "action": false, "timestamp": "2018-11-29T19:39:50+00:00" }, { "id": "10a74d74df5c47b89bbd819f35cc4ccb", "sender": "wumpus", "payload": "20 minutes left", "action": false, "timestamp": "2018-11-29T19:39:52+00:00" }, { "id": "05c57eb23564466ab71ab3008c7e0faa", "sender": "BlueMatt", "payload": "cool, release in 20 minutes? :p", "action": false, "timestamp": "2018-11-29T19:40:01+00:00" }, { "id": "de9dd8bdabe246c9b22552c180cabebe", "sender": "provoostenator", "payload": "BlueMatt is the main reason Ubutnu?", "action": false, "timestamp": "2018-11-29T19:40:02+00:00" }, { "id": "6e68d975435b48e08e28a806febae297", "sender": "meshcollider", "payload": "Lol", "action": false, "timestamp": "2018-11-29T19:40:07+00:00" }, { "id": "077590b0a9c24ef8bb259920e9f7e7dc", "sender": "BlueMatt", "payload": "provoostenator: also fedora, debian testing, .......", "action": false, "timestamp": "2018-11-29T19:40:16+00:00" }, { "id": "dc982682a9b64e15b8c2f4d759ca0518", "sender": "provoostenator", "payload": "Because an alternative could be 0.17.0.2 like we did with MacOS. Though I'm fine with proper 0.17.1 too", "action": false, "timestamp": "2018-11-29T19:40:22+00:00" }, { "id": "37ca73d1f4c7478eb6bff3b804fd5372", "sender": "wumpus", "payload": "still lots of backports open https://github.com/bitcoin/bitcoin/milestone/39 I guess they need review, or at least merging", "action": false, "timestamp": "2018-11-29T19:40:25+00:00" }, { "id": "f12e2e52781b4f129aee0073a3002b5b", "sender": "BlueMatt", "payload": "provoostenator: also there are a number of other bugs and backports that are done, etc", "action": false, "timestamp": "2018-11-29T19:40:33+00:00" }, { "id": "1cb3fe53bc2a4f71b5005c7f81760fbb", "sender": "sipa", "payload": "we have a bunch of bugfixes lined up too", "action": false, "timestamp": "2018-11-29T19:40:36+00:00" }, { "id": "f80bf8f7c5e640cb98bc889b6a0c448b", "sender": "wumpus", "payload": "nah let's do a proper release", "action": false, "timestamp": "2018-11-29T19:40:37+00:00" }, { "id": "6024dccdd2564e2d9b2d61233ee52f88", "sender": "sipa", "payload": "it may be time for a 0.17.1", "action": false, "timestamp": "2018-11-29T19:40:40+00:00" }, { "id": "b48f040768554aceb293405dfc517b15", "sender": "BlueMatt", "payload": "things already merged on master probably worth backporting", "action": false, "timestamp": "2018-11-29T19:40:41+00:00" }, { "id": "c3d1fed74f9045bf864d9604501d4ebb", "sender": "wumpus", "payload": "it's too late for 0.17.0.2 IMO", "action": false, "timestamp": "2018-11-29T19:40:43+00:00" }, { "id": "4a938f930da94e2598d49d6f200cb722", "sender": "BlueMatt", "payload": "cause there are a bunch there", "action": false, "timestamp": "2018-11-29T19:40:46+00:00" }, { "id": "d0ca71165bb64df6bd515a1ec0094316", "sender": "provoostenator", "payload": "Right, that would involve branching off of the 0.17.0.1 tag and get weird.", "action": false, "timestamp": "2018-11-29T19:41:16+00:00" }, { "id": "d2dee9c18360429aa52974bda9069897", "sender": "wumpus", "payload": ".z releases are really 'oops we did a release and need to fix this thing up'", "action": false, "timestamp": "2018-11-29T19:41:19+00:00" }, { "id": "5c5fbbbd7f9a48c0b770320861da6c15", "sender": "BlueMatt", "payload": "but, at least IMO, its super annoying that you cant build+run bitcoin-qt on the now-several-months-old versions of various OSs, and I presume other rolling distros (eg arch) also have this issue", "action": false, "timestamp": "2018-11-29T19:41:20+00:00" }, { "id": "85b491a182554121817bca2f7287e5dc", "sender": "wumpus", "payload": "not for month later", "action": false, "timestamp": "2018-11-29T19:41:24+00:00" }, { "id": "891532a938824f58ad8ba9563a523cc8", "sender": "instagibbs", "payload": "I keep crashing, get 14380 in please ;P", "action": false, "timestamp": "2018-11-29T19:41:33+00:00" }, { "id": "fc207c1d7a03463eacf8874d712d8340", "sender": "BlueMatt", "payload": "so I would prefer we not wait around for fixing getbalance :p", "action": false, "timestamp": "2018-11-29T19:41:50+00:00" }, { "id": "a0ab44e87fe34b2294d86978602d5d34", "sender": "wumpus", "payload": "I agree with BlueMatt on this", "action": false, "timestamp": "2018-11-29T19:41:56+00:00" }, { "id": "2a185b9778da4cbd97a82ee5e2bc0b4e", "sender": "BlueMatt", "payload": "#14380 looks more than merge-able from where I'm sitting in the ack department", "action": false, "timestamp": "2018-11-29T19:42:05+00:00" }, { "id": "4a89eb1726024799b76626f5ff316a78", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14380 | fix assert crash when specified change output spend size is unknown by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #14380 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:42:08+00:00" }, { "id": "885f5d0582dc4935ba384db12d1873fc", "sender": "BlueMatt", "payload": "and not too hard to backport", "action": false, "timestamp": "2018-11-29T19:42:13+00:00" }, { "id": "3f537d3824944b2dbfb3464552a151f0", "sender": "sipa", "payload": "can people have a look at #14780 too?", "action": false, "timestamp": "2018-11-29T19:42:44+00:00" }, { "id": "4d8aae3fdb3840fa9c4b16097470977d", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14780 | PSBT backports to 0.17 by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14780 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:42:46+00:00" }, { "id": "d8748875342c4ee9b2bfd8a509888ea0", "sender": "sipa", "payload": "(i'm mostly unfamiliar with the backports process)", "action": false, "timestamp": "2018-11-29T19:42:59+00:00" }, { "id": "0485a2b07e2b40e6ada1a7da75766686", "sender": "wumpus", "payload": "there is no agreement on getbalance yet, it can wait for 0.17.2", "action": false, "timestamp": "2018-11-29T19:43:01+00:00" }, { "id": "024ab953821b4dfb9808ffc95e831aac", "sender": "wumpus", "payload": "sipa: sure!", "action": false, "timestamp": "2018-11-29T19:43:07+00:00" }, { "id": "2cc97558585f4e4e89cb949af239e460", "sender": "sipa", "payload": "thanks", "action": false, "timestamp": "2018-11-29T19:43:14+00:00" }, { "id": "fb64aa11f36b45cd95ac194857fe735b", "sender": "wumpus", "payload": "sipa: please make sure you include the commit metadata (Github-Pull: #.... and Rebased-From: )", "action": false, "timestamp": "2018-11-29T19:43:59+00:00" }, { "id": "35d11779207d44c1855ff52e332c9b42", "sender": "BlueMatt", "payload": "next topic: the wall of text I just posted to bitcoin-dev :p", "action": false, "timestamp": "2018-11-29T19:44:10+00:00" }, { "id": "943cf48e628e40ad9a0ef4a2e98c066a", "sender": "wumpus", "payload": "(see the commits in other backport PRs such as https://github.com/bitcoin/bitcoin/pull/14835)", "action": false, "timestamp": "2018-11-29T19:44:13+00:00" }, { "id": "f0d09ed322994b40ad1b3941f8619d82", "sender": "sipa", "payload": "BlueMatt: not there", "action": false, "timestamp": "2018-11-29T19:44:32+00:00" }, { "id": "e633891cccff43739f8f2ec125900ee5", "sender": "wumpus", "payload": "next topic is moneyball's", "action": false, "timestamp": "2018-11-29T19:44:34+00:00" }, { "id": "de3ff8f063834e2d9c4ec89d34b0807b", "sender": "moneyball", "payload": "hi", "action": false, "timestamp": "2018-11-29T19:44:41+00:00" }, { "id": "be0de0b678544233bda99fd1f36235c8", "sender": "BlueMatt", "payload": "err, topic suggestion", "action": false, "timestamp": "2018-11-29T19:44:41+00:00" }, { "id": "9fb216c25aaf48d89c56c402acbe0d63", "sender": "BlueMatt", "payload": "anything else on 0.17.1?", "action": false, "timestamp": "2018-11-29T19:44:50+00:00" }, { "id": "763b2b75dac04f52bb01901f1419dd6c", "sender": "BlueMatt", "payload": "or just generally a \"lets take things that we can merge today, do backports, and get this thing going\"", "action": false, "timestamp": "2018-11-29T19:45:01+00:00" }, { "id": "ac86c843a0f6438c811989dd08e1974d", "sender": "provoostenator", "payload": "BlueMatt: \"IRC logs have been disabled due to the EU General Data Protection Regulation (GDPR).\"", "action": false, "timestamp": "2018-11-29T19:45:04+00:00" }, { "id": "e0bdb5160b0745509df50f5fb3591763", "sender": "wumpus", "payload": "BlueMatt: +1", "action": false, "timestamp": "2018-11-29T19:45:16+00:00" }, { "id": "6b96665c8b554d58887170b77b269afd", "sender": "provoostenator", "payload": "+1", "action": false, "timestamp": "2018-11-29T19:45:37+00:00" }, { "id": "c457be56ce174da38e077207127c8135", "sender": "wumpus", "payload": "#topic proposed meeting topics ahead of time (moneyball)", "action": false, "timestamp": "2018-11-29T19:45:57+00:00" }, { "id": "a30ebd1756004f98948cbb5f8b26f31b", "sender": "moneyball", "payload": "I\u00c3\u00a2\u00c2\u0080\u00c2\u0099d like to suggest that people can propose Core weekly meeting topics anytime throughout the week in this IRC channel using #proposedmeetingtopic tag. I will collect these throughout the week, add them to a gist that anyone can have a look at during the week. The primary intended benefit of this is to allow people to know which topics are going to be discussed ahead of time allowing them to prepare.", "action": false, "timestamp": "2018-11-29T19:46:02+00:00" }, { "id": "68f836ffe26b4339891bfd92da11eb91", "sender": "moneyball", "payload": "Any concerns?", "action": false, "timestamp": "2018-11-29T19:46:26+00:00" }, { "id": "8badb5588c8d4fa794bc282375bfb382", "sender": "BlueMatt", "payload": "seems reasonable", "action": false, "timestamp": "2018-11-29T19:46:31+00:00" }, { "id": "e7f10c1c9b9a42abb2cf24475af2f00a", "sender": "provoostenator", "payload": "+1", "action": false, "timestamp": "2018-11-29T19:46:36+00:00" }, { "id": "14b154b86d144439bbce34b0e6418074", "sender": "BlueMatt", "payload": "easier than remembering to mention them in meeting", "action": false, "timestamp": "2018-11-29T19:46:38+00:00" }, { "id": "6d2e475f036c47b8b528e3144819fc09", "sender": "Chris_Stewart_5", "payload": "+1", "action": false, "timestamp": "2018-11-29T19:46:42+00:00" }, { "id": "02fe61a70095446b9fc7cf64346dab8d", "sender": "meshcollider", "payload": "+1", "action": false, "timestamp": "2018-11-29T19:46:46+00:00" }, { "id": "496a95a7b4714a4c96bee0a31f756647", "sender": "BlueMatt", "payload": "next topic? #thatwaseasy", "action": false, "timestamp": "2018-11-29T19:46:52+00:00" }, { "id": "21f4acc087b147838e3adb14f03405c5", "sender": "wumpus", "payload": "yes, good idea, let's see how it goes", "action": false, "timestamp": "2018-11-29T19:47:03+00:00" }, { "id": "fc392f33cef048da9047e2d0a5a57fd4", "sender": "moneyball", "payload": "Ok I will start doing it!", "action": false, "timestamp": "2018-11-29T19:47:08+00:00" }, { "id": "8c57c6967a00489987bab258e8551c1e", "sender": "wumpus", "payload": "BlueMatt: ok what was your proposal, I'm not in #bitcoin-dev", "action": false, "timestamp": "2018-11-29T19:47:15+00:00" }, { "id": "53fb3ba475254ddc880bbdb4e49044fd", "sender": "provoostenator", "payload": "And there's no logs..", "action": false, "timestamp": "2018-11-29T19:47:22+00:00" }, { "id": "4adfca3ff0f04ede8bbe9ae4b961a62f", "sender": "BlueMatt", "payload": "oh, i meant bitcoin-dev ml but it hasnt passed moderation yet", "action": false, "timestamp": "2018-11-29T19:47:32+00:00" }, { "id": "898e7f915b9a48edae3ab08a23e97c5c", "sender": "BlueMatt", "payload": "anyway, I can tl;dr it in two lines", "action": false, "timestamp": "2018-11-29T19:47:36+00:00" }, { "id": "8829f22c3591413a9e5e0791540c18b2", "sender": "BlueMatt", "payload": "in lightning (and other related payment channel-y systems) you have the general issue of pre-signing transactions", "action": false, "timestamp": "2018-11-29T19:47:53+00:00" }, { "id": "9f8467991c28454791ae09ed7df50b44", "sender": "BlueMatt", "payload": "so you have to predict what the feerate on the network will be in the future at the time your counterparty misbehaves", "action": false, "timestamp": "2018-11-29T19:48:06+00:00" }, { "id": "4d5d7b9779554e5d912906d925af6008", "sender": "wumpus", "payload": "#topic pre-signing transactions (BlueMatt)", "action": false, "timestamp": "2018-11-29T19:48:09+00:00" }, { "id": "204670de82a74901bcaf2590db35820d", "sender": "BlueMatt", "payload": "this is....obviously insane", "action": false, "timestamp": "2018-11-29T19:48:09+00:00" }, { "id": "c5cdb2b7eaf844faaa0b64d80bb2cb7b", "sender": "BlueMatt", "payload": "instead, we'd (obviously) like to use cpfp", "action": false, "timestamp": "2018-11-29T19:48:18+00:00" }, { "id": "d535865bda8f4ca285286d053ede0718", "sender": "BlueMatt", "payload": "but there are the cpfp rbf-pinning-ish issues where your counterparty can fill up the package with big-bug-low-ish-feerate txn", "action": false, "timestamp": "2018-11-29T19:48:38+00:00" }, { "id": "eda8c1cec6cd4965bd02983246f76da5", "sender": "BlueMatt", "payload": "so that you cant cpfp it yourself", "action": false, "timestamp": "2018-11-29T19:48:43+00:00" }, { "id": "1c4333405e5646269ab7f61debaeed12", "sender": "BlueMatt", "payload": "I'd like to propose we tweak the cpfp/rbf rules", "action": false, "timestamp": "2018-11-29T19:49:03+00:00" }, { "id": "85c6a0e524be429baf866cb37230bec1", "sender": "BlueMatt", "payload": "The last transaction which is added to a package of dependent transactions in the mempool must:", "action": false, "timestamp": "2018-11-29T19:49:04+00:00" }, { "id": "83aaef197eee400083403e25bbd68036", "sender": "BlueMatt", "payload": "* Have no more than one unconfirmed parent,", "action": false, "timestamp": "2018-11-29T19:49:04+00:00" }, { "id": "86f9fc9b385c4001bfebd288e0ae0278", "sender": "BlueMatt", "payload": "* Be of size no greater than 1K in virtual size.", "action": false, "timestamp": "2018-11-29T19:49:04+00:00" }, { "id": "1ae6a1de203240a0bf7448c401bc606f", "sender": "BlueMatt", "payload": "(we'd, in practice, first decrease the package size limits by 1 1K-virtual-size transaction, then make it so that the last one must meet those rules)", "action": false, "timestamp": "2018-11-29T19:49:25+00:00" }, { "id": "ddd988c0d2d2471d94b80aae53031171", "sender": "BlueMatt", "payload": "this allows for cpfp by letting each side of a lightning channel independantly cpfp without considering what the other side may be doing", "action": false, "timestamp": "2018-11-29T19:49:47+00:00" }, { "id": "c5eb8a5bbb424cd7ba60f0b7d7c041a8", "sender": "provoostenator", "payload": "Current RBF rules for reference: https://github.com/bitcoin/bips/blob/master/bip-0125.mediawiki#Implementation_Details", "action": false, "timestamp": "2018-11-29T19:49:48+00:00" }, { "id": "9ff1241725a14fb2916c983ccf95901d", "sender": "BlueMatt", "payload": "that may be too much to think about before reading the full ml post, but mostly it just covers background material", "action": false, "timestamp": "2018-11-29T19:50:08+00:00" }, { "id": "d9417a27138742dcb27fe606116c90e5", "sender": "BlueMatt", "payload": "/fin", "action": false, "timestamp": "2018-11-29T19:50:18+00:00" }, { "id": "561f9cdb53da49d990ce57c56ddfa593", "sender": "BlueMatt", "payload": "(oh, also this fix requires package relay, which is another can of worms itself)", "action": false, "timestamp": "2018-11-29T19:50:37+00:00" }, { "id": "36ce63511f8249f4b034f7b7989796c2", "sender": "sipa", "payload": "here is another earlier proposed modification: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2018-February/015717.html", "action": false, "timestamp": "2018-11-29T19:50:40+00:00" }, { "id": "235c599631574658844cc598db213856", "sender": "sipa", "payload": "BlueMatt: oh dear", "action": false, "timestamp": "2018-11-29T19:50:47+00:00" }, { "id": "654ccab113ee4c04adbec62e4fd8b443", "sender": "BlueMatt", "payload": "well, luckily only one-deep package relay and only for small-ish packages", "action": false, "timestamp": "2018-11-29T19:51:16+00:00" }, { "id": "d90aefd56498402a8f966e72f26e6a79", "sender": "BlueMatt", "payload": "or at least small dependents", "action": false, "timestamp": "2018-11-29T19:51:19+00:00" }, { "id": "bfd66d83a0db48c1a399001c4f5340f2", "sender": "instagibbs", "payload": "sorry how does cpfp get pinned again?", "action": false, "timestamp": "2018-11-29T19:51:48+00:00" }, { "id": "bb06b03c6587463d92ae9566acb92bb9", "sender": "instagibbs", "payload": "RBF pinning I know", "action": false, "timestamp": "2018-11-29T19:51:57+00:00" }, { "id": "6f7f06a996514ac4a1dcd8d6b8ee1c68", "sender": "BlueMatt", "payload": "instagibbs: A -> B where B is 100KB but low feerate, but high fee. its the same issue for lightning", "action": false, "timestamp": "2018-11-29T19:52:05+00:00" }, { "id": "f658e2b8d37d4d50ae2c875be7b481f0", "sender": "BlueMatt", "payload": "cause you cant add to the package cause the package is max size", "action": false, "timestamp": "2018-11-29T19:52:11+00:00" }, { "id": "c303e93f8fcc4174ae503581392fa14d", "sender": "instagibbs", "payload": "ah", "action": false, "timestamp": "2018-11-29T19:52:22+00:00" }, { "id": "716466fce9c24af0afe856f216e0dfaa", "sender": "sipa", "payload": "my response to this is \"eh... sure.... but how?\"", "action": false, "timestamp": "2018-11-29T19:52:33+00:00" }, { "id": "1a7b5a3baa214964be0064d3b97633fc", "sender": "instagibbs", "payload": "so it's not something you can attach B'", "action": false, "timestamp": "2018-11-29T19:52:34+00:00" }, { "id": "0a9bbd1a73be427f8573f826177d19f8", "sender": "BlueMatt", "payload": "so you have to add to the package by rbf which means you have to increase total size, if you can even do that, which you'd have to tweak rbf rules to let you replace B with C, but you cant do that today", "action": false, "timestamp": "2018-11-29T19:52:36+00:00" }, { "id": "4391614559dd47a2b7b3bc1c5c211444", "sender": "BlueMatt", "payload": "sipa: hmm?", "action": false, "timestamp": "2018-11-29T19:52:46+00:00" }, { "id": "a0a1f8b34d224fad826a8c58de10a242", "sender": "sipa", "payload": "you need package relay...", "action": false, "timestamp": "2018-11-29T19:52:52+00:00" }, { "id": "102b67356c404c8d8c1526019f39524b", "sender": "sipa", "payload": "that's not just a small few-lines policy change", "action": false, "timestamp": "2018-11-29T19:53:17+00:00" }, { "id": "87f187ac29074a6db73d9b233db2ef19", "sender": "provoostenator", "payload": "An alternative, I forgot what the problem was, is to drop RBF rule \"3. The replacement transaction pays an absolute fee of at least the sum paid by the original transactions.\" and only consider fee rate of the replacement, regardless of size.", "action": false, "timestamp": "2018-11-29T19:53:19+00:00" }, { "id": "30bf0b828a894a8aa4ad634e6fee5e1d", "sender": "sdaftuar", "payload": "so i think package relay is a whole other topic", "action": false, "timestamp": "2018-11-29T19:53:23+00:00" }, { "id": "49f0e07193f54efc8ae26b5eee8ffa7c", "sender": "BlueMatt", "payload": "I have another bitcoin-dev post coming discussing package relay, but we have a few options there, if we just want to solve package relay for contracting applications like lightning, we can do so with limited consideration, I think", "action": false, "timestamp": "2018-11-29T19:53:27+00:00" }, { "id": "0c1aff464d44465d9aaa5f8870d24451", "sender": "BlueMatt", "payload": "if we want a more general solution I still think its possible but requires more in-depth ATMP refactoring", "action": false, "timestamp": "2018-11-29T19:53:48+00:00" }, { "id": "d5238b63319a44e08ee25297fc99c1ff", "sender": "sipa", "payload": "and P2P changes...", "action": false, "timestamp": "2018-11-29T19:54:00+00:00" }, { "id": "beea10698a2d4fb697197733946dfdec", "sender": "BlueMatt", "payload": "assuming we do at least one of those two, this is a simple (I think) policy change that I'm hoping people here dont object to", "action": false, "timestamp": "2018-11-29T19:54:05+00:00" }, { "id": "877e42a4965840119533da1138d290ac", "sender": "BlueMatt", "payload": "possibly, though actaully not neccessarily", "action": false, "timestamp": "2018-11-29T19:54:15+00:00" }, { "id": "ea74aa6decf64b41ad6dd1e30f5c6106", "sender": "BlueMatt", "payload": "but, probably", "action": false, "timestamp": "2018-11-29T19:54:21+00:00" }, { "id": "36f3de85cca6428397619aaa9a70e164", "sender": "sipa", "payload": "this sounds very preliminary to me", "action": false, "timestamp": "2018-11-29T19:54:34+00:00" }, { "id": "397e045c38f74cc99ae96e5882e27b9f", "sender": "BlueMatt", "payload": "i presume you mean the package relay part?", "action": false, "timestamp": "2018-11-29T19:54:48+00:00" }, { "id": "11dbcf417d81470ba1fff329499f7a8b", "sender": "BlueMatt", "payload": "the policy rule here is pretty straightforward, I think?", "action": false, "timestamp": "2018-11-29T19:54:59+00:00" }, { "id": "19742bc4050d4d2081271696f2f95e99", "sender": "BlueMatt", "payload": "and not very preliminary, again assuming some form of package relay", "action": false, "timestamp": "2018-11-29T19:55:06+00:00" }, { "id": "eb07aaad86dc45d4af98d570d2602aaa", "sender": "sipa", "payload": "i don't understand how you can say it's not preliminary, while relying on something we have no idea about yet", "action": false, "timestamp": "2018-11-29T19:55:32+00:00" }, { "id": "a16f25e2dad4424ab53bcb43eec2890a", "sender": "meshcollider", "payload": "Shall we discuss next week instead after reading the ml", "action": false, "timestamp": "2018-11-29T19:55:49+00:00" }, { "id": "5ddc5b1143b447c4bce56d717297bd47", "sender": "gmaxwell", "payload": "MAGIC", "action": false, "timestamp": "2018-11-29T19:55:50+00:00" }, { "id": "57a7aee46ba944d1bb74779da1af4486", "sender": "BlueMatt", "payload": "heh, sdaftuar and I have been batting package relay back and forth for months, but, whatever", "action": false, "timestamp": "2018-11-29T19:55:55+00:00" }, { "id": "f9d413b84b3844488201873764862a72", "sender": "BlueMatt", "payload": "agreed we need a harder proposal for package relay", "action": false, "timestamp": "2018-11-29T19:56:03+00:00" }, { "id": "2550ac19720a4c108f5b0df678a10481", "sender": "sdaftuar", "payload": "well i think it'd be useful to layout the design goals for a package relay ssytem", "action": false, "timestamp": "2018-11-29T19:56:11+00:00" }, { "id": "783a21779c95417f8a841ab174a86da7", "sender": "BlueMatt", "payload": "anyway, yes, we'll discuss package relay on the ml and then come back to this", "action": false, "timestamp": "2018-11-29T19:56:12+00:00" }, { "id": "c40597d171bf499cb2cfb48c4abdd1db", "sender": "sipa", "payload": "BlueMatt: yes i'd like to see a proposal for that :)", "action": false, "timestamp": "2018-11-29T19:56:13+00:00" }, { "id": "c027dbf617344335acd7029d8e34659f", "sender": "gmaxwell", "payload": "BlueMatt: the point is that the policy change justification is a black box to us for the moment. :P", "action": false, "timestamp": "2018-11-29T19:56:16+00:00" }, { "id": "376c01039b494c2faed2bb820dd1db91", "sender": "BlueMatt", "payload": "also what sdaftuar said", "action": false, "timestamp": "2018-11-29T19:56:20+00:00" }, { "id": "aa71d759649c435ebeafc38026cffa10", "sender": "BlueMatt", "payload": "this is also a motivator for package relay", "action": false, "timestamp": "2018-11-29T19:56:28+00:00" }, { "id": "da2042f333fa477f8f14510a160b8e2e", "sender": "BlueMatt", "payload": "as its yet another thing to feed into the design goals", "action": false, "timestamp": "2018-11-29T19:56:36+00:00" }, { "id": "58a66af089c34864b2ba44a0a1786721", "sender": "BlueMatt", "payload": "gmaxwell: I mean if you presume package relay it isnt at all a black box?", "action": false, "timestamp": "2018-11-29T19:56:57+00:00" }, { "id": "bee8953c62d042aa96544d1848f4c004", "sender": "BlueMatt", "payload": "huh", "action": false, "timestamp": "2018-11-29T19:56:58+00:00" }, { "id": "9a3bc653eee94d2e83a651f7de6dca43", "sender": "BlueMatt", "payload": "anyway, -> ml", "action": false, "timestamp": "2018-11-29T19:57:05+00:00" }, { "id": "2447b4995dfb4ee08e722f516a6a69e7", "sender": "provoostenator", "payload": "Topic suggestion: dandelion stempool relay :-P", "action": false, "timestamp": "2018-11-29T19:57:06+00:00" }, { "id": "c77c971d9f0b471b8ac747ab69068631", "sender": "sdaftuar", "payload": "hides", "action": true, "timestamp": "2018-11-29T19:57:14+00:00" }, { "id": "071ebab397c24056b2c29715096b4fd9", "sender": "BlueMatt", "payload": "4 minutes, GO", "action": false, "timestamp": "2018-11-29T19:57:14+00:00" }, { "id": "a241d46718a84025b0ef44d6417181af", "sender": "BlueMatt", "payload": "err, 3", "action": false, "timestamp": "2018-11-29T19:57:19+00:00" }, { "id": "23f1c6bf0c7946d7a49b0b40b6444797", "sender": "instagibbs", "payload": "sdaftuar, you ahve 3 minutes to resolve it", "action": false, "timestamp": "2018-11-29T19:57:27+00:00" }, { "id": "65f800a2cade4040bfc70d9901640787", "sender": "wumpus", "payload": "#topic wallet maintainer", "action": false, "timestamp": "2018-11-29T19:58:19+00:00" }, { "id": "4779672e4a234a99bd96d9044604ec4d", "sender": "wumpus", "payload": "meshcollider has shown interest in becoming wallet maintainer!", "action": false, "timestamp": "2018-11-29T19:58:42+00:00" }, { "id": "70d1888db0ed47bdafa04a42d598aaeb", "sender": "provoostenator", "payload": "Awesome", "action": false, "timestamp": "2018-11-29T19:59:01+00:00" }, { "id": "cfff5f22a0a14253ae9d261b2cbe3d0c", "sender": "sipa", "payload": "offers condolences", "action": true, "timestamp": "2018-11-29T19:59:03+00:00" }, { "id": "f17b2f5dd36e4f83b4a1afea41a6e6b5", "sender": "sipa", "payload": ":)", "action": false, "timestamp": "2018-11-29T19:59:12+00:00" }, { "id": "f91d2cf68ad34eadbbdfab2c98f80a9b", "sender": "provoostenator", "payload": "Anyoe against, no, done. End meeting :-)", "action": false, "timestamp": "2018-11-29T19:59:21+00:00" }, { "id": "5efeff96f42547b5af524a05e341609e", "sender": "wumpus", "payload": "which is great news, I think he'd be good at his, many of his his contributions have been to the wallet", "action": false, "timestamp": "2018-11-29T19:59:31+00:00" }, { "id": "b9044f4309bc41f5a11c5d5dbd49e05b", "sender": "meshcollider", "payload": ":)", "action": false, "timestamp": "2018-11-29T19:59:33+00:00" }, { "id": "f0133e6d102d499791f73482ffa8a403", "sender": "wumpus", "payload": "and he's been active contributor for quite a while", "action": false, "timestamp": "2018-11-29T19:59:59+00:00" }, { "id": "b789d584e941457d9f82846839fbba74", "sender": "sipa", "payload": "more seriously, sounds great to me", "action": false, "timestamp": "2018-11-29T20:00:00+00:00" }, { "id": "37d4bc857252447fae4b824d7673b678", "sender": "achow101", "payload": "+1", "action": false, "timestamp": "2018-11-29T20:00:05+00:00" }, { "id": "2d6a39719ad846b8a3d00a51e62d697f", "sender": "wumpus", "payload": "yes!", "action": false, "timestamp": "2018-11-29T20:00:05+00:00" }, { "id": "3e2cc35cf21c477cafd9fa2b8015f86d", "sender": "bitcoin-git", "payload": "[bitcoin] vim88 opened pull request #14846: Docs: Adds development guidelines about Scripts shebang to developer-notes.md. (master...add_scripts_development_guidelines) https://github.com/bitcoin/bitcoin/pull/14846", "action": false, "timestamp": "2018-11-29T20:00:27+00:00" }, { "id": "625f2db102334cd9bc00ab02d2814e56", "sender": "wumpus", "payload": "ok, that was a quick topic, was intending to bring it up sooner but BlueMatt kind of ninja'd it :)", "action": false, "timestamp": "2018-11-29T20:00:41+00:00" }, { "id": "9b7d35bba38b45dd983472d19203fce3", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2018-11-29T20:00:46+00:00" } ], "events": [ { "event_type": "START_MEETING", "message": { "id": "70559c785b874bfa8173dfd607d75ef5", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2018-11-29T19:00:25+00:00" }, "operand": null, "id": "70559c785b874bfa8173dfd607d75ef5", "timestamp": "2018-11-29T19:00:25+00:00" }, { "event_type": "TOPIC", "message": { "id": "eaf293183a3f4233b9191f2e0e16a4e2", "sender": "wumpus", "payload": "#topic high priority for review", "action": false, "timestamp": "2018-11-29T19:01:52+00:00" }, "operand": "high priority for review", "id": "eaf293183a3f4233b9191f2e0e16a4e2", "timestamp": "2018-11-29T19:01:52+00:00" }, { "event_type": "LINK", "message": { "id": "92699bcf2af04a0d9a8647ea9d4725b2", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14670 | http: Fix HTTP server shutdown by promag \u00c3\u0082\u00c2\u00b7 Pull Request #14670 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:32+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14670", "id": "92699bcf2af04a0d9a8647ea9d4725b2", "timestamp": "2018-11-29T19:02:32+00:00" }, { "event_type": "LINK", "message": { "id": "0580adb0b65d442b89f3888b973d09b1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/13932 | Additional utility RPCs for PSBT by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #13932 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:36+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/13932", "id": "0580adb0b65d442b89f3888b973d09b1", "timestamp": "2018-11-29T19:02:36+00:00" }, { "event_type": "LINK", "message": { "id": "179cbe03b6e345c2a695e0ded6ae30a8", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14646 | Add expansion cache functions to descriptors (unused for now) by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14646 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:38+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14646", "id": "179cbe03b6e345c2a695e0ded6ae30a8", "timestamp": "2018-11-29T19:02:38+00:00" }, { "event_type": "LINK", "message": { "id": "ebfd3a9d649043bf8a3737d1ea120e2c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14336 | net: implement poll by pstratem \u00c3\u0082\u00c2\u00b7 Pull Request #14336 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:02:42+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14336", "id": "ebfd3a9d649043bf8a3737d1ea120e2c", "timestamp": "2018-11-29T19:02:42+00:00" }, { "event_type": "LINK", "message": { "id": "adbf148e05f1484a86986782481d3281", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14565 | Overhaul importmulti logic by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14565 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:04:29+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14565", "id": "adbf148e05f1484a86986782481d3281", "timestamp": "2018-11-29T19:04:29+00:00" }, { "event_type": "LINK", "message": { "id": "f7a3bdaefde84d22aa1e7b031257b066", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14380 | fix assert crash when specified change output spend size is unknown by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #14380 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:05:12+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14380", "id": "f7a3bdaefde84d22aa1e7b031257b066", "timestamp": "2018-11-29T19:05:12+00:00" }, { "event_type": "LINK", "message": { "id": "b936edc0cb2c430e9c90bb68a79e777f", "sender": "wumpus", "payload": "https://github.com/bitcoin/bitcoin/milestone/39", "action": false, "timestamp": "2018-11-29T19:05:30+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/milestone/39", "id": "b936edc0cb2c430e9c90bb68a79e777f", "timestamp": "2018-11-29T19:05:30+00:00" }, { "event_type": "LINK", "message": { "id": "979a2e342ddc49588844d1d6ce451f77", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14646 | Add expansion cache functions to descriptors (unused for now) by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14646 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:07:16+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14646", "id": "979a2e342ddc49588844d1d6ce451f77", "timestamp": "2018-11-29T19:07:16+00:00" }, { "event_type": "LINK", "message": { "id": "11176d9503654a9fb096f5abdeccc03c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14602 | Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14602 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:08:58+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14602", "id": "11176d9503654a9fb096f5abdeccc03c", "timestamp": "2018-11-29T19:08:58+00:00" }, { "event_type": "LINK", "message": { "id": "40557a7f5c324a63847cc6b474f92169", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14782 | [0.17] Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14782 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:09:19+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14782", "id": "40557a7f5c324a63847cc6b474f92169", "timestamp": "2018-11-29T19:09:19+00:00" }, { "event_type": "TOPIC", "message": { "id": "54fab47b971e46b0979537e20be4a104", "sender": "wumpus", "payload": "#topic getbalance", "action": false, "timestamp": "2018-11-29T19:14:13+00:00" }, "operand": "getbalance", "id": "54fab47b971e46b0979537e20be4a104", "timestamp": "2018-11-29T19:14:13+00:00" }, { "event_type": "LINK", "message": { "id": "3a53521646234d9ea5f2d08d02dd1f37", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14602 | Bugfix: Correctly calculate balances when min_conf is used, and for getbalance(\"*\") by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #14602 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:28:22+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14602", "id": "3a53521646234d9ea5f2d08d02dd1f37", "timestamp": "2018-11-29T19:28:22+00:00" }, { "event_type": "TOPIC", "message": { "id": "38439800ac604c3bbdd87e2c6cafe4ca", "sender": "wumpus", "payload": "#topic 0.17.1", "action": false, "timestamp": "2018-11-29T19:39:43+00:00" }, "operand": "0.17.1", "id": "38439800ac604c3bbdd87e2c6cafe4ca", "timestamp": "2018-11-29T19:39:43+00:00" }, { "event_type": "LINK", "message": { "id": "4a89eb1726024799b76626f5ff316a78", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14380 | fix assert crash when specified change output spend size is unknown by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #14380 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:42:08+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14380", "id": "4a89eb1726024799b76626f5ff316a78", "timestamp": "2018-11-29T19:42:08+00:00" }, { "event_type": "LINK", "message": { "id": "4d8aae3fdb3840fa9c4b16097470977d", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14780 | PSBT backports to 0.17 by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #14780 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2018-11-29T19:42:46+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14780", "id": "4d8aae3fdb3840fa9c4b16097470977d", "timestamp": "2018-11-29T19:42:46+00:00" }, { "event_type": "TOPIC", "message": { "id": "c457be56ce174da38e077207127c8135", "sender": "wumpus", "payload": "#topic proposed meeting topics ahead of time (moneyball)", "action": false, "timestamp": "2018-11-29T19:45:57+00:00" }, "operand": "proposed meeting topics ahead of time (moneyball)", "id": "c457be56ce174da38e077207127c8135", "timestamp": "2018-11-29T19:45:57+00:00" }, { "event_type": "TOPIC", "message": { "id": "4d5d7b9779554e5d912906d925af6008", "sender": "wumpus", "payload": "#topic pre-signing transactions (BlueMatt)", "action": false, "timestamp": "2018-11-29T19:48:09+00:00" }, "operand": "pre-signing transactions (BlueMatt)", "id": "4d5d7b9779554e5d912906d925af6008", "timestamp": "2018-11-29T19:48:09+00:00" }, { "event_type": "TOPIC", "message": { "id": "65f800a2cade4040bfc70d9901640787", "sender": "wumpus", "payload": "#topic wallet maintainer", "action": false, "timestamp": "2018-11-29T19:58:19+00:00" }, "operand": "wallet maintainer", "id": "65f800a2cade4040bfc70d9901640787", "timestamp": "2018-11-29T19:58:19+00:00" }, { "event_type": "END_MEETING", "message": { "id": "9b7d35bba38b45dd983472d19203fce3", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2018-11-29T20:00:46+00:00" }, "operand": null, "id": "9b7d35bba38b45dd983472d19203fce3", "timestamp": "2018-11-29T20:00:46+00:00" } ], "aliases": {}, "vote_in_progress": false, "motion_index": null }