12016-04-03T00:20:04  *** justanotheruser has quit IRC
  22016-04-03T00:27:04  *** justanotheruser has joined #bitcoin-core-dev
  32016-04-03T00:28:11  *** Luke-Jr has quit IRC
  42016-04-03T00:28:46  *** Luke-Jr has joined #bitcoin-core-dev
  52016-04-03T00:35:42  *** Luke-Jr has quit IRC
  62016-04-03T00:35:57  *** Luke-Jr has joined #bitcoin-core-dev
  72016-04-03T01:15:52  *** cryptapus_afk is now known as cryptapus
  82016-04-03T01:30:46  *** Giszmo has quit IRC
  92016-04-03T01:32:06  *** Alopex has quit IRC
 102016-04-03T01:33:11  *** Alopex has joined #bitcoin-core-dev
 112016-04-03T01:37:04  *** da2ce7 has quit IRC
 122016-04-03T01:40:20  *** da2ce7 has joined #bitcoin-core-dev
 132016-04-03T01:51:03  *** Ylbam has quit IRC
 142016-04-03T02:16:47  *** zooko has joined #bitcoin-core-dev
 152016-04-03T02:27:06  *** Luke-Jr has quit IRC
 162016-04-03T02:27:42  *** Luke-Jr has joined #bitcoin-core-dev
 172016-04-03T02:28:17  *** Don_John_ has joined #bitcoin-core-dev
 182016-04-03T02:30:09  *** Don_John has quit IRC
 192016-04-03T02:35:01  *** Alopex has quit IRC
 202016-04-03T02:36:06  *** Alopex has joined #bitcoin-core-dev
 212016-04-03T02:55:14  *** justanotheruser has quit IRC
 222016-04-03T02:56:47  *** btcdrak has joined #bitcoin-core-dev
 232016-04-03T03:02:29  *** Luke-Jr has quit IRC
 242016-04-03T03:03:36  *** justanotheruser has joined #bitcoin-core-dev
 252016-04-03T03:03:42  *** justanotheruser has joined #bitcoin-core-dev
 262016-04-03T03:08:43  *** Luke-Jr has joined #bitcoin-core-dev
 272016-04-03T03:12:43  *** achow101 has quit IRC
 282016-04-03T03:28:01  *** Alopex has quit IRC
 292016-04-03T03:29:06  *** Alopex has joined #bitcoin-core-dev
 302016-04-03T03:47:08  *** Thireus has quit IRC
 312016-04-03T03:51:02  *** Alopex has quit IRC
 322016-04-03T03:52:07  *** Alopex has joined #bitcoin-core-dev
 332016-04-03T04:12:55  *** Luke-Jr has quit IRC
 342016-04-03T04:20:01  *** Alopex has quit IRC
 352016-04-03T04:21:07  *** Alopex has joined #bitcoin-core-dev
 362016-04-03T04:26:11  *** jtimon has quit IRC
 372016-04-03T04:36:02  *** Alopex has quit IRC
 382016-04-03T04:37:04  *** Luke-Jr has joined #bitcoin-core-dev
 392016-04-03T04:37:07  *** Alopex has joined #bitcoin-core-dev
 402016-04-03T04:40:27  *** dansmith_btc has quit IRC
 412016-04-03T04:47:08  *** cryptapus is now known as cryptapus_afk
 422016-04-03T04:54:21  *** zooko has quit IRC
 432016-04-03T05:07:52  *** phantomcircuit has quit IRC
 442016-04-03T05:38:00  *** molz has joined #bitcoin-core-dev
 452016-04-03T05:39:20  *** wallet42 has quit IRC
 462016-04-03T05:40:56  *** molly has quit IRC
 472016-04-03T05:54:45  *** d_t has quit IRC
 482016-04-03T06:22:01  *** Alopex has quit IRC
 492016-04-03T06:23:06  *** Alopex has joined #bitcoin-core-dev
 502016-04-03T06:36:54  *** cguida has quit IRC
 512016-04-03T06:37:03  *** cguida has joined #bitcoin-core-dev
 522016-04-03T07:00:01  *** Alopex has quit IRC
 532016-04-03T07:01:06  *** Alopex has joined #bitcoin-core-dev
 542016-04-03T07:16:02  *** Alopex has quit IRC
 552016-04-03T07:17:06  *** phantomcircuit has joined #bitcoin-core-dev
 562016-04-03T07:17:07  *** Alopex has joined #bitcoin-core-dev
 572016-04-03T08:08:31  *** Thireus has joined #bitcoin-core-dev
 582016-04-03T08:25:30  *** Guyver2 has joined #bitcoin-core-dev
 592016-04-03T08:29:52  *** wallet42 has joined #bitcoin-core-dev
 602016-04-03T09:04:27  *** frankenmint has quit IRC
 612016-04-03T09:05:31  *** frankenmint has joined #bitcoin-core-dev
 622016-04-03T09:10:24  *** frankenmint has quit IRC
 632016-04-03T09:23:40  *** Guyver2 has quit IRC
 642016-04-03T09:54:37  <GitHub45> [bitcoin] laanwj opened pull request #7792: depends: mac deploy Py3 compatibility (master...2016_04_fix_macosx_gitian) https://github.com/bitcoin/bitcoin/pull/7792
 652016-04-03T10:02:13  <GitHub117> [bitcoin] MarcoFalke opened pull request #7793: [doxygen] Fix member comments (master...Mf1604-doxygenMembers) https://github.com/bitcoin/bitcoin/pull/7793
 662016-04-03T10:06:01  *** frankenmint has joined #bitcoin-core-dev
 672016-04-03T10:10:52  *** Ylbam has joined #bitcoin-core-dev
 682016-04-03T10:11:30  *** frankenmint has quit IRC
 692016-04-03T11:06:49  *** frankenmint has joined #bitcoin-core-dev
 702016-04-03T11:11:52  *** frankenmint has quit IRC
 712016-04-03T11:16:31  <GitHub92> [bitcoin] laanwj opened pull request #7795: UpdateTip: log only one line at most per block (master...2016_04_block_warning_logging_inline) https://github.com/bitcoin/bitcoin/pull/7795
 722016-04-03T11:22:32  <GitHub86> [bitcoin] MarcoFalke opened pull request #7796: [amount] Add support for negative fee rates (master...Mf1604-amountNeg64) https://github.com/bitcoin/bitcoin/pull/7796
 732016-04-03T11:39:36  <gmaxwell> Is there some computationally efficient way that I can take a mempool txn and determine its chain depth in the mempool?
 742016-04-03T11:40:09  <sipa> i'd say check the code that limits it?
 752016-04-03T11:47:10  <gmaxwell> Fair recommendation.. looks like it walks, I guess thats not that awful, was thinking of sorting invs but it needs to be a topological sort.
 762016-04-03T11:48:09  <sipa> given that it is limited in depth, walking is worst-case constant time :)
 772016-04-03T11:48:33  <gmaxwell> it's a bit annoying because a txn can have multiple parents, and you need to walk all to discover the worst case depth.
 782016-04-03T11:48:52  <gmaxwell> can't easily be cached since new blocks will invalidate.
 792016-04-03T12:06:59  *** jtimon has joined #bitcoin-core-dev
 802016-04-03T12:07:29  *** frankenmint has joined #bitcoin-core-dev
 812016-04-03T12:12:32  *** frankenmint has quit IRC
 822016-04-03T12:14:27  <GitHub173> [bitcoin] mruddy opened pull request #7797: RPC: fix generatetoaddress failing to parse address (master...generatetoaddress-fix) https://github.com/bitcoin/bitcoin/pull/7797
 832016-04-03T12:16:03  *** laurentmt has joined #bitcoin-core-dev
 842016-04-03T12:16:13  *** laurentmt has quit IRC
 852016-04-03T12:32:23  <GitHub108> [bitcoin] MarcoFalke opened pull request #7798: [travis] Print the commit which was evaluated (master...Mf1604-travisCommitLog) https://github.com/bitcoin/bitcoin/pull/7798
 862016-04-03T12:39:20  *** wallet42 has quit IRC
 872016-04-03T13:07:16  <GitHub142> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/30c2dd8d0598...e0b3e19816b6
 882016-04-03T13:07:16  <GitHub142> bitcoin/master 4b9dfd6 Pavel Janík: Add note about using the Qt official binary installer.
 892016-04-03T13:07:17  <GitHub142> bitcoin/master e0b3e19 Wladimir J. van der Laan: Merge #7789: [Doc] Add note about using the Qt official binary installer....
 902016-04-03T13:07:25  <GitHub31> [bitcoin] laanwj closed pull request #7789: [Doc] Add note about using the Qt official binary installer. (master...20160402_Qt56_official_installer_note) https://github.com/bitcoin/bitcoin/pull/7789
 912016-04-03T13:13:26  *** frankenmint has joined #bitcoin-core-dev
 922016-04-03T13:19:04  *** frankenmint has quit IRC
 932016-04-03T13:35:36  *** syhex has joined #bitcoin-core-dev
 942016-04-03T13:37:45  *** syhex has left #bitcoin-core-dev
 952016-04-03T13:54:52  <GitHub60> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/e0b3e19816b6...ff5874bcf7e3
 962016-04-03T13:54:53  <GitHub60> bitcoin/master fa524d9 MarcoFalke: [qa] Use python2/3 syntax
 972016-04-03T13:54:54  <GitHub60> bitcoin/master fa2cea1 MarcoFalke: [qa] rpc-tests: Properly use integers, floats
 982016-04-03T13:54:54  <GitHub60> bitcoin/master faaa3c9 MarcoFalke: [qa] mininode: Catch exceptions in got_data
 992016-04-03T13:55:02  <GitHub197> [bitcoin] laanwj closed pull request #7778: [qa] Bug fixes and refactor (master...Mf1604-qaFixesRefactor) https://github.com/bitcoin/bitcoin/pull/7778
1002016-04-03T13:55:44  <MarcoFalke> Nice, looks like I can continue with the py3 switch.
1012016-04-03T14:06:32  *** achow101 has joined #bitcoin-core-dev
1022016-04-03T14:15:28  *** frankenmint has joined #bitcoin-core-dev
1032016-04-03T14:20:27  *** frankenmint has quit IRC
1042016-04-03T14:31:36  *** murch has joined #bitcoin-core-dev
1052016-04-03T14:43:23  *** paveljanik has joined #bitcoin-core-dev
1062016-04-03T14:43:40  *** zooko has joined #bitcoin-core-dev
1072016-04-03T15:10:49  *** zooko has quit IRC
1082016-04-03T15:34:24  <GitHub64> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ff5874bcf7e3...e662a7628801
1092016-04-03T15:34:24  <GitHub64> bitcoin/master 60361ca mruddy: RPC: fix generatetoaddress failing to parse address and add unit test
1102016-04-03T15:34:25  <GitHub64> bitcoin/master e662a76 Wladimir J. van der Laan: Merge #7797: RPC: fix generatetoaddress failing to parse address...
1112016-04-03T15:34:32  <GitHub143> [bitcoin] laanwj closed pull request #7797: RPC: fix generatetoaddress failing to parse address (master...generatetoaddress-fix) https://github.com/bitcoin/bitcoin/pull/7797
1122016-04-03T16:04:00  <btcdrak> MarcoFalke: after 0.12.1 is tagged, it might be worth backporting the stuff around python required for Ubuntu 16.04
1132016-04-03T16:04:20  *** molly has joined #bitcoin-core-dev
1142016-04-03T16:07:11  *** molz has quit IRC
1152016-04-03T16:08:53  <wumpus> yes, probably
1162016-04-03T16:11:45  <sdaftuar> gmaxwell: saw your comment above about topological sort
1172016-04-03T16:12:30  <sdaftuar> gmaxwell: if you don't actually care about "depth", and just want a sort that puts dependent transactions after their ancestors, then you can sort by nCountWithAncestors
1182016-04-03T16:12:52  <sdaftuar> which gets updated as blocks are found
1192016-04-03T16:13:03  <sipa> sdaftuar: i assume he also wants it sorted by decreasing feerate
1202016-04-03T16:13:16  <sdaftuar> ah
1212016-04-03T16:13:28  <sdaftuar> that sounds like what the new CNB does
1222016-04-03T16:16:14  *** frankenmint has joined #bitcoin-core-dev
1232016-04-03T16:20:34  *** frankenmint has quit IRC
1242016-04-03T16:22:43  *** molz has joined #bitcoin-core-dev
1252016-04-03T16:26:04  *** molly has quit IRC
1262016-04-03T16:33:10  *** molly has joined #bitcoin-core-dev
1272016-04-03T16:36:04  *** molz has quit IRC
1282016-04-03T16:54:21  <GitHub126> [bitcoin] btcdrak opened pull request #7800: [0.12] Update release notes (0.12...docs) https://github.com/bitcoin/bitcoin/pull/7800
1292016-04-03T17:04:40  *** Giszmo has joined #bitcoin-core-dev
1302016-04-03T17:16:58  *** frankenmint has joined #bitcoin-core-dev
1312016-04-03T17:19:07  *** frankenmint has quit IRC
1322016-04-03T17:19:15  *** frankenmint has joined #bitcoin-core-dev
1332016-04-03T17:30:48  *** wallet42 has joined #bitcoin-core-dev
1342016-04-03T17:39:48  *** molz has joined #bitcoin-core-dev
1352016-04-03T17:43:29  *** molly has quit IRC
1362016-04-03T17:55:13  *** d_t has joined #bitcoin-core-dev
1372016-04-03T18:00:24  *** challisto has quit IRC
1382016-04-03T18:21:11  *** d_t has quit IRC
1392016-04-03T18:25:06  *** PRab_ has joined #bitcoin-core-dev
1402016-04-03T18:26:52  *** PRab has quit IRC
1412016-04-03T18:26:54  *** PRab_ is now known as PRab
1422016-04-03T18:31:14  <gmaxwell> sdaftuar: no thats exactly what I want. Glad I asked.
1432016-04-03T18:35:10  *** Thireus has quit IRC
1442016-04-03T18:35:31  *** Thireus has joined #bitcoin-core-dev
1452016-04-03T18:51:52  *** moli has joined #bitcoin-core-dev
1462016-04-03T18:51:55  <gmaxwell> sdaftuar: ah, I didn't see that because I was on 0.12. :)
1472016-04-03T18:54:01  *** molz has quit IRC
1482016-04-03T19:12:51  <btcdrak> gmaxwell: I took a stab at 0.12 release notes for the softfork, if you have time, could you check for accuracy/completeness? https://github.com/bitcoin/bitcoin/pull/7800
1492016-04-03T19:12:51  *** Cheeseo has joined #bitcoin-core-dev
1502016-04-03T19:15:57  <sdaftuar> gmaxwell: if you're working on something that you intend for 0.12 (as opposed to master), i have some suggestions
1512016-04-03T19:22:38  <sdaftuar> gmaxwell: i wrote up code at one point (before i realized the nCountWithAncestors thing) that would walk the transaction graph, along the lines of Kahn's algorithm iirc
1522016-04-03T19:23:08  <sdaftuar> i could dig that up, but morcos had the idea of just first sorting by entry time to mempool, which should be close to right
1532016-04-03T19:24:23  <sdaftuar> and then looping over the resulting sorted list, and checking if the tx is clear of dependencies
1542016-04-03T19:33:09  <gmaxwell> I'm contemplating just using the same score to order them that createnewblock does now.
1552016-04-03T19:38:07  <GitHub11> [bitcoin] MarcoFalke opened pull request #7801: [qa] Remove misleading "errorString syntax" (master...Mf1604-qaTestErrorString) https://github.com/bitcoin/bitcoin/pull/7801
1562016-04-03T19:38:11  <GitHub101> [bitcoin] MarcoFalke opened pull request #7803: [qa] maxblocksinflight: Actually enable test (master...Mf1604-qaTestMaxBlocks) https://github.com/bitcoin/bitcoin/pull/7803
1572016-04-03T19:38:12  <GitHub61> [bitcoin] MarcoFalke opened pull request #7802: [qa] httpbasics: Actually test second connection (master...Mf1604-qaTestHttp) https://github.com/bitcoin/bitcoin/pull/7802
1582016-04-03T20:00:55  *** d_t has joined #bitcoin-core-dev
1592016-04-03T20:24:14  <GitHub135> [bitcoin] sipa opened pull request #7804: Track block download times per individual block (master...betterkicktimeout) https://github.com/bitcoin/bitcoin/pull/7804
1602016-04-03T20:25:29  <sipa> wumpus: i wonder if we could/should fetch the entire description (or even all discussion) of a PR, and include it in the merge commit...
1612016-04-03T20:27:31  <gmaxwell> I would like that.
1622016-04-03T20:27:56  <gmaxwell> it irritates the hell out of me that the git history often doesn't even include an adequate description of the change, because it's all in the PR.
1632016-04-03T20:28:40  <sipa> on the other hand, PR descriptions are sometimes not updated when the scope/implementation changes significantly
1642016-04-03T20:28:57  *** justanotheruser has quit IRC
1652016-04-03T20:29:21  <sipa> you can consider that a bad thing, but if you see the description just as the first message of the discussion, maybe the correct solution is just including the entire discussion
1662016-04-03T20:29:26  <sipa> text is cheap
1672016-04-03T20:29:37  *** justanotheruser has joined #bitcoin-core-dev
1682016-04-03T20:30:39  <sipa> that would make the commits also more independent from github (as currently the merge commit does list the github PR number, but to grasp all the details you often need to actually go to github)
1692016-04-03T20:32:12  *** Thireus has quit IRC
1702016-04-03T20:32:26  *** Guyver2 has joined #bitcoin-core-dev
1712016-04-03T20:33:40  *** Thireus has joined #bitcoin-core-dev
1722016-04-03T20:41:39  <btcdrak> sipa: yeah I tried that once but really the merge script would have to be redone in something like python so you can parse the github API data.
1732016-04-03T20:42:02  <gmaxwell> whole thing need not be redone, it could call a helper.
1742016-04-03T20:42:18  *** laurentmt has joined #bitcoin-core-dev
1752016-04-03T20:42:18  <gmaxwell> could also add the phase of the moon and current best block, if we wanted. :)
1762016-04-03T20:42:26  <btcdrak> sipa: check #5623
1772016-04-03T20:45:46  *** laurentmt has quit IRC
1782016-04-03T20:47:54  <MarcoFalke> btcdrak, the script is already python /contrib/devtools/github-merge.py
1792016-04-03T20:49:25  *** molz has joined #bitcoin-core-dev
1802016-04-03T20:49:37  <btcdrak> oh the shell script went away
1812016-04-03T20:51:09  <btcdrak> well then that's easy, just pull the title from a fetch of https://api.github.com/repos/$REPO/pulls/$PULL
1822016-04-03T20:51:20  <MarcoFalke> Should be trivial to include the text from GitHub, but I am worried that typing `git log` will make you scroll the whole day
1832016-04-03T20:51:22  <btcdrak> you can make a lot more interesting merge commit messages too
1842016-04-03T20:52:01  <MarcoFalke> we already fetch the title ;)
1852016-04-03T20:52:54  *** moli has quit IRC
1862016-04-03T20:53:02  <btcdrak> omg. have been under a rock.
1872016-04-03T20:54:11  <sipa> btcdrak: yes, that's why i brought it up, because it should be easy to add more now
1882016-04-03T20:55:00  <btcdrak> sipa: +1 on adding the entire discussion history to the merge commit
1892016-04-03T20:55:19  <sipa> MarcoFalke: that's a decent concern... we could fetch the github discussion, make a commit that puts it in $REPO/share/github-discussion/$PR.md or something, and include that commit in the merge...
1902016-04-03T20:56:07  <MarcoFalke> sipa, exactly my thought. Just make a blob and do an octomerge
1912016-04-03T20:57:30  <sipa> it's something i dislike right now: commits that refer to pull requests; the git source tree is something that should make sense without having access to github
1922016-04-03T21:06:35  <btcdrak> I think the commit message should contain the PR body text.
1932016-04-03T21:06:48  <btcdrak> merge* commit message
1942016-04-03T21:37:18  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1952016-04-03T21:44:21  <gmaxwell> Well this isn't a replacement for having a useful commit message.
1962016-04-03T22:13:55  *** murch has quit IRC
1972016-04-03T22:46:16  *** gevs has quit IRC
1982016-04-03T22:58:55  *** gevs has joined #bitcoin-core-dev
1992016-04-03T23:02:17  *** d_t has quit IRC
2002016-04-03T23:08:42  *** Guyver2 has quit IRC
2012016-04-03T23:14:15  *** gribble has quit IRC
2022016-04-03T23:16:49  *** gribble has joined #bitcoin-core-dev
2032016-04-03T23:26:16  *** Chris_Stewart_5 has quit IRC
2042016-04-03T23:48:38  *** StringerBell has joined #bitcoin-core-dev
2052016-04-03T23:49:20  *** wallet42 has quit IRC