1 2018-01-03 00:02:55	0|larafale|phantomcircuit: electrum return any utxos by address
  2 2018-01-03 00:58:02	0|phantomcircuit|larafale, ah
  3 2018-01-03 00:58:05	0|phantomcircuit|yeah then that will work
  4 2018-01-03 00:58:17	0|phantomcircuit|note that is querying a remote server to do that though
  5 2018-01-03 01:51:10	0|bitcoin-git|[13bitcoin] 15promag opened pull request #12079: Improve prioritisetransaction test coverage (06master...062018-01-prioritisetransaction) 02https://github.com/bitcoin/bitcoin/pull/12079
  6 2018-01-03 02:05:12	0|promag|BlueMatt: #11041 is green
  7 2018-01-03 02:05:14	0|gribble|https://github.com/bitcoin/bitcoin/issues/11041 | Add LookupBlockIndex by promag · Pull Request #11041 · bitcoin/bitcoin · GitHub
  8 2018-01-03 02:19:48	0|promag|should ParseHashV be used instead of ParseHashStr?
  9 2018-01-03 02:37:09	0|phantomcircuit|has anybody attempted to build bloomfilters of blocks? (without a commitment)
 10 2018-01-03 03:45:48	0|phantomcircuit|BlueMatt, the bloomfilter stuff matches against every data element in a scriptPubKey right
 11 2018-01-03 05:13:34	0|d3vt4r|sup everyone
 12 2018-01-03 07:54:46	0|sipa|phantomcircuit: neutrino
 13 2018-01-03 08:08:21	0|phantomcircuit|sipa, wat
 14 2018-01-03 08:08:23	0|phantomcircuit|i lost scroll back
 15 2018-01-03 08:14:15	0|sipa|you're asking if someone ever implemented block-wide bloomfilters without commitment
 16 2018-01-03 09:00:40	0|phantomcircuit|sipa, indeed, well i did as an rpc, i didn't see the bip pull 609 from roastbeef before hand
 17 2018-01-03 09:01:00	0|phantomcircuit|apparently there's something that's more space efficient
 18 2018-01-03 09:11:00	0|promag|wumpus: #623 is still valid I guess?
 19 2018-01-03 09:11:01	0|gribble|https://github.com/bitcoin/bitcoin/issues/623 | Address book cannot be searched · Issue #623 · bitcoin/bitcoin · GitHub
 20 2018-01-03 09:13:19	0|wumpus|promag: I think so, though in the interest of preventing address re-use, that's probably just as well
 21 2018-01-03 09:21:59	0|bitcoin-git|13bitcoin/06master 140d663f8 15251: Fixes issue #12067 `sendmany` curl example is wrong....
 22 2018-01-03 09:21:59	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/63a4dc10876b...c04cb48df961
 23 2018-01-03 09:22:00	0|bitcoin-git|13bitcoin/06master 14c04cb48 15Wladimir J. van der Laan: Merge #12077: [Docs] Correct `sendmany` curl example...
 24 2018-01-03 09:22:44	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #12077: [Docs] Correct `sendmany` curl example (06master...06patch/12067/fixes-sendmany-curl-example) 02https://github.com/bitcoin/bitcoin/pull/12077
 25 2018-01-03 09:33:33	0|bitcoin-git|13bitcoin/060.15 14ad83936 15Suhas Daftuar: Make boost::multi_index comparators const...
 26 2018-01-03 09:33:33	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 060.15: 02https://github.com/bitcoin/bitcoin/compare/2559a19e6fdb...cb7ef312ff34
 27 2018-01-03 09:33:34	0|bitcoin-git|13bitcoin/060.15 14cb7ef31 15Wladimir J. van der Laan: Merge #12032: [backport] #11847 Make boost::multi_index comparators const...
 28 2018-01-03 10:04:49	0|zelest|I hope I did the whole rebase thing correctly :o
 29 2018-01-03 10:10:36	0|bitcoin-git|13bitcoin/06master 14fadf60e 15MarcoFalke: qa: Note on test order in test_runner
 30 2018-01-03 10:10:36	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/c04cb48df961...d38d1a3e75aa
 31 2018-01-03 10:10:37	0|bitcoin-git|13bitcoin/06master 14d38d1a3 15MarcoFalke: Merge #11965: qa: Note on test order in test_runner...
 32 2018-01-03 10:11:11	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #11965: qa: Note on test order in test_runner (06master...06Mf1712-qaTestRunnerOrder) 02https://github.com/bitcoin/bitcoin/pull/11965
 33 2018-01-03 10:39:24	0|bitcoinzombie|Hi
 34 2018-01-03 11:40:41	0|promag|jnewbery: care to look #12079?
 35 2018-01-03 11:40:43	0|gribble|https://github.com/bitcoin/bitcoin/issues/12079 | Improve prioritisetransaction test coverage by promag · Pull Request #12079 · bitcoin/bitcoin · GitHub
 36 2018-01-03 13:33:55	0|bitcoin-git|13bitcoin/06master 14595a7ba 15Akira Takizawa: Increment MIT Licence copyright header year on files modified in 2017
 37 2018-01-03 13:33:55	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/d38d1a3e75aa...5961b23898ee
 38 2018-01-03 13:33:56	0|bitcoin-git|13bitcoin/06master 145961b23 15Wladimir J. van der Laan: Merge #12062: Increment MIT Licence copyright header year on files modified in 2017...
 39 2018-01-03 13:34:40	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #12062: Increment MIT Licence copyright header year on files modified in 2017 (06master...062018) 02https://github.com/bitcoin/bitcoin/pull/12062
 40 2018-01-03 15:06:00	0|bitcoin-git|13bitcoin/06master 14415f86c 15fanquake: [scripts] Add missing univalue file to copyright_header.py
 41 2018-01-03 15:06:00	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/5961b23898ee...c991b304dee3
 42 2018-01-03 15:06:01	0|bitcoin-git|13bitcoin/06master 14c991b30 15MarcoFalke: Merge #12075: [scripts] Add missing univalue file to copyright_header.py...
 43 2018-01-03 15:06:41	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #12075: [scripts] Add missing univalue file to copyright_header.py (06master...06copyright-add-missing) 02https://github.com/bitcoin/bitcoin/pull/12075
 44 2018-01-03 16:12:49	0|bitcoin-git|[13bitcoin] 15promag opened pull request #12080: Add support to search the address book (06master...062018-01-searchaddressbook) 02https://github.com/bitcoin/bitcoin/pull/12080
 45 2018-01-03 16:26:39	0|mrannanay|Are there tests for #12080 ?
 46 2018-01-03 16:26:41	0|gribble|https://github.com/bitcoin/bitcoin/issues/12080 | Add support to search the address book by promag · Pull Request #12080 · bitcoin/bitcoin · GitHub
 47 2018-01-03 16:26:52	0|promag|mrannanay: no
 48 2018-01-03 16:30:21	0|promag|sipa: https://github.com/bitcoin/bitcoin/pull/11403/files#diff-df7d84ff2f53fcb2a0dc15a3a51e55ceR241
 49 2018-01-03 16:30:44	0|promag|s/address_type/change_type ?
 50 2018-01-03 16:34:20	0|sipa|promag: why?
 51 2018-01-03 16:34:46	0|promag|sipa: left a comment
 52 2018-01-03 16:37:14	0|promag|I wonder if it could be just `type`, for instance, getnewaddress -type=bech32 or getrawchangeaddress -type=bech32
 53 2018-01-03 16:41:00	0|promag|MarcoFalke: #12076 nice
 54 2018-01-03 16:41:02	0|gribble|https://github.com/bitcoin/bitcoin/issues/12076 | qa: Use node_path wrapper in test framwork by MarcoFalke · Pull Request #12076 · bitcoin/bitcoin · GitHub
 55 2018-01-03 16:41:42	0|promag|MarcoFalke: you have a typo in the commit message
 56 2018-01-03 16:41:55	0|promag|and PR title
 57 2018-01-03 16:43:31	0|sipa|promag: the RPC is called getrawchange*address*
 58 2018-01-03 16:43:58	0|sipa|while regular change doesn't really turn into an address, for logistical reasons, when you use that RPC it is turned into one
 59 2018-01-03 16:45:39	0|promag|sipa: how about just `type`? anyway, probably a late suggestion
 60 2018-01-03 16:46:36	0|sipa|promag: that's far to generic i think
 61 2018-01-03 16:55:55	0|bitcoin-git|[13bitcoin] 15BTaskaya opened pull request #12081: It doesnt a coin system it is a AI's algorithm (06master...06master) 02https://github.com/bitcoin/bitcoin/pull/12081
 62 2018-01-03 16:59:20	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #12081: It doesnt a coin system it is a AI's algorithm (06master...06master) 02https://github.com/bitcoin/bitcoin/pull/12081
 63 2018-01-03 17:09:31	0|bitcoin-git|[13bitcoin] 15Christewart opened pull request #12082: Adding test case for SINGLE|ANYONECANPAY hash type in tx_valid.json (06master...06add_tx_valid_singleanyonecanpay) 02https://github.com/bitcoin/bitcoin/pull/12082
 64 2018-01-03 17:50:53	0|BlueMatt|someone wanna nack+close or otherwise look at #12051? (issue is adding gpl-2 stuff to dist tarball)
 65 2018-01-03 17:50:55	0|gribble|https://github.com/bitcoin/bitcoin/issues/12051 | add missing debian contrib files to tarball by puchu · Pull Request #12051 · bitcoin/bitcoin · GitHub
 66 2018-01-03 18:04:16	0|luke-jr|BlueMatt: if it's in git, it should be in the dist tarball
 67 2018-01-03 18:05:17	0|BlueMatt|I dont see why thats the case, its obviously a policy discussion we should discuss, not a hard rule, probably up to wumpus, I'd say
 68 2018-01-03 18:42:41	0|jimpo|If I want a stream interface to read from a byte vector, is my best option CDataStream? The downside is that it copies the the contents and in my use case, I can read from the vector by reference.
 69 2018-01-03 18:46:02	0|ryanofsky|jimpo, yeah i think that's the easiest thing to do. you could also write a new stream class that wraps already allocated storage
 70 2018-01-03 18:49:50	0|jimpo|OK, yeah, I was going to propose a CVectorReader (given that there's already a CVectorWriter), and wanted to check if there was some reason not to.
 71 2018-01-03 18:51:17	0|ryanofsky|it'd be nice to have. i wanted something like that for #10102, but was lazy and just used CDataStream instead
 72 2018-01-03 18:51:19	0|gribble|https://github.com/bitcoin/bitcoin/issues/10102 | bitcoin-qt: spawn bitcoind and communicate over pipe (Experimental, WIP, Depends on #10244) by ryanofsky · Pull Request #10102 · bitcoin/bitcoin · GitHub
 73 2018-01-03 18:54:33	0|sipa|jimpo: that may make sense; i wonder how many existing uses of CDataStream could use a CVectorReader
 74 2018-01-03 19:01:19	0|Chris_Stewart_5|so everytime you read from CDataStream everything is copied instead of referenced?
 75 2018-01-03 19:02:18	0|jimpo|Every time you create a CDataStream from a vector, the vector contents are copied
 76 2018-01-03 19:09:50	0|BlueMatt|jimpo: I have one implemented in fibre somewhere.....
 77 2018-01-03 19:10:20	0|BlueMatt|https://github.com/bitcoinfibre/bitcoinfibre/commit/bb1ffc5155ba7137ded82fa84291a6d54e1c8a5a
 78 2018-01-03 21:04:34	0|BlueMatt|0.15.1 erratta (backport-introduced bug): addnode $NODE onetry peers do not stay connected due to us assuming they are extra outbound nodes until #11456 makes them m_manual_connection nodes
 79 2018-01-03 21:04:36	0|gribble|https://github.com/bitcoin/bitcoin/issues/11456 | Replace relevant services logic with a function suite. by TheBlueMatt · Pull Request #11456 · bitcoin/bitcoin · GitHub
 80 2018-01-03 22:07:50	0|achow101|are the keypaths for the keys in a wallet stored anywhere?
 81 2018-01-03 22:10:00	0|ryanofsky|they should be in CKeyMetadata::hdKeypath
 82 2018-01-03 22:11:15	0|achow101|ugh, that's a string :(
 83 2018-01-03 22:13:56	0|phantomcircuit|achow101, indeed it is
 84 2018-01-03 22:14:05	0|phantomcircuit|why?
 85 2018-01-03 22:16:00	0|achow101|I'm doing things where they need to be integers and I would rather not parse a string
 86 2018-01-03 22:19:46	0|aadalbert|a
 87 2018-01-03 22:32:53	0|phantomcircuit|achow101, ... ok what things
 88 2018-01-03 22:33:27	0|achow101|implementing and writing tests for BIP 174
 89 2018-01-03 22:34:07	0|achow101|part of the spec is to include the bip32 derivation path for a pubkey as a vector of uint32's
 90 2018-01-03 22:35:40	0|phantomcircuit|achow101, isn't partially signed transactions implemented as simply missing the signature right now?
 91 2018-01-03 22:36:00	0|achow101|phantomcircuit: kind of
 92 2018-01-03 22:36:23	0|achow101|the whole point of bip 174 is that someone without access the the utxo set can sign a transaction
 93 2018-01-03 22:40:30	0|jb55|wait transaction broadcast is default off? was trying to figure out why sendtoaddress wasn't working
 94 2018-01-03 22:43:43	0|sipa|jb55: ?
 95 2018-01-03 22:43:51	0|jb55|6f252627b2843ff5072cb702b47e241f4ffbed92
 96 2018-01-03 22:44:05	0|sipa|?
 97 2018-01-03 22:44:12	0|jb55|added a switch to disable wallet broadcast, but it's default off
 98 2018-01-03 22:44:52	0|jb55|err
 99 2018-01-03 22:45:33	0|jb55|broadcast is default off I mean
100 2018-01-03 22:45:33	0|sipa|looks like default on to me
101 2018-01-03 22:45:48	0|jb55|line 552
102 2018-01-03 22:45:51	0|jb55|522
103 2018-01-03 22:46:24	0|jb55|https://github.com/bitcoin/bitcoin/commit/6f252627b2843ff5072cb702b47e241f4ffbed92#diff-12635a58447c65585f51d32b7e04075bR522
104 2018-01-03 22:46:31	0|sipa|that's just in CWallet
105 2018-01-03 22:46:46	0|sipa|in practice it's always overridden by command line handling
106 2018-01-03 22:47:12	0|sipa|it's also a patch from 2015
107 2018-01-03 22:47:24	0|jb55|I dont have -walletbroadcast set to anything, and broadcast is still disabled
108 2018-01-03 22:47:30	0|jb55|so ¯\_(ツ)_/¯?
109 2018-01-03 22:47:36	0|sipa|how do you know it is disabled?
110 2018-01-03 22:47:45	0|jb55|sec
111 2018-01-03 22:48:45	0|jb55|sipa: in CommitTransaction  there's a if (fBroadcastTransaction)
112 2018-01-03 22:48:59	0|jb55|I added a printf and it isn't entering the block
113 2018-01-03 22:49:19	0|sipa|when creating a new transaction?
114 2018-01-03 22:49:26	0|jb55|sendtoaddress
115 2018-01-03 22:50:13	0|sipa|i'm very skeptical about this, as disabling broadcast (since 2015!) would have pretty much killed the network...
116 2018-01-03 22:50:18	0|jb55|right!
117 2018-01-03 22:50:33	0|jb55|I'm very confused...
118 2018-01-03 22:50:53	0|jb55|oh wait I'm dumb I think
119 2018-01-03 22:51:00	0|achow101|jb55: are you sure it isn't set in your bitcoin.conf?
120 2018-01-03 22:51:38	0|jb55|I think I was assuming testnet used it's own bitcoin.conf?
121 2018-01-03 22:52:43	0|sipa|no
122 2018-01-03 22:53:03	0|jb55|my ~/.bitcoin/bitcoin.conf has it disabled, I guess testnet uses that?
123 2018-01-03 22:53:21	0|sipa|if you don't specify a dofferent confog file or datador, yes
124 2018-01-03 22:53:37	0|sipa|i seem to have a problem hotting the i key
125 2018-01-03 22:53:46	0|jb55|indeed
126 2018-01-03 22:54:02	0|sipa|there is a PR to add network-specific sections to confog foles
127 2018-01-03 22:55:16	0|jb55|so I need to do bitcoind -testnet -conf=~/.bitcoin/testnet3/bitcoin.conf ?
128 2018-01-03 22:55:30	0|jb55|for some reason I thought it would do that... thx sipa!
129 2018-01-03 23:40:16	0|promag|uint256S("foo") is happy, should `getblock foo` give block not found of invalid parameter?
130 2018-01-03 23:40:36	0|promag|I vote for invalid parameter..
131 2018-01-03 23:40:48	0|promag|s/of/or
132 2018-01-03 23:43:09	0|sipa|invalid parameter, agree
133 2018-01-03 23:44:34	0|promag|it's a breaking change
134 2018-01-03 23:44:40	0|promag|fine for 16?