12016-04-27T00:02:59  *** PRab has joined #bitcoin-core-dev
  22016-04-27T00:05:52  *** supasonic has quit IRC
  32016-04-27T00:14:09  *** belcher has quit IRC
  42016-04-27T00:18:42  *** BonyM has joined #bitcoin-core-dev
  52016-04-27T00:19:15  *** supasonic has joined #bitcoin-core-dev
  62016-04-27T00:20:48  *** BonyM1 has joined #bitcoin-core-dev
  72016-04-27T00:21:30  *** BonyM has quit IRC
  82016-04-27T00:21:30  *** BonyM1 has quit IRC
  92016-04-27T00:21:45  *** BonyM has joined #bitcoin-core-dev
 102016-04-27T00:23:12  *** BonyM has quit IRC
 112016-04-27T00:23:30  *** BonyM has joined #bitcoin-core-dev
 122016-04-27T00:26:06  *** Samdney has quit IRC
 132016-04-27T00:31:08  *** Ylbam has quit IRC
 142016-04-27T00:39:04  <achow101> The current master fails to cross compile for windows for me. Anyone else see this?
 152016-04-27T00:47:39  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 162016-04-27T00:56:21  *** BonyM has quit IRC
 172016-04-27T01:12:23  *** NotAnNSAgent has joined #bitcoin-core-dev
 182016-04-27T01:19:44  *** BonyM has joined #bitcoin-core-dev
 192016-04-27T01:30:28  *** dermoth has quit IRC
 202016-04-27T01:31:02  *** dermoth has joined #bitcoin-core-dev
 212016-04-27T02:01:44  *** BonyM has quit IRC
 222016-04-27T02:08:26  *** moli has quit IRC
 232016-04-27T02:08:50  *** moli has joined #bitcoin-core-dev
 242016-04-27T02:10:37  *** arowser has quit IRC
 252016-04-27T02:11:03  *** arowser has joined #bitcoin-core-dev
 262016-04-27T02:19:16  *** Chris_Stewart_5 has quit IRC
 272016-04-27T02:21:12  *** laurentmt has joined #bitcoin-core-dev
 282016-04-27T02:21:44  *** laurentmt has quit IRC
 292016-04-27T02:21:45  *** Giszmo has quit IRC
 302016-04-27T02:27:57  *** Cory has quit IRC
 312016-04-27T02:30:27  *** Pasha has joined #bitcoin-core-dev
 322016-04-27T02:34:21  <GitHub193> [bitcoin] achow101 opened pull request #7953: Create signmessagewithprivkey rpc (master...signmessagewithprivkey) https://github.com/bitcoin/bitcoin/pull/7953
 332016-04-27T02:37:20  *** Pasha is now known as Cory
 342016-04-27T02:50:50  *** zooko has quit IRC
 352016-04-27T02:51:02  *** fkhan has quit IRC
 362016-04-27T02:59:01  *** fengling has joined #bitcoin-core-dev
 372016-04-27T03:11:35  *** TomMc has joined #bitcoin-core-dev
 382016-04-27T03:18:44  *** TomMc has quit IRC
 392016-04-27T03:19:52  *** achow101 has quit IRC
 402016-04-27T03:20:13  *** fkhan has joined #bitcoin-core-dev
 412016-04-27T03:24:04  *** pmienk has quit IRC
 422016-04-27T03:37:25  *** pmienk has joined #bitcoin-core-dev
 432016-04-27T03:39:07  *** cryptapus_afk has quit IRC
 442016-04-27T03:47:01  *** Alopex has quit IRC
 452016-04-27T03:48:06  *** Alopex has joined #bitcoin-core-dev
 462016-04-27T04:03:01  *** Alopex has quit IRC
 472016-04-27T04:04:06  *** Alopex has joined #bitcoin-core-dev
 482016-04-27T04:29:01  *** Alopex has quit IRC
 492016-04-27T04:30:06  *** Alopex has joined #bitcoin-core-dev
 502016-04-27T04:35:23  *** zooko has joined #bitcoin-core-dev
 512016-04-27T04:52:59  *** NotAnNSAgent has quit IRC
 522016-04-27T05:02:14  *** pedrobranco has joined #bitcoin-core-dev
 532016-04-27T05:06:52  *** pedrobranco has quit IRC
 542016-04-27T05:18:39  <GitHub115> [bitcoin] theuni opened pull request #7954: build: quiet annoying warnings without adding new ones (master...clean-warnings) https://github.com/bitcoin/bitcoin/pull/7954
 552016-04-27T05:24:10  <GitHub131> [bitcoin] pstratem opened pull request #7955: [WIP] sync mempool after new block (master...2016-04-26-mempoolsync) https://github.com/bitcoin/bitcoin/pull/7955
 562016-04-27T05:30:13  *** xiangfu has joined #bitcoin-core-dev
 572016-04-27T05:34:52  *** xiangfu has quit IRC
 582016-04-27T05:35:56  *** fengling has quit IRC
 592016-04-27T05:36:23  *** xiangfu has joined #bitcoin-core-dev
 602016-04-27T05:41:51  *** supasonic has quit IRC
 612016-04-27T05:42:21  *** supasonic has joined #bitcoin-core-dev
 622016-04-27T05:52:41  *** roidster has quit IRC
 632016-04-27T06:00:04  *** dermoth has quit IRC
 642016-04-27T06:00:54  *** dermoth has joined #bitcoin-core-dev
 652016-04-27T06:03:57  *** arowser has quit IRC
 662016-04-27T06:04:27  *** arowser has joined #bitcoin-core-dev
 672016-04-27T06:06:53  *** ThomasV has joined #bitcoin-core-dev
 682016-04-27T06:08:09  *** fengling has joined #bitcoin-core-dev
 692016-04-27T06:17:33  *** BashCo has quit IRC
 702016-04-27T06:20:09  *** ThomasV has quit IRC
 712016-04-27T06:29:58  *** Thireus has quit IRC
 722016-04-27T06:31:51  *** Ylbam has joined #bitcoin-core-dev
 732016-04-27T06:36:58  <gmaxwell> So -- we have bytessent_per_msg and bytesrecv_per_msg ... would it be insane to ahve a bytes_implied to track bytes of blocks implicitly sent/recieved via comprblock/blocktxn in matt's block relay code?
 742016-04-27T06:37:55  *** ThomasV has joined #bitcoin-core-dev
 752016-04-27T06:41:05  *** BashCo has joined #bitcoin-core-dev
 762016-04-27T06:52:46  <da2ce7_mobile> is there a easy way to 'reset' an unconfirmed transaction in Bitcoin-QT?
 772016-04-27T06:54:11  *** cryptocoder has joined #bitcoin-core-dev
 782016-04-27T06:58:40  *** binns has quit IRC
 792016-04-27T06:58:40  *** Expanse has quit IRC
 802016-04-27T06:59:16  *** ibrightly has quit IRC
 812016-04-27T06:59:19  *** Guyver2 has joined #bitcoin-core-dev
 822016-04-27T07:01:13  *** ibrightly has joined #bitcoin-core-dev
 832016-04-27T07:01:15  *** binns has joined #bitcoin-core-dev
 842016-04-27T07:05:38  *** jtimon has quit IRC
 852016-04-27T07:06:01  *** Expanse has joined #bitcoin-core-dev
 862016-04-27T07:06:37  *** Thireus has joined #bitcoin-core-dev
 872016-04-27T07:07:56  *** Thireus has quit IRC
 882016-04-27T07:08:06  <da2ce7_mobile> I should be able to right click on a unconfirmed transaction and go 'delete' and then make a replacement one. (if I wish).
 892016-04-27T07:08:43  <da2ce7_mobile> But I suppose reddit would go crazy if you could do that. "meh-unconfirmed-tx-double-spends"
 902016-04-27T07:13:20  *** Thireus has joined #bitcoin-core-dev
 912016-04-27T07:16:22  *** xiangfu has quit IRC
 922016-04-27T07:17:41  *** cjcj has joined #bitcoin-core-dev
 932016-04-27T07:18:07  *** xiangfu has joined #bitcoin-core-dev
 942016-04-27T07:18:27  <sipa> da2ce7_mobile: there is a pull request for adding support for abandoning a transactionz and RPCs for respending with higher fee
 952016-04-27T07:29:47  <luke-jr> da2ce7_mobile: the right click option needs to do the replacement too, so it can ensure there's a shared input
 962016-04-27T07:31:04  *** ThomasV has quit IRC
 972016-04-27T07:31:19  *** gevs has quit IRC
 982016-04-27T07:33:14  <da2ce7_mobile> what is the best way to do it *now* if I don't want to compile bitcoin with that pull request merged?
 992016-04-27T07:34:05  <luke-jr> da2ce7_mobile: the abandon RPC + coin control to ensure you double-spend it
1002016-04-27T07:35:15  <da2ce7_mobile> I try: abandontransaction "txid"  but it fails.
1012016-04-27T07:35:32  *** cryptocoder has quit IRC
1022016-04-27T07:35:34  <da2ce7_mobile> "TX is not eligible for abandonment (code -5)"
1032016-04-27T07:35:45  <da2ce7_mobile> I cannot find the documentation to explain what code -5 is.
1042016-04-27T07:38:10  <sipa> da2ce7_mobile: the code will not tell you anything more than the text
1052016-04-27T07:40:23  <sipa> da2ce7_mobile: and the rpc help for abandontransaction will explain more
1062016-04-27T07:43:34  *** gevs has joined #bitcoin-core-dev
1072016-04-27T07:43:34  *** gevs has joined #bitcoin-core-dev
1082016-04-27T07:48:19  *** Samdney has joined #bitcoin-core-dev
1092016-04-27T07:56:53  *** Samdney has quit IRC
1102016-04-27T08:01:15  *** ThomasV has joined #bitcoin-core-dev
1112016-04-27T08:26:21  *** murch has joined #bitcoin-core-dev
1122016-04-27T08:33:53  *** justanotheruser has quit IRC
1132016-04-27T08:42:27  *** earlest has joined #bitcoin-core-dev
1142016-04-27T08:44:07  *** AaronvanW has joined #bitcoin-core-dev
1152016-04-27T08:46:04  *** muuqwaul has quit IRC
1162016-04-27T08:52:35  *** jannes has joined #bitcoin-core-dev
1172016-04-27T08:53:39  <GitHub28> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/e26b62093ae2...5c7df7022bcd
1182016-04-27T08:53:40  <GitHub28> bitcoin/master 5555528 MarcoFalke: [qa] mininode: Unfiddle strings into bytes
1192016-04-27T08:53:40  <GitHub28> bitcoin/master fada064 MarcoFalke: [qa] test_framework: Properly print exceptions and assert empty dict
1202016-04-27T08:53:41  <GitHub28> bitcoin/master 5c7df70 MarcoFalke: Merge #7951: [qa] test_framework: Properly print exception...
1212016-04-27T08:53:49  <GitHub148> [bitcoin] MarcoFalke closed pull request #7951: [qa] test_framework: Properly print exception (master...Mf1604-qaMinorFixes) https://github.com/bitcoin/bitcoin/pull/7951
1222016-04-27T09:01:48  *** jarret has quit IRC
1232016-04-27T09:01:49  *** arowser_ has joined #bitcoin-core-dev
1242016-04-27T09:02:39  *** arowser has quit IRC
1252016-04-27T09:08:21  *** supasonic has quit IRC
1262016-04-27T09:08:40  *** pedrobranco has joined #bitcoin-core-dev
1272016-04-27T09:08:51  *** pedrobranco has joined #bitcoin-core-dev
1282016-04-27T09:11:18  *** frankenmint has quit IRC
1292016-04-27T09:11:40  *** cjcj has quit IRC
1302016-04-27T09:12:32  *** frankenmint has joined #bitcoin-core-dev
1312016-04-27T09:13:32  <GitHub56> [bitcoin] laanwj opened pull request #7956: test: Add more thorough test for dbwrapper iterators (master...2016_04_dbwrapper_iterator_tess) https://github.com/bitcoin/bitcoin/pull/7956
1322016-04-27T09:17:07  *** frankenmint has quit IRC
1332016-04-27T09:20:02  *** ThomasV has quit IRC
1342016-04-27T09:27:25  *** pmienk has quit IRC
1352016-04-27T09:27:54  *** xiangfu has quit IRC
1362016-04-27T09:35:20  *** frankenmint has joined #bitcoin-core-dev
1372016-04-27T09:55:46  *** ThomasV has joined #bitcoin-core-dev
1382016-04-27T10:08:29  *** pmienk has joined #bitcoin-core-dev
1392016-04-27T10:29:36  <gmaxwell> https://bugzilla.redhat.com/show_bug.cgi?id=1020292  < any idea why fedora is building core with tests disabled?
1402016-04-27T10:31:42  *** cjcj has joined #bitcoin-core-dev
1412016-04-27T10:33:13  <luke-jr> gmaxwell: dependencies perhaps?
1422016-04-27T10:33:33  <luke-jr> although I think it's just python-zmq, IIRC we need a rather new version of it
1432016-04-27T10:33:42  <luke-jr> and that's just Python tests
1442016-04-27T10:34:01  <sipa> libsecp256k1 tests try to link and run with OpenSSL EC
1452016-04-27T10:34:06  <sipa> maybe that matters as well?
1462016-04-27T10:34:43  <gmaxwell> they'll work without openssl, but may not handle the goofed up openssl ec in fedora.
1472016-04-27T10:38:38  *** frankenmint has quit IRC
1482016-04-27T10:39:18  *** Samdney has joined #bitcoin-core-dev
1492016-04-27T10:44:18  *** xiangfu has joined #bitcoin-core-dev
1502016-04-27T10:52:15  *** achow101 has joined #bitcoin-core-dev
1512016-04-27T10:55:16  <wumpus> bah, so it should detect deliberately broken openssl as well
1522016-04-27T10:55:44  <wumpus> luke-jr: only the RPC tests need python-zmq though; I doubt they'd run those
1532016-04-27T10:56:01  <wumpus> this must be related to either test_bitcoin or secp256k1 tests
1542016-04-27T10:58:01  <GitHub24> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5c7df7022bcd...08b37c5e06bf
1552016-04-27T10:58:01  <GitHub24> bitcoin/master 63b3111 Cory Fields: build: quiet annoying warnings without adding new ones...
1562016-04-27T10:58:02  <GitHub24> bitcoin/master 08b37c5 Wladimir J. van der Laan: Merge #7954: build: quiet annoying warnings without adding new ones...
1572016-04-27T10:58:11  <GitHub89> [bitcoin] laanwj closed pull request #7954: build: quiet annoying warnings without adding new ones (master...clean-warnings) https://github.com/bitcoin/bitcoin/pull/7954
1582016-04-27T11:00:47  *** laurentmt has joined #bitcoin-core-dev
1592016-04-27T11:01:12  *** laurentmt has quit IRC
1602016-04-27T11:03:15  *** cjcj has quit IRC
1612016-04-27T11:10:02  *** ThomasV has quit IRC
1622016-04-27T11:19:23  *** cryptapus has joined #bitcoin-core-dev
1632016-04-27T11:24:08  *** Guyver2 has quit IRC
1642016-04-27T11:35:16  *** fengling has quit IRC
1652016-04-27T11:45:30  *** JackH has quit IRC
1662016-04-27T11:48:31  *** jarret has joined #bitcoin-core-dev
1672016-04-27T11:52:39  *** Samdney has quit IRC
1682016-04-27T11:58:15  *** JackH has joined #bitcoin-core-dev
1692016-04-27T11:59:20  <GitHub156> [bitcoin] laanwj closed pull request #7899: optionally provide privkey to signmessage RPC (master...signmessage-using-privkey) https://github.com/bitcoin/bitcoin/pull/7899
1702016-04-27T12:00:44  *** ThomasV has joined #bitcoin-core-dev
1712016-04-27T12:03:43  *** jonasschnelli has quit IRC
1722016-04-27T12:03:43  *** jonasschnelli has joined #bitcoin-core-dev
1732016-04-27T12:12:03  <GitHub122> [bitcoin] laanwj pushed 3 new commits to 0.12: https://github.com/bitcoin/bitcoin/compare/89ae85484c8b...18b3c3ced812
1742016-04-27T12:12:04  <GitHub122> bitcoin/0.12 9ca957b Wladimir J. van der Laan: tests: Make proxy_test work on travis servers without IPv6...
1752016-04-27T12:12:04  <GitHub122> bitcoin/0.12 564aaa2 Cory Fields: travis: switch to Trusty...
1762016-04-27T12:12:05  <GitHub122> bitcoin/0.12 18b3c3c Wladimir J. van der Laan: Merge #7950: [0.12] travis: switch to Trusty...
1772016-04-27T12:12:05  <GitHub85> [bitcoin] laanwj closed pull request #7950: [0.12] travis: switch to Trusty (0.12...Mf1604-012travisTrusty) https://github.com/bitcoin/bitcoin/pull/7950
1782016-04-27T12:13:09  *** davec has quit IRC
1792016-04-27T12:13:41  *** justanotheruser has joined #bitcoin-core-dev
1802016-04-27T12:13:53  *** davec has joined #bitcoin-core-dev
1812016-04-27T12:33:09  *** gevs has quit IRC
1822016-04-27T12:49:14  *** xiangfu has quit IRC
1832016-04-27T12:50:42  *** xiangfu has joined #bitcoin-core-dev
1842016-04-27T12:51:40  *** Samdney has joined #bitcoin-core-dev
1852016-04-27T12:57:10  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1862016-04-27T13:05:47  *** xiangfu has quit IRC
1872016-04-27T13:07:21  *** xiangfu has joined #bitcoin-core-dev
1882016-04-27T13:09:09  <Chris_Stewart_5> Should this test case fail as a INVALID_STACK_OPERATION instead of BAD_OPCODE?
1892016-04-27T13:09:12  <Chris_Stewart_5> ["0x4c01","0x01 NOP", "P2SH,STRICTENC","BAD_OPCODE", "PUSHDATA1 with not enough bytes"]
1902016-04-27T13:11:27  <afk11> Chris_Stewart_5, the test fails because of line 265 I imagine - script.GetOp is responsible for loading the opcode, and pushdata if any. That case has pushdata1 + the length marker, but is missing a byte the interpreter knows should be there
1912016-04-27T13:11:52  <afk11> hence that function would fail, resulting in SCRIPT_ERR_BAD_OPCODE
1922016-04-27T13:12:11  <afk11> oops, I'm talking about interpreter.cpp's line 265
1932016-04-27T13:13:49  *** TomMc has joined #bitcoin-core-dev
1942016-04-27T13:14:01  <sipa> there is no problem with the stack in that code :)
1952016-04-27T13:14:17  <sipa> the stack isn't even touched by the failing instruction
1962016-04-27T13:18:27  <sipa> afk11: indeed
1972016-04-27T13:23:42  <Chris_Stewart_5> I guess I was thinking in the sense that it is failing to push something onto the stack.. however if we continue that though almost every error would be an invalid stack op
1982016-04-27T13:23:49  <Chris_Stewart_5> thought*
1992016-04-27T13:30:31  *** NotAnNSAgent has joined #bitcoin-core-dev
2002016-04-27T13:33:32  *** fengling has joined #bitcoin-core-dev
2012016-04-27T13:37:13  <afk11> Chris_Stewart_5, it's only thrown once in the codebase - if the opcode is totally invalid
2022016-04-27T13:38:17  *** fengling has quit IRC
2032016-04-27T13:38:27  <Chris_Stewart_5> afk11: Not sure what you mean by that. In the case we are talking about the op code is valid, however the arguments for the op are invalid
2042016-04-27T13:39:37  *** Samdney has quit IRC
2052016-04-27T13:41:18  <Chris_Stewart_5> I guess the semantics for SCRIPT_ERROR_BAD_OPCODE extend to the op code AND its implicit arguments
2062016-04-27T13:42:59  <GitHub11> [bitcoin] jonasschnelli opened pull request #7957: [RPC][Bitcoin-TX] Add support for sequence number (master...2016/04/rbf_base) https://github.com/bitcoin/bitcoin/pull/7957
2072016-04-27T13:44:56  <afk11> Chris_Stewart_5, indeed. check here to see why https://github.com/bitcoin/bitcoin/blob/master/src/script/script.h#L503
2082016-04-27T13:52:21  *** ThomasV has quit IRC
2092016-04-27T13:55:19  <Chris_Stewart_5> Thanks afk11
2102016-04-27T14:08:40  *** jtimon has joined #bitcoin-core-dev
2112016-04-27T14:10:33  *** xiangfu has quit IRC
2122016-04-27T14:14:19  *** Giszmo has joined #bitcoin-core-dev
2132016-04-27T14:15:20  *** xiangfu has joined #bitcoin-core-dev
2142016-04-27T14:19:52  *** Samdney has joined #bitcoin-core-dev
2152016-04-27T14:29:13  *** lysobit is now known as aRUMtest
2162016-04-27T14:29:26  *** aRUMtest is now known as lysobit
2172016-04-27T14:34:36  *** xiangfu has quit IRC
2182016-04-27T14:37:18  *** teward has quit IRC
2192016-04-27T14:39:49  *** teward has joined #bitcoin-core-dev
2202016-04-27T14:43:30  <Chris_Stewart_5> Are there any script_tests that test CHECKSEQUENCEVERIFY? I did a quick search and it appears that term doesn't appear in script_tests.json
2212016-04-27T14:50:31  *** BashCo has quit IRC
2222016-04-27T15:02:19  *** aburan28 has joined #bitcoin-core-dev
2232016-04-27T15:10:26  *** xiangfu has joined #bitcoin-core-dev
2242016-04-27T15:10:59  *** aburan28 has quit IRC
2252016-04-27T15:16:47  *** BashCo has joined #bitcoin-core-dev
2262016-04-27T15:20:59  *** paveljanik has joined #bitcoin-core-dev
2272016-04-27T15:24:15  *** aburan28 has joined #bitcoin-core-dev
2282016-04-27T15:24:43  <sipa> Chris_Stewart_5: sure, in qa/rpc-tests there are some
2292016-04-27T15:27:04  *** xiangfu has quit IRC
2302016-04-27T15:32:47  *** ThomasV has joined #bitcoin-core-dev
2312016-04-27T15:34:10  *** bysherper has joined #bitcoin-core-dev
2322016-04-27T15:36:56  *** fengling has joined #bitcoin-core-dev
2332016-04-27T15:37:04  *** earlest has quit IRC
2342016-04-27T15:37:59  *** zooko has quit IRC
2352016-04-27T15:39:08  *** gevs has joined #bitcoin-core-dev
2362016-04-27T15:41:36  *** fengling has quit IRC
2372016-04-27T15:58:13  *** earlest has joined #bitcoin-core-dev
2382016-04-27T16:01:04  *** bysherper has quit IRC
2392016-04-27T16:01:47  <GitHub78> [bitcoin] paveljanik closed pull request #7810: Refactor AlertNotifyOnce out of UpdateTip (master...20160405_AlertNotifyOnce) https://github.com/bitcoin/bitcoin/pull/7810
2402016-04-27T16:04:28  *** bysherper has joined #bitcoin-core-dev
2412016-04-27T16:07:34  *** earlest has quit IRC
2422016-04-27T16:09:40  <GitHub25> [bitcoin] paveljanik opened pull request #7958: Remove useless argument to AlertNotify. (master...20160427_AlertNotify_remove_arg) https://github.com/bitcoin/bitcoin/pull/7958
2432016-04-27T16:19:34  *** pedrobranco has quit IRC
2442016-04-27T16:20:11  *** pedrobranco has joined #bitcoin-core-dev
2452016-04-27T16:25:43  *** achow101 has quit IRC
2462016-04-27T16:27:07  *** supasonic has joined #bitcoin-core-dev
2472016-04-27T16:30:31  *** achow101 has joined #bitcoin-core-dev
2482016-04-27T16:35:02  *** pmienk has quit IRC
2492016-04-27T16:43:33  <GitHub147> [bitcoin] kazcw opened pull request #7959: fix race that could fail to persist a ban (master...banmap_race) https://github.com/bitcoin/bitcoin/pull/7959
2502016-04-27T16:48:54  *** xiangfu has joined #bitcoin-core-dev
2512016-04-27T16:50:15  *** pmienk has joined #bitcoin-core-dev
2522016-04-27T16:51:19  <kanzure> still seeing "Request-sent" errors from a python application using python-bitcoinlib's rpc connection object, originally i reported this as #6454 which was fixed by using rpckeepalive=0 (#6454 had an additional issue where rpc would become completely locked up).
2532016-04-27T16:52:01  <kanzure> my more recent encounters of "Request-sent" are unfortunately not associated with that rpc thread locking behavior
2542016-04-27T16:53:17  <kanzure> unsure whether this is a python issue or a bitcoind issue. i've found that creating new python-bitcoinlib rpc connection objects (instead of reusing a single object over the course of a few hundred rpc requests and a few minutes) does not cause the "Request-sent" error..
2552016-04-27T16:55:16  <kanzure> (specifically the exception is "http.client.CannotSendRequest: Request-sent")
2562016-04-27T17:02:29  *** xiangfu has quit IRC
2572016-04-27T17:04:03  <kanzure> https://github.com/python/cpython/blob/436064bfe71ffd782dc5b942f65b80f7f0d7d2cd/Lib/http/client.py#L1002
2582016-04-27T17:12:44  *** ThomasV has quit IRC
2592016-04-27T17:14:09  *** imsaguy is now known as tripleslash
2602016-04-27T17:21:27  <jl2012> Will all disabled op codes (e.g. OP_CAT) fail the script, even in an unexecuted branch?
2612016-04-27T17:23:43  <cfields> MarcoFalke: ping
2622016-04-27T17:23:57  <MarcoFalke> pong]
2632016-04-27T17:24:25  <cfields> MarcoFalke: not sure if you saw yesterday... have you looked into parallelizing the rpc tests?
2642016-04-27T17:25:05  <MarcoFalke> Not yet, other things are holding me back right now.
2652016-04-27T17:25:45  <cfields> ok. But there's not any major reason not to?
2662016-04-27T17:25:58  <MarcoFalke> no
2672016-04-27T17:26:32  <cfields> great. I experimented with it yesterday, and it seemed to work quite well. Unfortunately my python is terrible though, so it was just hacking and slashing
2682016-04-27T17:26:56  <MarcoFalke> Oh, nice!
2692016-04-27T17:27:19  <MarcoFalke> My python-foo is not that high, either :)
2702016-04-27T17:27:50  <cfields> heh, that's not true at all :)
2712016-04-27T17:29:06  <cfields> only major change necessary seems to be listening port definition. atm we use the pid..pid+3 range, which causes overlaps in tests (assuming they're launched in separate processes as opposed to threads). My solution was to pass in the port range everywhere instead
2722016-04-27T17:29:42  <cfields> er, func(pid...pid+3), not the pids themselves
2732016-04-27T17:31:57  *** Guest83827 has joined #bitcoin-core-dev
2742016-04-27T17:31:58  <MarcoFalke> you mean passing in from rpc-tests.py ?
2752016-04-27T17:33:07  <MarcoFalke> I can have a look at the patch if you push it
2762016-04-27T17:33:22  *** Guest83827 is now known as roidster
2772016-04-27T17:34:23  <cfields> yea, the framework accepts a new option for port range, which defaults to the old behavior of func(pid). In the pull-tester case, it overrides
2782016-04-27T17:40:12  *** fengling has joined #bitcoin-core-dev
2792016-04-27T17:44:57  *** fengling has quit IRC
2802016-04-27T17:51:33  *** Arnavion has quit IRC
2812016-04-27T18:16:00  *** murch has quit IRC
2822016-04-27T18:16:17  *** murch has joined #bitcoin-core-dev
2832016-04-27T18:19:14  <GitHub19> [bitcoin] sdaftuar opened pull request #7960: Only use AddInventoryKnown for transactions (master...block-inv-filter) https://github.com/bitcoin/bitcoin/pull/7960
2842016-04-27T18:22:10  *** pedrobranco has quit IRC
2852016-04-27T18:22:42  *** pedrobranco has joined #bitcoin-core-dev
2862016-04-27T18:24:04  *** pedrobranco has quit IRC
2872016-04-27T18:24:26  *** pedrobranco has joined #bitcoin-core-dev
2882016-04-27T18:28:57  *** pedrobranco has quit IRC
2892016-04-27T18:41:22  *** earlest has joined #bitcoin-core-dev
2902016-04-27T18:42:38  *** Arnavion has joined #bitcoin-core-dev
2912016-04-27T18:44:34  *** bysherper has quit IRC
2922016-04-27T18:45:24  *** bysherper has joined #bitcoin-core-dev
2932016-04-27T18:46:57  *** pedrobranco has joined #bitcoin-core-dev
2942016-04-27T18:47:28  *** muuqwaul has joined #bitcoin-core-dev
2952016-04-27T18:48:34  *** earlest has quit IRC
2962016-04-27T18:48:38  *** treehug88 has joined #bitcoin-core-dev
2972016-04-27T18:50:04  *** bysherper has quit IRC
2982016-04-27T18:51:19  *** pedrobranco has quit IRC
2992016-04-27T18:53:59  *** ThomasV has joined #bitcoin-core-dev
3002016-04-27T18:56:23  *** earlest has joined #bitcoin-core-dev
3012016-04-27T18:58:22  *** bysherper has joined #bitcoin-core-dev
3022016-04-27T18:59:34  *** muuqwaul has quit IRC
3032016-04-27T19:00:28  *** muuqwaul has joined #bitcoin-core-dev
3042016-04-27T19:01:34  *** earlest has quit IRC
3052016-04-27T19:02:53  *** earlest has joined #bitcoin-core-dev
3062016-04-27T19:03:04  *** bysherper has quit IRC
3072016-04-27T19:06:04  *** muuqwaul has quit IRC
3082016-04-27T19:07:49  *** Chris_Stewart_5 has quit IRC
3092016-04-27T19:23:12  *** aburan28 has quit IRC
3102016-04-27T19:28:28  *** ThomasV has quit IRC
3112016-04-27T19:36:42  *** aburan28 has joined #bitcoin-core-dev
3122016-04-27T19:43:20  *** fengling has joined #bitcoin-core-dev
3132016-04-27T19:48:16  *** fengling has quit IRC
3142016-04-27T20:03:47  *** Chris_Stewart_5 has joined #bitcoin-core-dev
3152016-04-27T20:05:35  *** pmienk has quit IRC
3162016-04-27T20:07:32  *** cryptapus has quit IRC
3172016-04-27T20:11:57  *** aburan28 has quit IRC
3182016-04-27T20:14:08  *** pmienk has joined #bitcoin-core-dev
3192016-04-27T20:14:21  *** aburan28 has joined #bitcoin-core-dev
3202016-04-27T20:16:57  <sipa> cfields: do you have any idea whether memset_s is something common in recent C compilers?
3212016-04-27T20:17:12  <sipa> cfields: because if it's there, we should use it instead of OpenSSL_cleanse
3222016-04-27T20:19:33  *** BlueMatt has quit IRC
3232016-04-27T20:21:01  *** BlueMatt has joined #bitcoin-core-dev
3242016-04-27T20:25:42  *** ThomasV has joined #bitcoin-core-dev
3252016-04-27T20:31:27  *** Chris_Stewart_5 has quit IRC
3262016-04-27T20:35:10  *** AaronvanW has quit IRC
3272016-04-27T20:46:04  *** amiller has quit IRC
3282016-04-27T20:50:58  *** ThomasV has quit IRC
3292016-04-27T20:52:14  *** Guest47532 has joined #bitcoin-core-dev
3302016-04-27T20:52:54  *** Guest47532 is now known as amiller
3312016-04-27T20:52:54  *** amiller has joined #bitcoin-core-dev
3322016-04-27T20:56:50  *** frankenmint has joined #bitcoin-core-dev
3332016-04-27T20:58:59  <luke-jr> sipa: I don't even have a manpage for memset_s :p
3342016-04-27T21:08:48  <sipa> luke-jr: eh, neither do i!
3352016-04-27T21:17:20  *** ThomasV has joined #bitcoin-core-dev
3362016-04-27T21:20:21  *** Giszmo has quit IRC
3372016-04-27T21:23:13  *** TomMc has quit IRC
3382016-04-27T21:25:25  <gmaxwell> sipa: its an appendix of the langauge, and seemingly not quick in coming.
3392016-04-27T21:25:39  <gmaxwell> looks like we should implement our own. :(
3402016-04-27T21:27:24  <sipa> this may be interesting: https://mail-index.netbsd.org/tech-userlevel/2012/02/25/msg006157.html
3412016-04-27T21:28:26  <luke-jr> hm
3422016-04-27T21:28:52  <sipa> (not suggesting we use NetBSD, but that approach may be useful)
3432016-04-27T21:29:46  <luke-jr> right
3442016-04-27T21:30:48  <sipa> or we could just include a piece of asm for common platforms *ducks*
3452016-04-27T21:32:39  <gmaxwell> sipa: it was my expectation that we had to do that.. the argument that you could use a volitile function pointer is new to me...
3462016-04-27T21:33:00  <sipa> (the extended asm syntax supported by gcc and clang can indicate that it has side-effects, and the compiler is not allowed to remove it or duplicate it)
3472016-04-27T21:33:47  <sipa> gmaxwell: how about you make two wrappers around memset, each which modifies a TLS pointer to point to the other one
3482016-04-27T21:34:00  <sipa> not only is it volatile then, it's even actually changing :)
3492016-04-27T21:35:07  <gmaxwell> the only disagreement I might have with that volitile pointer argument is that for a long time many compilers frequently miscompiled code when volitile was in use (sometimes even worse than just ignoring the volitile)-- though in the last 5 years that has gotten _much_ better.
3502016-04-27T21:37:33  *** cryptapus has joined #bitcoin-core-dev
3512016-04-27T21:37:33  *** cryptapus has joined #bitcoin-core-dev
3522016-04-27T21:52:34  *** ThomasV has quit IRC
3532016-04-27T21:55:45  *** MarcoFalke has quit IRC
3542016-04-27T21:58:28  *** bysherper has joined #bitcoin-core-dev
3552016-04-27T21:58:41  <luke-jr> inb4 it compiles to: if (memset_volatile != memset) memset_volatile(…);
3562016-04-27T22:00:50  *** AaronvanW has joined #bitcoin-core-dev
3572016-04-27T22:01:34  *** earlest has quit IRC
3582016-04-27T22:04:14  *** treehug88 has quit IRC
3592016-04-27T22:14:20  *** randy-waterhouse has joined #bitcoin-core-dev
3602016-04-27T22:16:55  *** Samdney has left #bitcoin-core-dev
3612016-04-27T22:27:18  *** droark has quit IRC
3622016-04-27T22:31:58  *** Chris_Stewart_5 has joined #bitcoin-core-dev
3632016-04-27T22:35:39  *** aburan28 has quit IRC
3642016-04-27T22:58:35  *** aburan28 has joined #bitcoin-core-dev
3652016-04-27T23:08:43  *** belcher has joined #bitcoin-core-dev
3662016-04-27T23:08:53  *** belcher has quit IRC
3672016-04-27T23:08:53  *** belcher has joined #bitcoin-core-dev
3682016-04-27T23:37:05  *** murch has quit IRC
3692016-04-27T23:49:20  *** frankenmint has quit IRC
3702016-04-27T23:55:50  <cfields> sipa: i've wrestled with that one. We could do an autoconf check for it, but if it's not there we'd fall back to something else. So ultimately we're stuck with a reliance on something else, or we roll our own and don't bother checking for memset_s