12018-03-28T00:00:03  *** weez17 has quit IRC
  22018-03-28T00:00:48  *** weez17 has joined #bitcoin-core-dev
  32018-03-28T00:01:57  *** karimofthecrop has quit IRC
  42018-03-28T00:04:44  *** karimofthecrop has joined #bitcoin-core-dev
  52018-03-28T00:07:21  *** Chris_Stewart_5 has quit IRC
  62018-03-28T00:09:47  *** karimofthecrop has quit IRC
  72018-03-28T00:13:26  *** meshcollider has quit IRC
  82018-03-28T00:18:47  *** jojeyh has quit IRC
  92018-03-28T00:24:25  *** meshcollider has joined #bitcoin-core-dev
 102018-03-28T00:27:34  *** karimofthecrop has joined #bitcoin-core-dev
 112018-03-28T00:31:57  *** karimofthecrop has quit IRC
 122018-03-28T00:40:36  *** Murch has quit IRC
 132018-03-28T00:42:33  *** Strepsils has quit IRC
 142018-03-28T00:52:46  *** jamesob_ has joined #bitcoin-core-dev
 152018-03-28T00:56:20  *** promag has joined #bitcoin-core-dev
 162018-03-28T00:58:08  *** karimofthecrop has joined #bitcoin-core-dev
 172018-03-28T01:00:58  *** promag has quit IRC
 182018-03-28T01:06:16  *** AaronvanW has joined #bitcoin-core-dev
 192018-03-28T01:08:48  *** jamesob_ has quit IRC
 202018-03-28T01:09:10  *** jamesob_ has joined #bitcoin-core-dev
 212018-03-28T01:11:48  *** jamesob_ has quit IRC
 222018-03-28T01:12:14  *** jamesob_ has joined #bitcoin-core-dev
 232018-03-28T01:12:33  *** Randolf has joined #bitcoin-core-dev
 242018-03-28T01:14:13  *** jamesob_ has quit IRC
 252018-03-28T01:14:21  *** jamesob__ has joined #bitcoin-core-dev
 262018-03-28T01:22:53  *** jojeyh has joined #bitcoin-core-dev
 272018-03-28T01:29:01  *** d9b4bef9 has quit IRC
 282018-03-28T01:30:07  *** d9b4bef9 has joined #bitcoin-core-dev
 292018-03-28T01:30:43  *** jamesob___ has joined #bitcoin-core-dev
 302018-03-28T01:31:55  *** karimofthecrop has quit IRC
 312018-03-28T01:33:38  *** jamesob__ has quit IRC
 322018-03-28T01:34:19  *** karimofthecrop has joined #bitcoin-core-dev
 332018-03-28T01:43:27  *** dermoth has quit IRC
 342018-03-28T01:44:31  *** dermoth has joined #bitcoin-core-dev
 352018-03-28T01:49:14  *** harrymm has quit IRC
 362018-03-28T01:49:35  *** jojeyh has quit IRC
 372018-03-28T02:06:56  *** jojeyh has joined #bitcoin-core-dev
 382018-03-28T02:11:00  *** Samdney has quit IRC
 392018-03-28T02:21:07  *** contrapumpkin has joined #bitcoin-core-dev
 402018-03-28T02:21:22  *** jb55 has joined #bitcoin-core-dev
 412018-03-28T02:22:49  *** karimofthecrop has quit IRC
 422018-03-28T02:23:50  *** karimofthecrop has joined #bitcoin-core-dev
 432018-03-28T02:28:33  *** karimofthecrop has quit IRC
 442018-03-28T02:36:49  *** jamesob___ has quit IRC
 452018-03-28T02:43:04  *** belcher has quit IRC
 462018-03-28T02:47:13  *** promag has joined #bitcoin-core-dev
 472018-03-28T02:48:26  *** rex4539 has joined #bitcoin-core-dev
 482018-03-28T02:52:17  *** promag has quit IRC
 492018-03-28T02:57:21  *** Randolf has quit IRC
 502018-03-28T03:02:45  *** rex4539 has quit IRC
 512018-03-28T03:17:19  *** Krellan has quit IRC
 522018-03-28T03:35:09  *** jb55 has quit IRC
 532018-03-28T03:49:49  *** karimofthecrop has joined #bitcoin-core-dev
 542018-03-28T03:53:51  *** karimofthecrop has quit IRC
 552018-03-28T04:03:33  *** Randolf has joined #bitcoin-core-dev
 562018-03-28T04:17:56  *** promag has joined #bitcoin-core-dev
 572018-03-28T04:23:11  *** promag has quit IRC
 582018-03-28T04:24:36  *** karimofthecrop has joined #bitcoin-core-dev
 592018-03-28T04:30:11  *** aruns has joined #bitcoin-core-dev
 602018-03-28T04:31:25  *** karimofthecrop has quit IRC
 612018-03-28T04:38:56  *** r251d has quit IRC
 622018-03-28T04:39:21  *** r251d has joined #bitcoin-core-dev
 632018-03-28T04:46:54  *** lnostdal has quit IRC
 642018-03-28T04:53:46  *** Randolf has quit IRC
 652018-03-28T05:02:43  *** jtimon has quit IRC
 662018-03-28T05:03:01  *** lnostdal has joined #bitcoin-core-dev
 672018-03-28T05:06:52  *** promag has joined #bitcoin-core-dev
 682018-03-28T05:11:30  *** promag has quit IRC
 692018-03-28T05:15:52  *** karimofthecrop has joined #bitcoin-core-dev
 702018-03-28T05:20:09  *** karimofthecrop has quit IRC
 712018-03-28T05:33:16  *** Randolf has joined #bitcoin-core-dev
 722018-03-28T05:33:53  *** aruns has quit IRC
 732018-03-28T05:34:42  *** games_ is now known as bloatlet
 742018-03-28T05:35:16  *** grafcaps has quit IRC
 752018-03-28T05:37:06  *** grafcaps has joined #bitcoin-core-dev
 762018-03-28T06:01:41  *** Krellan has joined #bitcoin-core-dev
 772018-03-28T06:07:11  *** Krellan has quit IRC
 782018-03-28T06:16:27  *** promag has joined #bitcoin-core-dev
 792018-03-28T06:17:03  *** indistylo has joined #bitcoin-core-dev
 802018-03-28T06:17:30  *** flotwig_ has joined #bitcoin-core-dev
 812018-03-28T06:18:35  *** flotwig has quit IRC
 822018-03-28T06:20:24  *** flotwig_ is now known as flotwig
 832018-03-28T06:20:45  *** flotwig has joined #bitcoin-core-dev
 842018-03-28T06:21:24  *** promag has quit IRC
 852018-03-28T06:25:21  *** flotwig has quit IRC
 862018-03-28T06:28:07  *** qrestlove has quit IRC
 872018-03-28T06:29:16  *** commavir has quit IRC
 882018-03-28T06:30:29  *** flotwig has joined #bitcoin-core-dev
 892018-03-28T06:30:47  <bitcoin-git> [bitcoin] murrayn opened pull request #12809: Formatting changes to --help code for increased readability. (master...help_formatting) https://github.com/bitcoin/bitcoin/pull/12809
 902018-03-28T06:34:10  *** flotwig has quit IRC
 912018-03-28T06:34:32  *** flotwig has joined #bitcoin-core-dev
 922018-03-28T06:40:04  *** commavir has joined #bitcoin-core-dev
 932018-03-28T06:40:36  *** qrestlove has joined #bitcoin-core-dev
 942018-03-28T06:42:55  *** Cogito_Ergo_Sum has joined #bitcoin-core-dev
 952018-03-28T06:42:55  *** Cogito_Ergo_Sum has joined #bitcoin-core-dev
 962018-03-28T06:44:53  *** jdgs has joined #bitcoin-core-dev
 972018-03-28T06:51:29  *** promag has joined #bitcoin-core-dev
 982018-03-28T06:53:30  <bitcoin-git> [bitcoin] romanz opened pull request #12810: [Tests] Fix a typo at assert_start_raises_init_error() and update its invocation (master...fix-blocksdir-test) https://github.com/bitcoin/bitcoin/pull/12810
 992018-03-28T06:55:14  *** indistylo has quit IRC
1002018-03-28T06:55:51  *** promag has quit IRC
1012018-03-28T06:56:29  *** promag has joined #bitcoin-core-dev
1022018-03-28T06:58:15  *** promag has quit IRC
1032018-03-28T07:08:05  *** indistylo has joined #bitcoin-core-dev
1042018-03-28T07:10:38  *** aruns has joined #bitcoin-core-dev
1052018-03-28T07:13:36  *** indistylo has quit IRC
1062018-03-28T07:16:13  *** karimofthecrop has joined #bitcoin-core-dev
1072018-03-28T07:17:56  *** Krellan has joined #bitcoin-core-dev
1082018-03-28T07:21:23  *** karimofthecrop has quit IRC
1092018-03-28T07:24:32  *** Krellan has quit IRC
1102018-03-28T07:29:26  *** jtimon has joined #bitcoin-core-dev
1112018-03-28T07:29:37  *** zarez has joined #bitcoin-core-dev
1122018-03-28T07:32:04  *** bloatlet is now known as games_
1132018-03-28T07:32:54  *** Victorsueca has quit IRC
1142018-03-28T07:33:20  *** karimofthecrop has joined #bitcoin-core-dev
1152018-03-28T07:34:12  *** Victorsueca has joined #bitcoin-core-dev
1162018-03-28T07:34:22  <wumpus> I've added jnewbery to the issue management team: there's now fanquake, meshcollider, cfields and jnewbery that can help with issues and PRs on github, and open/close them
1172018-03-28T07:38:16  *** karimofthecrop has quit IRC
1182018-03-28T07:41:57  *** asoltys has quit IRC
1192018-03-28T07:43:10  *** asoltys has joined #bitcoin-core-dev
1202018-03-28T07:51:52  *** aruns has quit IRC
1212018-03-28T07:53:47  *** setpill has joined #bitcoin-core-dev
1222018-03-28T07:55:36  *** laurentmt has joined #bitcoin-core-dev
1232018-03-28T08:10:27  <aj> wumpus: if you haven't noticed already, #12806 is needed to fix travis failure on master
1242018-03-28T08:10:29  <gribble> https://github.com/bitcoin/bitcoin/issues/12806 | qa: Fix function names in feature_blocksdir by MarcoFalke · Pull Request #12806 · bitcoin/bitcoin · GitHub
1252018-03-28T08:16:51  *** timothy has joined #bitcoin-core-dev
1262018-03-28T08:21:20  *** drizztbsd has joined #bitcoin-core-dev
1272018-03-28T08:21:35  *** jtimon has quit IRC
1282018-03-28T08:22:11  *** timothy has quit IRC
1292018-03-28T08:24:20  *** setpill has quit IRC
1302018-03-28T08:24:42  *** setpill has joined #bitcoin-core-dev
1312018-03-28T08:51:40  *** indistylo has joined #bitcoin-core-dev
1322018-03-28T08:57:42  <wumpus> aj: thanks
1332018-03-28T08:57:46  *** BashCo_ has joined #bitcoin-core-dev
1342018-03-28T08:58:08  <wumpus> aj: it's strange, I don't have errors while running the test suite locally
1352018-03-28T08:58:09  *** BashCo has quit IRC
1362018-03-28T08:58:58  <aj> wumpus: really? feature_blocksdir should be failing pretty trivially
1372018-03-28T08:59:25  <wumpus> let me check again
1382018-03-28T08:59:51  <wumpus> first want to be sure because I might have a stale file around or such
1392018-03-28T09:01:08  *** jouke has quit IRC
1402018-03-28T09:01:11  *** JackH has quit IRC
1412018-03-28T09:01:22  *** jouke_ has joined #bitcoin-core-dev
1422018-03-28T09:02:31  <provoostenator> I'll run it on macOS. Though in my experience the functional test suite rarely passes in a single attempt on my machine.
1432018-03-28T09:04:16  <wumpus> ok yes it's failing
1442018-03-28T09:06:10  <wumpus> let's make the FAILED on the last line red when things fail
1452018-03-28T09:06:21  <aj> wumpus: there's #12810 as well, but i hadn't noticed it causing a bug
1462018-03-28T09:06:22  <gribble> https://github.com/bitcoin/bitcoin/issues/12810 | [Tests] Fix a typo at assert_start_raises_init_error() by romanz · Pull Request #12810 · bitcoin/bitcoin · GitHub
1472018-03-28T09:06:41  <wumpus> it must have just scrolled by yesterday as feature_blocksdir.py is one of the first in the long list, apparently didn't notice
1482018-03-28T09:11:08  *** jouke has joined #bitcoin-core-dev
1492018-03-28T09:11:33  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ac898b689c66...18606eb475fb
1502018-03-28T09:11:33  <bitcoin-git> bitcoin/master d71bedb MarcoFalke: qa: Fix function names in feature_blocksdir
1512018-03-28T09:11:34  <bitcoin-git> bitcoin/master 18606eb Wladimir J. van der Laan: Merge #12806: qa: Fix function names in feature_blocksdir...
1522018-03-28T09:12:27  <bitcoin-git> [bitcoin] laanwj closed pull request #12806: qa: Fix function names in feature_blocksdir (master...Mf1803-qaBlocksdirFixup) https://github.com/bitcoin/bitcoin/pull/12806
1532018-03-28T09:22:53  <bitcoin-git> [bitcoin] laanwj opened pull request #12811: test: Make summary row bold-red if any test failed (master...2018_03_tests_summaryrow) https://github.com/bitcoin/bitcoin/pull/12811
1542018-03-28T09:24:12  *** AaronvanW has joined #bitcoin-core-dev
1552018-03-28T09:25:39  *** Aaronvan_ has joined #bitcoin-core-dev
1562018-03-28T09:29:49  *** AaronvanW has quit IRC
1572018-03-28T09:30:57  *** laurentmt has quit IRC
1582018-03-28T09:34:04  *** karimofthecrop has joined #bitcoin-core-dev
1592018-03-28T09:38:09  *** karimofthecrop has quit IRC
1602018-03-28T09:38:16  *** aruns__ has joined #bitcoin-core-dev
1612018-03-28T09:40:49  *** indistylo has quit IRC
1622018-03-28T09:45:47  *** SopaXorzTaker has joined #bitcoin-core-dev
1632018-03-28T09:46:35  *** jdgs has quit IRC
1642018-03-28T09:59:56  <aj> wumpus: if you're making test_runner nicer, might work to just report all the failed tests at the end so they're easy to find. https://github.com/laanwj/bitcoin/pull/6 has sample code
1652018-03-28T10:00:22  <wumpus> yes, there's probably a ton of different things that could be done
1662018-03-28T10:00:38  <wumpus> I like making the row red though
1672018-03-28T10:01:04  <wumpus> though could do what you propose in addition to that, as it saves scrolling to find what failed
1682018-03-28T10:01:06  <aj> wumpus: i meant as well, not instead :)
1692018-03-28T10:01:09  <wumpus> right :)
1702018-03-28T10:03:03  <wumpus> I'll pull in your commit
1712018-03-28T10:03:35  <aj> wumpus: feel free to squash it, i didn't write a good commit message :)
1722018-03-28T10:04:09  *** indistylo has joined #bitcoin-core-dev
1732018-03-28T10:04:26  <wumpus> instead of adding a sort_key we change the natural sorting order of the object
1742018-03-28T10:04:53  *** suldev has joined #bitcoin-core-dev
1752018-03-28T10:05:06  <wumpus> probably want to sort by 1) status, then within that 2) name
1762018-03-28T10:05:17  <wumpus> oh it does
1772018-03-28T10:05:37  *** aruns__ has quit IRC
1782018-03-28T10:13:23  *** rafalcpp has quit IRC
1792018-03-28T10:14:10  *** rafalcpp has joined #bitcoin-core-dev
1802018-03-28T10:15:31  <bitcoin-git> [bitcoin] Empact opened pull request #12812: Test ReadConfigFile (master...test-read-config-file) https://github.com/bitcoin/bitcoin/pull/12812
1812018-03-28T10:17:30  *** arubi has quit IRC
1822018-03-28T10:17:59  *** arubi has joined #bitcoin-core-dev
1832018-03-28T10:19:05  *** CubicEarths has quit IRC
1842018-03-28T10:25:16  <bitcoin-git> [bitcoin] murrayn opened pull request #12813: Update man pages (master...man_pages) https://github.com/bitcoin/bitcoin/pull/12813
1852018-03-28T10:34:05  *** CubicEarths has joined #bitcoin-core-dev
1862018-03-28T10:38:48  *** fanquake has joined #bitcoin-core-dev
1872018-03-28T10:43:15  <aj> wumpus: right, could define __lt__(self, other) by the looks; but i think dealing with status would be more painful that way
1882018-03-28T10:43:40  <wumpus> yes, I looked at it and decided not to bother, it works fine like this, no use in overthinking it
1892018-03-28T10:44:36  <wumpus> in a way having the sort key explicit is kind of nice instead of wondering what kind of magic makes it work
1902018-03-28T10:48:08  *** fanquake has quit IRC
1912018-03-28T10:48:57  *** suldev has quit IRC
1922018-03-28T10:58:47  <bitcoin-git> [bitcoin] murrayn closed pull request #12813: Update man pages (master...man_pages) https://github.com/bitcoin/bitcoin/pull/12813
1932018-03-28T10:59:52  <jnewbery> Is there a way to block user https://github.com/btcerichcom from opening issues/PRs?
1942018-03-28T11:05:22  *** drizztbsd is now known as timothy
1952018-03-28T11:05:29  <wumpus> yes, I'll do it
1962018-03-28T11:05:56  <bitcoin-git> [bitcoin] jnewbery closed pull request #11614: ZMQ: Add decodedtx topic for JSON tx publishing (master...master) https://github.com/bitcoin/bitcoin/pull/11614
1972018-03-28T11:07:35  *** indistylo has quit IRC
1982018-03-28T11:09:33  *** indistylo has joined #bitcoin-core-dev
1992018-03-28T11:18:39  *** Krellan has joined #bitcoin-core-dev
2002018-03-28T11:23:09  *** Giszmo has quit IRC
2012018-03-28T11:23:44  *** Krellan has quit IRC
2022018-03-28T11:25:05  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/18606eb475fb...174d0160cb6b
2032018-03-28T11:25:05  <bitcoin-git> bitcoin/master adc2586 MarcoFalke: doc: Refer to witness reserved value as spec. in the BIP
2042018-03-28T11:25:06  <bitcoin-git> bitcoin/master 174d016 Wladimir J. van der Laan: Merge #12798: doc: Refer to witness reserved value as spec. in the BIP...
2052018-03-28T11:25:48  <bitcoin-git> [bitcoin] laanwj closed pull request #12798: doc: Refer to witness reserved value as spec. in the BIP (master...Mf1803-docWitnessReservedValue) https://github.com/bitcoin/bitcoin/pull/12798
2062018-03-28T11:27:46  *** Muis has quit IRC
2072018-03-28T11:34:23  *** karimofthecrop has joined #bitcoin-core-dev
2082018-03-28T11:36:43  *** Giszmo has joined #bitcoin-core-dev
2092018-03-28T11:38:49  *** karimofthecrop has quit IRC
2102018-03-28T11:44:59  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2112018-03-28T11:52:29  <provoostenator> What is bitdb in #11625 ryanofsky?
2122018-03-28T11:52:31  <gribble> https://github.com/bitcoin/bitcoin/issues/11625 | WIP: Add BitcoinApplication & RPCConsole tests by ryanofsky · Pull Request #11625 · bitcoin/bitcoin · GitHub
2132018-03-28T12:11:33  *** promag has joined #bitcoin-core-dev
2142018-03-28T12:22:06  *** zarez has quit IRC
2152018-03-28T12:23:32  *** promag has quit IRC
2162018-03-28T12:26:00  *** promag has joined #bitcoin-core-dev
2172018-03-28T12:29:46  *** zarez has joined #bitcoin-core-dev
2182018-03-28T12:30:26  *** Chris_Stewart_5 has quit IRC
2192018-03-28T12:31:03  *** esotericnonsense has quit IRC
2202018-03-28T12:45:17  *** jtimon has joined #bitcoin-core-dev
2212018-03-28T12:52:11  *** aruns has joined #bitcoin-core-dev
2222018-03-28T12:54:22  *** promag has quit IRC
2232018-03-28T12:55:09  *** indistylo has quit IRC
2242018-03-28T12:57:19  *** moneyball has joined #bitcoin-core-dev
2252018-03-28T13:00:41  *** aruns__ has joined #bitcoin-core-dev
2262018-03-28T13:03:05  *** aruns has quit IRC
2272018-03-28T13:21:02  *** d9b4bef9 has quit IRC
2282018-03-28T13:22:32  *** moneyball has quit IRC
2292018-03-28T13:24:16  *** d9b4bef9 has joined #bitcoin-core-dev
2302018-03-28T13:27:10  *** belcher has joined #bitcoin-core-dev
2312018-03-28T13:30:14  *** esotericnonsense has joined #bitcoin-core-dev
2322018-03-28T13:31:49  *** promag has joined #bitcoin-core-dev
2332018-03-28T13:31:53  *** promag has quit IRC
2342018-03-28T13:32:40  *** Guyver2 has joined #bitcoin-core-dev
2352018-03-28T13:34:06  *** marcoagner has joined #bitcoin-core-dev
2362018-03-28T13:42:00  *** setpill has quit IRC
2372018-03-28T13:45:32  *** AaronvanW has joined #bitcoin-core-dev
2382018-03-28T13:47:45  *** Aaronvan_ has quit IRC
2392018-03-28T13:50:36  *** Aaronvan_ has joined #bitcoin-core-dev
2402018-03-28T13:51:07  *** jamesob___ has joined #bitcoin-core-dev
2412018-03-28T13:53:35  *** AaronvanW has quit IRC
2422018-03-28T13:56:45  *** aruns__ has quit IRC
2432018-03-28T13:57:17  *** Victorsueca has quit IRC
2442018-03-28T13:58:27  *** Victorsueca has joined #bitcoin-core-dev
2452018-03-28T14:01:04  <bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/174d0160cb6b...0d8fc8de076d
2462018-03-28T14:01:05  <bitcoin-git> bitcoin/master f92541f Wladimir J. van der Laan: test: Make summary row bold-red if any test failed...
2472018-03-28T14:01:06  <bitcoin-git> bitcoin/master ffb033a Anthony Towns: test: List any failed tests at the end of test_runner output...
2482018-03-28T14:01:06  <bitcoin-git> bitcoin/master 0d8fc8d Wladimir J. van der Laan: Merge #12811: test: Make summary row bold-red if any test failed and show failed tests at end of table...
2492018-03-28T14:01:55  <bitcoin-git> [bitcoin] laanwj closed pull request #12811: test: Make summary row bold-red if any test failed and show failed tests at end of table (master...2018_03_tests_summaryrow) https://github.com/bitcoin/bitcoin/pull/12811
2502018-03-28T14:10:46  <bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/0d8fc8de076d...624bee96597c
2512018-03-28T14:10:47  <bitcoin-git> bitcoin/master bc6fdf2 John Newbery: Change all python files to use Python3
2522018-03-28T14:10:47  <bitcoin-git> bitcoin/master 1874058 Evan Klitzke: Make base58 python contrib code work with python3
2532018-03-28T14:10:48  <bitcoin-git> bitcoin/master 624bee9 Wladimir J. van der Laan: Merge #11881: Remove Python2 support...
2542018-03-28T14:11:07  <bitcoin-git> [bitcoin] laanwj closed pull request #11881: Remove Python2 support (master...remove_python2) https://github.com/bitcoin/bitcoin/pull/11881
2552018-03-28T14:16:47  *** karimofthecrop has joined #bitcoin-core-dev
2562018-03-28T14:21:20  *** karimofthecrop has quit IRC
2572018-03-28T14:33:16  *** Chris_Stewart_5 has joined #bitcoin-core-dev
2582018-03-28T14:33:26  *** meshcollider has quit IRC
2592018-03-28T14:39:40  *** moneyball has joined #bitcoin-core-dev
2602018-03-28T14:40:51  *** Murch has joined #bitcoin-core-dev
2612018-03-28T14:43:17  <bitcoin-git> [bitcoin] Sjors opened pull request #12818: [qt] TransactionView: highlight replacement tx after fee bump (master...2018/03/bump-fee-focus) https://github.com/bitcoin/bitcoin/pull/12818
2622018-03-28T14:52:16  *** jamesob___ is now known as jamesob_
2632018-03-28T14:59:58  *** zarez has quit IRC
2642018-03-28T15:09:56  *** commavir has quit IRC
2652018-03-28T15:10:41  *** flotwig has quit IRC
2662018-03-28T15:10:59  *** flotwig has joined #bitcoin-core-dev
2672018-03-28T15:11:10  *** commavir has joined #bitcoin-core-dev
2682018-03-28T15:14:08  *** Samdney has joined #bitcoin-core-dev
2692018-03-28T15:18:55  *** Krellan has joined #bitcoin-core-dev
2702018-03-28T15:19:30  *** marcoagner has quit IRC
2712018-03-28T15:23:51  *** Krellan has quit IRC
2722018-03-28T15:25:44  *** Dizzle has joined #bitcoin-core-dev
2732018-03-28T15:47:17  <BlueMatt> #10762 looks very merge-able
2742018-03-28T15:47:20  <BlueMatt> wumpus: ^
2752018-03-28T15:47:20  <gribble> https://github.com/bitcoin/bitcoin/issues/10762 | [wallet] Remove Wallet dependencies from init.cpp by jnewbery · Pull Request #10762 · bitcoin/bitcoin · GitHub
2762018-03-28T15:48:28  *** belcher has quit IRC
2772018-03-28T15:50:23  *** marcoagner has joined #bitcoin-core-dev
2782018-03-28T15:50:25  *** Randolf has quit IRC
2792018-03-28T15:58:21  *** marcoagner has quit IRC
2802018-03-28T16:02:59  *** flotwig_ has joined #bitcoin-core-dev
2812018-03-28T16:03:57  *** flotwig has quit IRC
2822018-03-28T16:03:58  *** flotwig_ is now known as flotwig
2832018-03-28T16:05:15  <jnewbery> release-notes.md conflicts are really irritating. Any thoughts on how to make them stop? Perhaps have a separate release-notes file for each PR that requires them, eg release-notes-pr1234.md, and then concatenate/edit them before the release? Release notes need a lot of editing anyway, so might not add too much overhead
2842018-03-28T16:06:48  *** jb55 has joined #bitcoin-core-dev
2852018-03-28T16:09:54  *** linasv has quit IRC
2862018-03-28T16:09:54  *** Alekszej has quit IRC
2872018-03-28T16:11:07  <sipa> jnewbery: that doesn't seem like more overhead than the to-wiki-and-back dance we do already at the end
2882018-03-28T16:12:21  *** jb55 has quit IRC
2892018-03-28T16:13:38  *** marcoagner has joined #bitcoin-core-dev
2902018-03-28T16:14:54  <BlueMatt> jimpo: I'd really rather the paralell-txindex stuff not have its own thread
2912018-03-28T16:15:27  <BlueMatt> we have too much thread proliferation already, can it not fit cleanly into just running in the scheduler/validationinterface thread(s) (and we can add another thread there when things get too full)
2922018-03-28T16:25:04  *** esotericnonsense has quit IRC
2932018-03-28T16:25:37  *** grafcaps has quit IRC
2942018-03-28T16:34:25  *** esotericnonsense has joined #bitcoin-core-dev
2952018-03-28T16:34:51  *** AaronvanW has joined #bitcoin-core-dev
2962018-03-28T16:36:04  <jimpo> BlueMatt the thread is just for when the index needs to catch up to the blockchain state from way behind (like behind BlockConnected callbacks).
2972018-03-28T16:36:26  <jimpo> Once it gets in sync once, the thread exits and the index is kept in sync by the validation interface.
2982018-03-28T16:36:42  <BlueMatt> ah, ok, right, will finish reviewing before I comment more :p
2992018-03-28T16:37:00  <jimpo> \o/ Code review is happening!
3002018-03-28T16:38:50  *** MarcoFalke_trave has joined #bitcoin-core-dev
3012018-03-28T16:38:50  *** Aaronvan_ has quit IRC
3022018-03-28T16:38:50  *** Victorsueca has quit IRC
3032018-03-28T16:39:01  <MarcoFalke_trave> wumpus, Are you around for merges? I might send you a list of pulls
3042018-03-28T16:40:03  *** Victorsueca has joined #bitcoin-core-dev
3052018-03-28T16:40:05  *** moneyball has quit IRC
3062018-03-28T16:43:39  *** grafcaps has joined #bitcoin-core-dev
3072018-03-28T16:51:00  *** tryphe_ has joined #bitcoin-core-dev
3082018-03-28T16:53:10  *** tryphe has quit IRC
3092018-03-28T16:55:12  *** Aaronvan_ has joined #bitcoin-core-dev
3102018-03-28T16:57:51  *** AaronvanW has quit IRC
3112018-03-28T17:04:56  *** MarcoFalke_trave has quit IRC
3122018-03-28T17:05:13  *** MarcoFalke_trave has joined #bitcoin-core-dev
3132018-03-28T17:12:54  *** moneyball has joined #bitcoin-core-dev
3142018-03-28T17:14:35  *** paulg222 has joined #bitcoin-core-dev
3152018-03-28T17:14:46  *** timothy has quit IRC
3162018-03-28T17:15:24  *** Strepsils has joined #bitcoin-core-dev
3172018-03-28T17:16:58  <paulg222> with bitcoin 0.16 in regtest mode I'm seeing that coinbase transactions are not going to a segwit p2sh wrapped address, but to a regular p2pkh address. Is that expected?
3182018-03-28T17:21:30  <arubi> paulg222, it's going to a p2pk, not p2pkh.  use generatetoaddress
3192018-03-28T17:21:48  <arubi> (if you want a different address that is)
3202018-03-28T17:22:46  *** aruns__ has joined #bitcoin-core-dev
3212018-03-28T17:23:02  <instagibbs> jnewbery, ive had to rebase a couple PRs a couple times each due to that :/
3222018-03-28T17:25:01  *** d9b4bef9 has quit IRC
3232018-03-28T17:26:09  *** d9b4bef9 has joined #bitcoin-core-dev
3242018-03-28T17:26:32  *** marcoagner has quit IRC
3252018-03-28T17:27:33  <sipa> instagibbs: easy rebases, thankfully :)
3262018-03-28T17:28:26  <instagibbs> muh acks!
3272018-03-28T17:48:19  *** paulg222 has quit IRC
3282018-03-28T17:55:41  *** Dizzle has quit IRC
3292018-03-28T18:02:43  *** timothy has joined #bitcoin-core-dev
3302018-03-28T18:03:02  *** goatpig has joined #bitcoin-core-dev
3312018-03-28T18:04:50  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #12820: contrib: Fix check-doc script regexes (master...Mf1803-qaCheckDocRegex) https://github.com/bitcoin/bitcoin/pull/12820
3322018-03-28T18:05:17  *** timothy has quit IRC
3332018-03-28T18:07:17  *** TD-Linux has quit IRC
3342018-03-28T18:07:24  *** TD--Linux has joined #bitcoin-core-dev
3352018-03-28T18:15:05  *** Ge0rges has quit IRC
3362018-03-28T18:20:16  *** Ge0rges has joined #bitcoin-core-dev
3372018-03-28T18:21:41  *** marcoagner has joined #bitcoin-core-dev
3382018-03-28T18:22:50  *** cdecker has joined #bitcoin-core-dev
3392018-03-28T18:26:12  *** JackH has joined #bitcoin-core-dev
3402018-03-28T18:37:07  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #12821: contrib: Remove unused import string (master...Mf1803-contribUnusedImportClangFormatDiff) https://github.com/bitcoin/bitcoin/pull/12821
3412018-03-28T18:44:36  <BlueMatt> https://github.com/bitcoin/bitcoin/commit/7deba93bdc76616011a9f493cbc203d60084416f <-- wtf? we dont bump trusted-git-root when a pgp key expires...this is what allow-revsit-commits is for. let alone doing so without removing the key from trusted-keys :(
3422018-03-28T18:46:01  *** aruns__ has quit IRC
3432018-03-28T18:52:15  *** marcoagner has quit IRC
3442018-03-28T18:56:42  *** Strepsils has quit IRC
3452018-03-28T18:58:04  <bitcoin-git> [bitcoin] TheBlueMatt opened pull request #12822: Revert 7deba93bdc76616011a9f493cbc203d60084416f and fix expired-key-sigs properly (master...master) https://github.com/bitcoin/bitcoin/pull/12822
3462018-03-28T18:58:23  *** Randolf has joined #bitcoin-core-dev
3472018-03-28T18:59:59  <MarcoFalke_trave> BlueMatt, they keys in trusted-keys are the root keys
3482018-03-28T19:00:07  <MarcoFalke_trave> Can't remove them
3492018-03-28T19:00:20  <MarcoFalke_trave> Imo we should change them to the specific subkeys instead
3502018-03-28T19:00:39  <BlueMatt> the key in question expired
3512018-03-28T19:00:44  <MarcoFalke_trave> sipa, around? Mind to merge #12821
3522018-03-28T19:00:45  <gribble> https://github.com/bitcoin/bitcoin/issues/12821 | contrib: Remove unused import string by MarcoFalke · Pull Request #12821 · bitcoin/bitcoin · GitHub
3532018-03-28T19:00:46  <BlueMatt> we tried that, it didnt work very well
3542018-03-28T19:00:54  <BlueMatt> we started with the subkeys listed
3552018-03-28T19:00:56  <BlueMatt> and things blew up regularly
3562018-03-28T19:01:30  <BlueMatt> oh, you're saying your root isnt expired, yea, ok
3572018-03-28T19:01:38  <BlueMatt> either way, shouldnt have updated trusted-git-root
3582018-03-28T19:01:44  <BlueMatt> let alone doing so without a fucking PR
3592018-03-28T19:02:18  <MarcoFalke_trave> Having the root key in there means I can't ever put a subkey on a different machine
3602018-03-28T19:02:49  <MarcoFalke_trave> :s/different/slightly compromised/
3612018-03-28T19:02:52  <BlueMatt> if thats your concern you should have a git-signing separate primary key
3622018-03-28T19:03:03  <BlueMatt> fully-separate them instead of partially combining them
3632018-03-28T19:03:24  <eklitzke> maco when you're traveling your nick should be MarcoPolo
3642018-03-28T19:05:42  *** tryphe has joined #bitcoin-core-dev
3652018-03-28T19:06:02  <instagibbs> sdaftuar the `IsDust` use in both relay and wallet policy seems like it makes reasoning about p2sh outputs weird to me. I guess you just pretend the output is p2sh-p2wpkh, and use that for relay dust calculation?
3662018-03-28T19:06:21  <instagibbs> (I'm leaving it for now, just thinking aloud)
3672018-03-28T19:06:28  <sdaftuar> instagibbs: yeah i was wondering what effect this has on relay policy too -- but figure we might as well just make it right for our wallet at least
3682018-03-28T19:07:17  *** MarcoFalke_trave is now known as MarcoPolo
3692018-03-28T19:07:21  <instagibbs> the functional test already is pretty coarse-grained
3702018-03-28T19:07:25  <sdaftuar> so we could just add a way to indicate we want the p2sh-p2wpkh assumption and use that only for our wallet, if we don't want to change relay policy
3712018-03-28T19:07:30  <instagibbs> checks that you dump 100 satoshis
3722018-03-28T19:07:36  *** MarcoPolo is now known as MarcoFalke_trave
3732018-03-28T19:07:43  <instagibbs> right
3742018-03-28T19:08:06  <instagibbs> or just split the concerns to allow us to move faster with wallet stuff
3752018-03-28T19:08:41  <sdaftuar> oh hmm.  i guess the reason we have a discard rate in the first place is to be sure that we have a buffer above the prevailing network dust-fee
3762018-03-28T19:08:47  <sdaftuar> so splitting might be counter to that a little
3772018-03-28T19:08:48  *** marcoagner has joined #bitcoin-core-dev
3782018-03-28T19:09:08  *** tryphe_ has quit IRC
3792018-03-28T19:09:10  <sdaftuar> in that we might decouple our calculation from the network-wide policy
3802018-03-28T19:10:56  <sdaftuar> oh.. so this is unfortunate i guess, in that current network nodes are applying a dust threshold that is "too high" for our p2sh-segwit wallet. good thing we have a discard rate in the first place i guess!  but given that it gets capped at the long term fee estimate, i'm not sure what value we're actually using now
3812018-03-28T19:11:46  *** nitramiz has joined #bitcoin-core-dev
3822018-03-28T19:14:52  <instagibbs> ill update the test at least to be a bit tighter
3832018-03-28T19:26:46  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #12823: doc: Add .gitattributes file for release-notes.md (master...Mf1803-docGitattributes) https://github.com/bitcoin/bitcoin/pull/12823
3842018-03-28T19:33:22  <BlueMatt> #12754 does not look like an "upstream" issue - the reporter indicating some things we should probably do that make upstream behave the proper way
3852018-03-28T19:33:23  <gribble> https://github.com/bitcoin/bitcoin/issues/12754 | [zmq] pub/sub is not reliable at all · Issue #12754 · bitcoin/bitcoin · GitHub
3862018-03-28T19:33:26  <BlueMatt> should lose the upstream tag
3872018-03-28T19:33:39  *** Aaronvan_ has quit IRC
3882018-03-28T19:33:59  <MarcoFalke_trave> Removed
3892018-03-28T19:34:22  *** nitramiz has quit IRC
3902018-03-28T19:34:31  *** nitramiz has joined #bitcoin-core-dev
3912018-03-28T19:44:42  *** SopaXorzTaker has quit IRC
3922018-03-28T19:49:14  *** Murch has quit IRC
3932018-03-28T19:50:07  *** nitramiz has quit IRC
3942018-03-28T19:50:14  *** Murch has joined #bitcoin-core-dev
3952018-03-28T19:53:46  *** Randolf has quit IRC
3962018-03-28T19:54:05  *** Tennis has joined #bitcoin-core-dev
3972018-03-28T19:57:41  *** belcher has joined #bitcoin-core-dev
3982018-03-28T20:01:36  *** marcoagner has quit IRC
3992018-03-28T20:04:20  <hkjn0> hey, maybe I'm just missing something basic here, but I had a tx which sendrawtransaction accepted and returned txid, but getrawtransaction (with txindex=1) still can't find.. how can this happen?
4002018-03-28T20:05:34  <hkjn0> I'm assuming that if the fee was lower than minrelaytxfee we'd return an error instead of a txid..
4012018-03-28T20:08:12  *** dafunkiz_ has joined #bitcoin-core-dev
4022018-03-28T20:10:57  <jnewbery> hkjn0: #bitcoin please. This channel is for bitcoin core development
4032018-03-28T20:16:04  *** nitramiz has joined #bitcoin-core-dev
4042018-03-28T20:18:22  *** dafunkiz_ has quit IRC
4052018-03-28T20:20:25  *** Cheeseo has joined #bitcoin-core-dev
4062018-03-28T20:24:07  *** Victorsueca has quit IRC
4072018-03-28T20:25:06  *** dafunkiz_ has joined #bitcoin-core-dev
4082018-03-28T20:25:33  *** Victorsueca has joined #bitcoin-core-dev
4092018-03-28T20:30:21  *** Chris_Stewart_5 has quit IRC
4102018-03-28T20:30:33  *** MarcoFalke_trave has quit IRC
4112018-03-28T20:32:49  *** drexl has joined #bitcoin-core-dev
4122018-03-28T20:34:44  *** meshcollider has joined #bitcoin-core-dev
4132018-03-28T20:34:45  *** Krellan has joined #bitcoin-core-dev
4142018-03-28T20:35:16  *** StopAndDecrypt_ has joined #bitcoin-core-dev
4152018-03-28T20:35:24  *** StopAndDecrypt has quit IRC
4162018-03-28T20:41:02  *** d9b4bef9 has quit IRC
4172018-03-28T20:41:22  <hkjn0> yup, sorry, realized after writing it was off-topic
4182018-03-28T20:42:08  *** d9b4bef9 has joined #bitcoin-core-dev
4192018-03-28T20:49:34  *** suldev has joined #bitcoin-core-dev
4202018-03-28T20:52:25  *** Strepsils has joined #bitcoin-core-dev
4212018-03-28T20:55:11  *** cheese_ has joined #bitcoin-core-dev
4222018-03-28T20:57:33  *** Cheeseo has quit IRC
4232018-03-28T21:03:43  *** dafunkiz_ has quit IRC
4242018-03-28T21:06:49  *** dafunkiz_ has joined #bitcoin-core-dev
4252018-03-28T21:10:22  *** Guyver2 has quit IRC
4262018-03-28T21:15:41  *** moneyball has quit IRC
4272018-03-28T21:16:31  *** Randolf has joined #bitcoin-core-dev
4282018-03-28T21:21:01  *** dafunkiz_ has quit IRC
4292018-03-28T21:22:43  *** Murch has quit IRC
4302018-03-28T21:23:11  *** Murch has joined #bitcoin-core-dev
4312018-03-28T21:27:49  *** dafunkiz_ has joined #bitcoin-core-dev
4322018-03-28T21:36:08  *** jb55 has joined #bitcoin-core-dev
4332018-03-28T21:38:47  *** suldev has quit IRC
4342018-03-28T21:44:20  *** Randolf has quit IRC
4352018-03-28T21:49:23  *** Strepsils has quit IRC
4362018-03-28T21:49:28  *** ProfMac is now known as ProfMac_
4372018-03-28T21:52:33  *** jb55 has quit IRC
4382018-03-28T21:55:05  *** Victorsueca has quit IRC
4392018-03-28T21:56:25  *** arbitrary_guy has joined #bitcoin-core-dev
4402018-03-28T21:56:32  *** Victorsueca has joined #bitcoin-core-dev
4412018-03-28T22:00:49  *** laurentmt has joined #bitcoin-core-dev
4422018-03-28T22:01:23  *** Cogito_Ergo_Sum has quit IRC
4432018-03-28T22:02:19  *** dafunkiz_ has quit IRC
4442018-03-28T22:06:00  *** dafunkiz_ has joined #bitcoin-core-dev
4452018-03-28T22:11:13  *** AaronvanW has joined #bitcoin-core-dev
4462018-03-28T22:20:44  *** jcorgan has quit IRC
4472018-03-28T22:22:32  *** TD--Linux is now known as TD-Linux
4482018-03-28T22:22:33  *** TD-Linux has joined #bitcoin-core-dev
4492018-03-28T22:26:51  *** dermoth has quit IRC
4502018-03-28T22:28:55  *** dermoth has joined #bitcoin-core-dev
4512018-03-28T22:31:27  *** laurentmt has quit IRC
4522018-03-28T22:38:27  *** marcinja has joined #bitcoin-core-dev
4532018-03-28T22:41:06  *** marcinja has quit IRC
4542018-03-28T22:44:16  *** Randolf has joined #bitcoin-core-dev
4552018-03-28T22:49:49  *** tyrick has quit IRC
4562018-03-28T23:09:07  *** ProfMac_ is now known as ProfMac
4572018-03-28T23:28:05  *** dafunkiz_ has quit IRC
4582018-03-28T23:28:29  *** lnostdal has quit IRC
4592018-03-28T23:30:03  *** drexl has quit IRC
4602018-03-28T23:30:10  *** dafunkiz_ has joined #bitcoin-core-dev
4612018-03-28T23:35:12  *** Tennis has quit IRC
4622018-03-28T23:39:07  *** dafunkiz_ has quit IRC