12015-12-11T00:02:01  *** PRab has quit IRC
  22015-12-11T00:18:06  *** JackH has quit IRC
  32015-12-11T00:20:48  *** evoskuil has quit IRC
  42015-12-11T00:23:42  *** JackH has joined #bitcoin-core-dev
  52015-12-11T00:25:12  *** evoskuil has joined #bitcoin-core-dev
  62015-12-11T00:30:30  *** Tera2342 has joined #bitcoin-core-dev
  72015-12-11T00:50:12  *** zookolaptop has joined #bitcoin-core-dev
  82015-12-11T00:54:45  *** [1]evoskuil has joined #bitcoin-core-dev
  92015-12-11T00:56:26  *** evoskuil has quit IRC
 102015-12-11T00:56:26  *** [1]evoskuil is now known as evoskuil
 112015-12-11T00:56:30  *** raedah has quit IRC
 122015-12-11T01:02:44  *** bawong has quit IRC
 132015-12-11T01:03:19  *** Tera2342 has quit IRC
 142015-12-11T01:23:30  *** Taek42 is now known as Taek
 152015-12-11T01:55:01  *** jcorgan|away is now known as jcorgan
 162015-12-11T02:01:27  *** Ylbam has quit IRC
 172015-12-11T02:11:48  *** randy-waterhouse has quit IRC
 182015-12-11T02:15:06  *** Thireus has quit IRC
 192015-12-11T02:38:57  *** jcorgan is now known as jcorgan|away
 202015-12-11T02:46:29  *** Tera2342 has joined #bitcoin-core-dev
 212015-12-11T02:52:53  *** Tera2342 has quit IRC
 222015-12-11T02:54:26  *** jamesob has quit IRC
 232015-12-11T03:00:22  *** xiangfu has quit IRC
 242015-12-11T03:00:39  *** xiangfu has joined #bitcoin-core-dev
 252015-12-11T03:01:33  *** ParadoxSpiral_ has joined #bitcoin-core-dev
 262015-12-11T03:03:23  *** belcher has quit IRC
 272015-12-11T03:04:34  *** ParadoxSpiral has quit IRC
 282015-12-11T03:07:58  *** tripleslash has quit IRC
 292015-12-11T03:14:59  *** PRab has joined #bitcoin-core-dev
 302015-12-11T03:41:32  *** xiangfu has quit IRC
 312015-12-11T03:41:49  *** xiangfu has joined #bitcoin-core-dev
 322015-12-11T03:53:18  *** oldbrew has joined #bitcoin-core-dev
 332015-12-11T05:00:21  *** tripleslash has joined #bitcoin-core-dev
 342015-12-11T05:02:40  *** Tera2342 has joined #bitcoin-core-dev
 352015-12-11T05:35:53  *** jcorgan|away is now known as jcorgan
 362015-12-11T05:36:54  *** Arnavion has quit IRC
 372015-12-11T05:39:01  *** Arnavion has joined #bitcoin-core-dev
 382015-12-11T05:44:47  *** BashCo has joined #bitcoin-core-dev
 392015-12-11T06:39:31  *** Thireus has joined #bitcoin-core-dev
 402015-12-11T06:48:06  *** sdaftuar has quit IRC
 412015-12-11T06:48:20  *** morcos has quit IRC
 422015-12-11T06:48:22  *** zxzzt has quit IRC
 432015-12-11T06:50:04  *** sdaftuar has joined #bitcoin-core-dev
 442015-12-11T06:50:10  *** zxzzt has joined #bitcoin-core-dev
 452015-12-11T06:50:15  *** morcos has joined #bitcoin-core-dev
 462015-12-11T07:06:50  *** Ylbam has joined #bitcoin-core-dev
 472015-12-11T07:40:13  *** Thireus has quit IRC
 482015-12-11T07:52:08  *** MarcoFalke has joined #bitcoin-core-dev
 492015-12-11T07:52:27  *** oldbrew has quit IRC
 502015-12-11T07:59:02  <jonasschnelli> Luke-Jr: IIRC right, modifying .DS_Stores is non trivial. I guess you can't just change it with hexeditor like things. But i'm happy to try if you pass me over a .DS_Store.
 512015-12-11T07:59:31  <jonasschnelli> I guess there are tools to modify a DS_Store (which is a non-open apple binary format)
 522015-12-11T08:00:28  <Luke-Jr> http://luke.dashjr.org/tmp/code/DS_Store
 532015-12-11T08:12:16  <jonasschnelli> Luke-Jr: it looks like its easier to test if you add it to your PR (i can do a gitian build). Okay?
 542015-12-11T08:12:55  <Luke-Jr> ok, pushed
 552015-12-11T08:14:19  <jonasschnelli> Okay. Building caa7c8fabef027b9640561dff8472766f5e3296d (takes 30-40mins)
 562015-12-11T08:14:39  <Luke-Jr> jonasschnelli: that's not the top commit
 572015-12-11T08:14:58  <jonasschnelli> Sorry. Meant: c903ae4d6794bb448d72100744036b7835481450
 582015-12-11T08:15:41  <jonasschnelli> I skip win and linux builds... so 12mins will it be~.
 592015-12-11T08:17:15  *** Thireus has joined #bitcoin-core-dev
 602015-12-11T08:18:16  *** MarcoFalke has quit IRC
 612015-12-11T08:19:04  <wumpus> gmaxwell: so should we revert #4906?
 622015-12-11T08:23:51  <GitHub187> [bitcoin] laanwj closed pull request #7196: Doxygen (master...doxygen-a) https://github.com/bitcoin/bitcoin/pull/7196
 632015-12-11T08:25:21  <GitHub39> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5dc63ed1ca8a...f7f44b1bdd15
 642015-12-11T08:25:21  <GitHub39> bitcoin/master 00423e1 Suriyaa Kudo: Set link from http:// to https://...
 652015-12-11T08:25:22  <GitHub39> bitcoin/master f7f44b1 Wladimir J. van der Laan: Merge pull request #7197...
 662015-12-11T08:25:31  <GitHub16> [bitcoin] laanwj closed pull request #7197: Set link from http:// to https:// (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7197
 672015-12-11T08:30:54  <jonasschnelli> Luke-Jr: seems to have build issues on OSX: https://bitcoin.jonasschnelli.ch/pulls/7192/build-osx.log
 682015-12-11T08:31:08  <Luke-Jr> your SSL is still broken :/
 692015-12-11T08:31:58  <jonasschnelli> Yes. I know... need to fix it. Will do *soon*.
 702015-12-11T08:32:29  <Luke-Jr> I guess I can hopefully reproduce it in gitian myself :x
 712015-12-11T08:33:01  * Luke-Jr wonders why gitian has started demanding he enter a password
 722015-12-11T08:35:21  <Luke-Jr> debug1: Skipping ssh-dss key ./var/id_dsa for not in PubkeyAcceptedKeyTypes
 732015-12-11T08:42:44  <wumpus> bleh, can you bisect it to a change in gitian?
 742015-12-11T08:42:58  <Luke-Jr> looks like it's OpenSSH 7.0
 752015-12-11T08:43:14  <Luke-Jr> no longer accepts DSS keys by default
 762015-12-11T08:43:39  <wumpus> oh that's good to know
 772015-12-11T08:44:05  <Luke-Jr>  * Support for ssh-dss, ssh-dss-cert-* host and user keys is disabled by default at run-time. These may be re-enabled using the instructions at http://www.openssh.com/legacy.html
 782015-12-11T08:44:18  * wumpus quickly checks if he has any servers/VPSes with dss keys he could lose access to
 792015-12-11T08:48:17  *** moli has quit IRC
 802015-12-11T08:48:45  <Luke-Jr> looks like I'm updating Qt in gitian
 812015-12-11T08:49:18  <wumpus> also wonder why gitian insists on creating a dsa key in the first place
 822015-12-11T08:50:57  <Luke-Jr> devrandom: ^
 832015-12-11T08:51:58  <gmaxwell> it would be fast.
 842015-12-11T08:52:54  <wumpus> maybe switch to ssh-ed25519
 852015-12-11T08:53:59  <gmaxwell> I dunno how widely supported that is, but perhaps that doesn't matter. E.g. centos won't have support for it.
 862015-12-11T08:54:42  <wumpus> what matters for gitian is what ubuntu versions support it, at least 14.04 does, but don't know if e.g. 12.04 does
 872015-12-11T08:56:01  <wumpus> then again, as this is only used to communciate with a vm on the local host, may be good enough to just add the force-use-dss option. Heck, using no cipher and auth would probably be acceptable...
 882015-12-11T08:57:22  <gmaxwell> or just set it to use 2kbit rsa, the generation time is probably not that bad on anything that can compile bitcoin.
 892015-12-11T08:57:23  <wumpus> (using ssh to communicate with something running on localhost is kind of redundant :-) )
 902015-12-11T08:58:49  <wumpus> changing that would mean regenerating all the base images though
 912015-12-11T09:03:29  <wumpus> Luke-Jr: if you change the ssh line in libexec/on-target to include -oHostKeyAlgorithms=+ssh-dss , does that work around the issue?
 922015-12-11T09:03:57  <Luke-Jr> you want PubkeyAcceptedKeyTypes=+ssh-dss
 932015-12-11T09:04:24  <wumpus> ok
 942015-12-11T09:05:06  <wumpus> but in this case it's about what your ssh *sends*, not what is accepted, which shouldn't have changed
 952015-12-11T09:06:19  <wumpus> none of the ubuntu versions used for gitian building has OpenSSH 7.0+, it must be the outer ssh refusing to send a dss key
 962015-12-11T09:07:37  <Luke-Jr> yes
 972015-12-11T09:08:33  <wumpus> so does it work?
 982015-12-11T09:08:48  <Luke-Jr> yes
 992015-12-11T09:09:19  <wumpus> great
1002015-12-11T09:16:05  <Luke-Jr> ugh, the build failure blocks caching the Qt? :/
1012015-12-11T09:18:13  <wumpus> yes that's kind of annoying
1022015-12-11T09:18:28  <wumpus> any build failure prevents caching of anything
1032015-12-11T09:19:09  <wumpus> cached dependencies are not handled separately from the build product, if it fails, it fails completely
1042015-12-11T09:20:03  <wumpus> may be possible to split up between a descriptor that just builds the dependencies and one that builds bitcoin, or some other trick
1052015-12-11T09:21:37  <wumpus> the idea behind the current design is AFAIK that you use gitian *if* you are virtually sure that the build will pass, as a gitian build is/should be equivalent to a depends builds outside gitian
1062015-12-11T09:22:27  <wumpus> (of course, subtle issues with e.g. the VM setup can always interfere so it's a bit brittle...)
1072015-12-11T09:54:37  <GitHub18> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f7f44b1bdd15...d1e17ff6408c
1082015-12-11T09:54:37  <GitHub18> bitcoin/master 9bbe71b Wladimir J. van der Laan: net: Add and document network messages in protocol.h...
1092015-12-11T09:54:38  <GitHub18> bitcoin/master d1e17ff Wladimir J. van der Laan: Merge pull request #7181...
1102015-12-11T09:54:42  <GitHub31> [bitcoin] laanwj closed pull request #7181: net: Add and document network messages in protocol.h (master...2015_12_network_messages) https://github.com/bitcoin/bitcoin/pull/7181
1112015-12-11T10:19:32  <GitHub141> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/8fc174aae64882d43549ad57bece0c23805e567b
1122015-12-11T10:19:32  <GitHub141> bitcoin/0.12 8fc174a Wladimir J. van der Laan: net: Add and document network messages in protocol.h...
1132015-12-11T10:28:50  *** randy-waterhouse has joined #bitcoin-core-dev
1142015-12-11T10:52:36  * Luke-Jr ponders if the cache ought to be a sshfs mount so it doesn't copy all 1 GB of files that don't get used every time
1152015-12-11T10:54:54  <Luke-Jr> hm, KVM can do a builtin SMB server for the host..
1162015-12-11T11:25:26  <wumpus> yea - if there is some built-in way to share filesystems in the virtualization solution, I think that makes more sense than using sshfs, which is slow
1172015-12-11T11:26:18  <wumpus> lxc can simply mount 'host' directories as well
1182015-12-11T11:26:37  <wumpus> so using a scratchpad instead of copying everything every time could work
1192015-12-11T11:28:56  <Luke-Jr> right
1202015-12-11T11:30:20  <wumpus> avoids both the ssh and copy overhead - but it's work, no one ever got around to implementing that, and testing all that
1212015-12-11T11:32:02  <wumpus> btw apparently marcofalke had already switched the key type to rsa with https://github.com/devrandom/gitian-builder/pull/107, I do think it should carry a clearer warning that now all the base vms have to be regenerated
1222015-12-11T11:33:12  <Luke-Jr> :x
1232015-12-11T11:33:49  <wumpus> although re: dependencies, arguably the copy overhead is minimal compared to the time it takes to build, or even prepare a VM for building (all this package upgrading..) so I'm not sure it's that much of a win in practice compared to the work
1242015-12-11T11:34:56  *** randy-waterhouse has quit IRC
1252015-12-11T11:37:03  <Luke-Jr> maybe not, just annoying ;)
1262015-12-11T11:37:43  <Luke-Jr> perhaps I'll just change it to use rsync so the download-to-host step is quicker
1272015-12-11T11:39:33  <Luke-Jr> oh blah, we're on ancient Ubuntu
1282015-12-11T11:49:00  <wumpus> using rsync would already be nice
1292015-12-11T11:49:28  <wumpus> not to ancient to have rsync I suppose?
1302015-12-11T11:57:19  *** Thireus has quit IRC
1312015-12-11T11:57:44  *** Thireus has joined #bitcoin-core-dev
1322015-12-11T11:58:49  *** Thireus has joined #bitcoin-core-dev
1332015-12-11T12:03:46  *** Thireus has quit IRC
1342015-12-11T12:15:22  * jonasschnelli is a fan of rsync/rdiff and rdiff backup
1352015-12-11T12:33:55  <morcos> wumpus: 7156 looks like it has a lot of ACK's (remove cs_main from createrawtransaction)
1362015-12-11T12:34:14  <wumpus> morcos: thanks
1372015-12-11T12:47:03  <jonasschnelli> I agree. It's merge ready.
1382015-12-11T12:53:12  *** BashCo has quit IRC
1392015-12-11T12:53:41  *** BashCo has joined #bitcoin-core-dev
1402015-12-11T12:58:29  *** jcorgan has quit IRC
1412015-12-11T13:29:01  *** BashCo has quit IRC
1422015-12-11T13:29:31  *** BashCo has joined #bitcoin-core-dev
1432015-12-11T13:42:53  *** Thireus has joined #bitcoin-core-dev
1442015-12-11T13:46:35  *** JackH_ has joined #bitcoin-core-dev
1452015-12-11T13:48:16  *** JackH has quit IRC
1462015-12-11T13:49:02  *** corb has joined #bitcoin-core-dev
1472015-12-11T13:51:26  *** JackH_ has quit IRC
1482015-12-11T13:56:08  <GitHub177> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d1e17ff6408c...9ee02cf564d1
1492015-12-11T13:56:09  <GitHub177> bitcoin/master 6e76587 Wladimir J. van der Laan: rpc: remove cs_main lock from `createrawtransaction`...
1502015-12-11T13:56:09  <GitHub177> bitcoin/master 9ee02cf Wladimir J. van der Laan: Merge pull request #7156...
1512015-12-11T13:56:13  <GitHub94> [bitcoin] laanwj closed pull request #7156: rpc: remove cs_main lock from `createrawtransaction` (master...2015_12_createrawtransaction_nolock) https://github.com/bitcoin/bitcoin/pull/7156
1522015-12-11T14:22:19  *** Tera2342 has quit IRC
1532015-12-11T14:33:47  *** Guyver2 has joined #bitcoin-core-dev
1542015-12-11T15:05:55  *** jannes has joined #bitcoin-core-dev
1552015-12-11T15:06:48  *** jcorgan has joined #bitcoin-core-dev
1562015-12-11T15:06:48  *** jcorgan has quit IRC
1572015-12-11T15:06:48  *** jcorgan has joined #bitcoin-core-dev
1582015-12-11T15:27:25  *** Thireus has quit IRC
1592015-12-11T15:27:47  *** Thireus has joined #bitcoin-core-dev
1602015-12-11T15:32:05  *** Thireus has quit IRC
1612015-12-11T15:36:53  *** Thireus has joined #bitcoin-core-dev
1622015-12-11T15:40:35  *** jtimon has joined #bitcoin-core-dev
1632015-12-11T15:52:59  *** moli has joined #bitcoin-core-dev
1642015-12-11T16:03:08  *** bawong has joined #bitcoin-core-dev
1652015-12-11T16:07:26  *** tripleslash has quit IRC
1662015-12-11T16:12:51  *** Thireus has quit IRC
1672015-12-11T16:13:41  *** bawong has quit IRC
1682015-12-11T16:17:03  *** bawong has joined #bitcoin-core-dev
1692015-12-11T16:17:18  *** Thireus has joined #bitcoin-core-dev
1702015-12-11T16:22:48  *** bawong has quit IRC
1712015-12-11T16:24:55  *** Thireus has quit IRC
1722015-12-11T16:32:26  *** jamesob has joined #bitcoin-core-dev
1732015-12-11T16:38:11  <jamesob> anyone have suggestions for parts of core that are under-unittested?
1742015-12-11T16:42:47  <morcos> jamesob: AcceptToMemoryPool but its one of the more difficult thngs to test.
1752015-12-11T16:43:13  <jamesob> in terms of getting the necessary fixture context set up, or something else?
1762015-12-11T16:43:36  <morcos> I was going to rework miner_test.cpp at some point, b/c many of them were designed to make sure the mining code did not assume the mempool contained txs that couldn't be mined, but now the mining code does assume that
1772015-12-11T16:43:54  <jamesob> hah, I see
1782015-12-11T16:44:09  <morcos> yes, in terms of setup.  i think there is a test that uses ATMP, but does it on regtest.
1792015-12-11T16:44:25  <morcos> which isn't really the right thing for more general ATMP tests
1802015-12-11T16:44:46  <morcos> the miner_tests actually construct blocks that can be used on mainnet i think
1812015-12-11T16:44:56  <morcos> so maybe that should be abstracted out into a setup that other tests can use
1822015-12-11T16:45:02  <jamesob> ATMP?
1832015-12-11T16:45:09  <morcos> AcceptToMemoryPool
1842015-12-11T16:45:13  <jamesob> ah
1852015-12-11T16:45:32  <jamesob> (first cup of coffee has yet to make it down ;)
1862015-12-11T16:46:24  <jamesob> yeah, I'll take a look, see if I can grok. may PM you if I get any ideas.
1872015-12-11T16:46:50  <morcos> sure
1882015-12-11T17:05:50  *** jouke has quit IRC
1892015-12-11T17:18:26  *** fkhan has quit IRC
1902015-12-11T17:32:48  *** fkhan has joined #bitcoin-core-dev
1912015-12-11T17:33:16  *** moli has quit IRC
1922015-12-11T17:33:40  *** moli has joined #bitcoin-core-dev
1932015-12-11T17:54:22  *** jcorgan is now known as jcorgan|away
1942015-12-11T19:08:31  *** larrysalibra has joined #bitcoin-core-dev
1952015-12-11T19:09:22  *** Luke-Jr has quit IRC
1962015-12-11T19:09:57  *** Luke-Jr has joined #bitcoin-core-dev
1972015-12-11T19:15:31  <gmaxwell> wumpus: No, we shouldn't revert #4906 we should just move forward from it.
1982015-12-11T19:15:35  *** larrysalibra has quit IRC
1992015-12-11T19:42:13  *** Thireus has joined #bitcoin-core-dev
2002015-12-11T19:44:40  *** jcorgan|away is now known as jcorgan
2012015-12-11T19:54:46  *** fkhan has quit IRC
2022015-12-11T19:55:41  *** tripleslash has joined #bitcoin-core-dev
2032015-12-11T20:03:26  *** jamesob has quit IRC
2042015-12-11T20:05:35  *** Thireus has quit IRC
2052015-12-11T20:08:32  *** fkhan has joined #bitcoin-core-dev
2062015-12-11T20:09:44  *** tripleslash_a has joined #bitcoin-core-dev
2072015-12-11T20:10:37  *** jamesob has joined #bitcoin-core-dev
2082015-12-11T20:11:22  *** tripleslash has quit IRC
2092015-12-11T20:12:14  *** PaulCapestany has quit IRC
2102015-12-11T20:15:16  *** PaulCapestany has joined #bitcoin-core-dev
2112015-12-11T20:26:15  *** Thireus has joined #bitcoin-core-dev
2122015-12-11T20:36:52  *** tripleslash_a has quit IRC
2132015-12-11T20:45:26  *** corb has quit IRC
2142015-12-11T20:45:46  *** raedah has joined #bitcoin-core-dev
2152015-12-11T20:47:38  *** jamesob has quit IRC
2162015-12-11T21:07:26  *** fkhan has quit IRC
2172015-12-11T21:09:46  *** jtimon has quit IRC
2182015-12-11T21:21:14  *** fkhan has joined #bitcoin-core-dev
2192015-12-11T21:41:13  *** Guyver2 has quit IRC
2202015-12-11T21:43:21  *** paulbernard has joined #bitcoin-core-dev
2212015-12-11T21:59:40  *** belcher has joined #bitcoin-core-dev
2222015-12-11T22:01:25  *** randy-waterhouse has joined #bitcoin-core-dev
2232015-12-11T22:47:35  *** randy-waterhouse has quit IRC
2242015-12-11T22:47:55  *** randy-waterhouse has joined #bitcoin-core-dev
2252015-12-11T23:14:13  *** bawong has joined #bitcoin-core-dev
2262015-12-11T23:27:18  *** raedah has quit IRC
2272015-12-11T23:27:35  <gmaxwell> I've been giving some though to bringing back the tiered fee behavior bitcoin had untul mid 2012. The earlier logic had a rate table where transactions were added in arrival order, with the required fee gradually going up as the non-fee-sorted candidate block became bigger. In 2012 it was changed to the sorting based selection on the basis that this was the rational behavior to miners, and the '
2282015-12-11T23:27:41  <gmaxwell> fullness' of a candidate block was pretty opaque to users.
2292015-12-11T23:28:25  <gmaxwell> What the logic was was that up to half the target size, the ordinary fee behavior applied. above half the target the required fee was multiplied by target/(target-current).
2302015-12-11T23:29:48  <gmaxwell> What we lost in that change is any backpressure at all, until the limit was met.  The belief that this was good for miners was mistaken in any case, since fees relative to orphan impact are small.
2312015-12-11T23:31:11  <gmaxwell> And then the need to constantly keep bumping the soft limit to avoid that cliff from the system being poorly stabalized also meant that blocks found fast would glup down a pile of spamflood.
2322015-12-11T23:37:17  <gmaxwell> I think we've probably not given this enough thought due to paying too much attention to what short term rational miner behavior would be; when probably really optimizing miner behavior would already not be including most of the transactions today at current feerates.
2332015-12-11T23:42:37  <zookolaptop> Right. Bounded rationality. I'm a businessman, and if there are two sources of revenue, one of which gives 100X what the other one gives, then I would tend to say "FUCK that other one! Don't bother me. Don't spend your time thinking about that other one.".
2342015-12-11T23:43:08  <zookolaptop> 100X is correct, currently, right?
2352015-12-11T23:45:15  *** raedah has joined #bitcoin-core-dev
2362015-12-11T23:45:36  <zookolaptop> That's what I wrote on twitter the other day, on my way home from Hong Kong. https://twitter.com/zooko/status/674422815299231744
2372015-12-11T23:55:07  *** droark has joined #bitcoin-core-dev
2382015-12-11T23:56:58  <gmaxwell> The other thing is that the rational marginal feerate needed to add an attitional transaction to a block actually goes down at the block gets bigger; not up.
2392015-12-11T23:58:10  <zookolaptop> Good point.
2402015-12-11T23:58:33  <gmaxwell> This is because the possion distributions of blocks, has an exponential distribution of interblock intervals, which has a slope that decreases with more delay.  Basically, if you're already screwed latency wise, the marginal harm of including another transaction is less.