--- Log opened Wed Jun 16 00:00:39 2021 00:05 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 00:08 -!- sibilant_ [~sibilant@user/sibilant] has quit [Quit: Leaving] 00:08 -!- sibilant_ [~sibilant@host-95-197-90-27.mobileonline.telia.com] has joined #bitcoin-core-dev 00:08 -!- sibilant_ [~sibilant@host-95-197-90-27.mobileonline.telia.com] has quit [Changing host] 00:08 -!- sibilant_ [~sibilant@user/sibilant] has joined #bitcoin-core-dev 00:10 -!- sibilant_ [~sibilant@user/sibilant] has quit [Client Quit] 00:10 -!- sibilant [~sibilant@user/sibilant] has joined #bitcoin-core-dev 00:11 -!- kallewoof is now known as kalle 00:13 -!- jackielove4u [uid43977@user/jackielove4u] has joined #bitcoin-core-dev 00:42 -!- kabaum [~kabaum@host-78-77-216-135.mobileonline.telia.com] has joined #bitcoin-core-dev 00:54 -!- evias [~evias__@user/evias] has joined #bitcoin-core-dev 01:02 -!- lkqwejhhgasdjhgn [~kljkljklk@p200300d46f03bc00adf9c5ad604010d2.dip0.t-ipconnect.de] has joined #bitcoin-core-dev 01:27 < vasild> Is my understanding correct that we don't send addresses or transactions via block-only connections, but the peer on the other end of a block-only connection may still send us addresses or transactions? Is the peer even aware that it is block-only connection for us? 01:33 -!- arnabsen [~arnabsen@45.116.190.221] has quit [Ping timeout: 250 seconds] 01:41 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has joined #bitcoin-core-dev 02:02 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has quit [Remote host closed the connection] 02:15 -!- lightlike [~lightlike@user/lightlike] has joined #bitcoin-core-dev 02:20 -!- leqock [~leqock@46.49.116.215] has quit [Quit: Leaving] 02:21 -!- gribble [~gribble@bitcoin/bot/gribble] has quit [Remote host closed the connection] 02:27 -!- gribble [~gribble@bitcoin/bot/gribble] has joined #bitcoin-core-dev 02:27 -!- mode/#bitcoin-core-dev [+o gribble] by ChanServ 02:31 -!- vincenzopalazzo [~vincenzop@2001:470:69fc:105::a67] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- tutwidi[m] [~tutwidima@2001:470:69fc:105::ead] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- poucatreta[m] [~poucatret@2001:470:69fc:105::20ae] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- robertspigler [~robertspi@2001:470:69fc:105::2d53] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- prusnak[m] [~stickmatr@2001:470:69fc:105::98c] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- orionwl[m] [~orionwlx0@2001:470:69fc:105::80] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- devrandom [~devrandom@2001:470:69fc:105::d4d] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- dongcarl[m] [~dongcarlm@2001:470:69fc:105::82] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- kvaciral[m] [~kvaciralx@2001:470:69fc:105::17b] has quit [Quit: Bridge terminating on SIGTERM] 02:31 -!- mrjumper[m] [~mr-jumper@2001:470:69fc:105::7f1] has quit [Quit: Bridge terminating on SIGTERM] 02:32 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has joined #bitcoin-core-dev 02:33 -!- orionwl[m] [~orionwlx0@2001:470:69fc:105::80] has joined #bitcoin-core-dev 02:34 -!- devrandom [~devrandom@2001:470:69fc:105::d4d] has joined #bitcoin-core-dev 02:34 -!- vincenzopalazzo [~vincenzop@2001:470:69fc:105::a67] has joined #bitcoin-core-dev 02:34 -!- prusnak[m] [~stickmatr@2001:470:69fc:105::98c] has joined #bitcoin-core-dev 02:34 -!- robertspigler [~robertspi@2001:470:69fc:105::2d53] has joined #bitcoin-core-dev 02:34 -!- poucatreta[m] [~poucatret@2001:470:69fc:105::20ae] has joined #bitcoin-core-dev 02:34 -!- dongcarl[m] [~dongcarlm@2001:470:69fc:105::82] has joined #bitcoin-core-dev 02:34 -!- kvaciral[m] [~kvaciralx@2001:470:69fc:105::17b] has joined #bitcoin-core-dev 02:34 -!- tutwidi[m] [~tutwidima@2001:470:69fc:105::ead] has joined #bitcoin-core-dev 02:34 -!- mrjumper[m] [~mr-jumper@2001:470:69fc:105::7f1] has joined #bitcoin-core-dev 02:36 -!- gleb [~gleb@178.150.137.228] has quit [Quit: Ping timeout (120 seconds)] 02:37 -!- gleb [~gleb@178.150.137.228] has joined #bitcoin-core-dev 02:40 < lightlike> vasild: We signal in the version message whether we want txRelay. I believe if a peer still send a tx or a tx inv in violation, they will be disconnected. 02:40 < lightlike> Addrs will be ignored on block-relay-only connections but processed in -blocksonly mode (our peer cannot easily distinguish between these two), no disconnections there 02:41 < vasild> hmm 02:46 -!- Jaamg [jaamg@kapsi.fi] has joined #bitcoin-core-dev 02:47 -!- kabaum [~kabaum@host-78-77-216-135.mobileonline.telia.com] has quit [Ping timeout: 244 seconds] 02:56 -!- cdecker [~cdecker@243.86.254.84.ftth.as8758.net] has quit [Ping timeout: 244 seconds] 02:58 -!- cdecker [~cdecker@243.86.254.84.ftth.as8758.net] has joined #bitcoin-core-dev 03:04 < provoostenator> I'd like to nominate #21934 for v22 because monitoring signalling in the next few months is quite useful. 03:04 <@gribble> https://github.com/bitcoin/bitcoin/issues/21934 | RPC/blockchain: getblockchaininfo: Include versionbits signalling details during LOCKED_IN by luke-jr · Pull Request #21934 · bitcoin/bitcoin · GitHub 03:08 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 03:08 < bitcoin-git> [bitcoin] hebasto closed pull request #22240: build: Re-enable -Wdeprecated-copy (master...210614-revert) https://github.com/bitcoin/bitcoin/pull/22240 03:08 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 03:20 -!- Guest75 [~Guest75@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has joined #bitcoin-core-dev 03:20 -!- Guest75 [~Guest75@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has quit [Client Quit] 03:21 -!- Guest8 [~Guest8@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has joined #bitcoin-core-dev 03:21 -!- Guest8 [~Guest8@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has quit [Client Quit] 03:21 -!- Guest98 [~Guest98@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has joined #bitcoin-core-dev 03:21 -!- Guest98 [~Guest98@2409:4063:6e8f:1d37:f905:caef:ac29:240c] has quit [Client Quit] 03:23 < laanwj> vasild: yes i remember during the bip155 discussion there were some ideas to also incorporate 'do not send me addresses at all', but this was not done, i had to think of this with #22245 03:23 <@gribble> https://github.com/bitcoin/bitcoin/issues/22245 | [p2p] Stop sending SENDADDRV2 message to block-relay-only peers by amitiuttarwar · Pull Request #22245 · bitcoin/bitcoin · GitHub 03:24 < vasild> I am composing a rely to that PR with a link to that discussion... 03:24 < laanwj> i don't think there is any way to signal to not want addresses so i'm not sure why addrv1/addrv2 matters 03:24 < laanwj> not even from a bandwidth perspective: addrv2 might be *more* efficient to ignore 03:25 < laanwj> e.g. ipv4 addresses (the most common) take fewer bytes 03:28 -!- whatsupboy [~whatsupbo@user/scobydoo] has joined #bitcoin-core-dev 03:36 < jnewbery_> laanwj: bandwidth is irrelevant. Nodes only send one addr message per 24 hours on average, and the max size is 1000 entries. 03:36 -!- arnabsen [~arnabsen@45.116.190.221] has joined #bitcoin-core-dev 03:37 -!- jnewbery_ is now known as jnewbery 03:37 < laanwj> right, it's the only possible concern i could think of anyhow 03:43 -!- Guest73 [~Guest73@2402:f000:2:d001:912c:3873:8807:5b1f] has joined #bitcoin-core-dev 03:45 -!- Guest73 [~Guest73@2402:f000:2:d001:912c:3873:8807:5b1f] has quit [Client Quit] 03:52 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has quit [Ping timeout: 272 seconds] 03:53 -!- gleb [~gleb@178.150.137.228] has quit [Ping timeout: 268 seconds] 03:54 -!- arnabsen [~arnabsen@45.116.190.221] has quit [Ping timeout: 250 seconds] 04:05 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 04:05 < bitcoin-git> [bitcoin] MarcoFalke opened pull request #22258: build: Disable deprecated-copy warning only when external warnings are enabled (master...2106-buildEnableWarnDeprecatedCopy) https://github.com/bitcoin/bitcoin/pull/22258 04:05 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 04:08 -!- bitdex [~bitdex@gateway/tor-sasl/bitdex] has quit [Quit: = ""] 04:18 -!- whatsupboy [~whatsupbo@user/scobydoo] has quit [Quit: WeeChat 2.8] 04:22 -!- mihir [~mihir@103.216.176.46] has joined #bitcoin-core-dev 04:23 -!- mihir [~mihir@103.216.176.46] has quit [Client Quit] 04:24 < jonatack> the disable_tx discussions left me with the recollection that, rather than encapsulating complex state in connection types, we might be heading back to separate flags, e.g. disable tx, disable addr, etc. 04:25 < jonatack> in the BIP155 discussions nearly a year ago, it was decided that disable addr was a separate concern from BIP155-capable, thus the current implementation 04:26 -!- sagi [~sagi@176.230.171.92] has quit [Ping timeout: 244 seconds] 04:28 < jonatack> those do seem like separate concerns / flags / BIPs etc 04:28 < jonatack> messages 04:30 < vasild> +1 04:31 < vasild> I opened https://github.com/bitcoin/bips/pull/1134 to clarify BIP155 04:32 < jonatack> vasild: nice! 04:34 < vasild> I never thought that "send me addrv2 instead of addr" may imply a preference to receive address messages (them being addrv2 or addr). My Enflisch is nae good. 04:35 < vasild> jnewbery: ^ 04:35 < jonatack> vasild: just saw your review, agree 04:40 < jonatack> (communication may have been a bit siloed on some topics, by habit / time zones / affinities etc) 04:40 < jonatack> (reviewers of certain areas, and so on) 04:46 < laanwj> vasild: me neither 04:46 < laanwj> it wasn't my intent at least when writing it 04:47 < laanwj> i really meant it as a specification of what is supported by the client (remember that initially i started with a protocol version number bump only, the message came later because that turned out to be the preferred signaling mechanism now) 04:47 < laanwj> then the protocol version bump was added again because other implementations 04:50 < laanwj> i never once considered it a preference with regard to receiving messages 05:13 < jnewbery> vasild: why would I say "send me red apples instead of green apples" when I don't eat apples at all? 05:13 < jnewbery> And if you send me an apple I'm going to put it straight in the trash 05:13 < laanwj> that's just how the BIP was intended, it could just as well have been a service flag which is always sent 05:14 < laanwj> as I said above, the fact that it's an extra message is an implementation detail 05:15 < jnewbery> I'm very confused about why it's controversial to not send a message that has zero impact on our own processing 05:15 < laanwj> there is no message to say "I don't want to receive address messages" 05:15 < vasild> jnewbery: bad English, sorry 05:15 < jnewbery> Why would I send a message to say "please send me addrv2 messages that I'll throw away instead of sending addr messages that I'll throw away"? 05:15 < laanwj> because you will still receive v1 messages that you'll throw away 05:15 < laanwj> it doesn't matter 05:15 < jnewbery> so why send the message? 05:15 < laanwj> but anyhow, ok, I don't understand why this is such a topic of contention teither 05:16 < laanwj> never minds 05:17 < vasild> natural language is vague, we should be talking with "if ()", ">" and "==" ;-) 05:17 < laanwj> I did think a signal whether a peer is interested in address messages or not is orthogonal to what messages it supports or not, there was discussion to include this in BIP155 at some point but it wasnm't 05:18 < laanwj> I do think such a signal is useful fwiw 05:18 < laanwj> but I don't see how sendaddrv2 is helpful here 05:18 < jnewbery> laanwj: but that's not what the PR does? 05:18 < jnewbery> it doesn't prevent message relay 05:18 < laanwj> I understand that 05:18 < jonatack> there was a real discussion about it, no need to rehash a year later 05:18 < laanwj> if it prevented message relay it'd be a much more useful change 05:19 < jnewbery> laanwj: I'm confused. I thought that's what you're arguing against 05:19 < laanwj> "I don't want address messages from block-only peers" fine 05:19 < laanwj> no, I'm arguing against misinterpreting BIP155 05:20 < laanwj> https://github.com/bitcoin/bips/pull/1134 clears it up for me 05:20 < jnewbery> but this isn't misinterpreting BIP155. BIP155 is totally irrelevant for links that don't support address relay 05:21 < laanwj> but was you say this doesnt disable address relay 05:21 < laanwj> address relay isn't disabled for the connection 05:21 < jnewbery> laanwj: what happens when I open an outbound blocks-only connection and receive an addr message on that connection? 05:22 < vasild> jnewbery: there is more to this - a peer may maintain a useful database of addresses but not participate in address relay, so he is interested in receiving address gossip but is a black hole wrt further address propagation 05:22 < vasild> jnewbery: it is ignored 05:23 < jnewbery> if it's ignored then how can "he's interested in receiving address gossip" be true? 05:23 < vasild> the peer I mentioned is not blocks-only 05:23 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has joined #bitcoin-core-dev 05:24 < vasild> may be a light client 05:24 < jnewbery> so how's that relevant to PR 22245? 05:24 < vasild> he is interested in receiving address gossip in order to maintain his own addresses databas 05:24 < jnewbery> 22245 is only for blocks-relay-only connections 05:25 < jnewbery> talking about light clients seems completely irrelevant to the PR 05:25 < vasild> 22245 implies we put a new semantic to sendaddrv2: "However, if we move forward with the approached proposed in that PR" 05:25 < vasild> that PR == #21528 05:25 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 05:26 < jnewbery> 22245 doesn't change the semantics of sendaddrv2 at all. It simply doesn't send the message when it's irrelevant 05:26 < lightlike> jnewbery: i think we expect to receive addr messages on block-relay-only connections currently, because out peer doesn't know from the version messages whether this is a block-relay-only link or a regular connection in -blocksonly mode (for which we want addrs) 05:27 < jnewbery> lightlike: the peer doesn't even know that the connection is not a full-relay-connection 05:28 < jnewbery> vasild: I don't think your 4 state analysis in https://github.com/bitcoin/bitcoin/pull/22245#issuecomment-862279099 is quite right. When you say: 05:28 < jnewbery> > !A and !B (it follows that if you send me a reply to my getaddr it should be in the old addr format because I don't support the new addrv2) 05:29 < vasild> jnewbery: #21528 changes the semantic of sendaddrv2 and #22245 only makes sense if #21528 is accepted 05:29 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 05:29 <@gribble> https://github.com/bitcoin/bitcoin/issues/22245 | [p2p] Stop sending SENDADDRV2 message to block-relay-only peers by amitiuttarwar · Pull Request #22245 · bitcoin/bitcoin · GitHub 05:29 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 05:29 < jnewbery> that doesn't make sense here. If we receive a reply to a getaddr on an outbound block-relay-only connection, then we drop it. 05:29 < jnewbery> 22245 seems like a good change independently from 21528 to me 05:30 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has joined #bitcoin-core-dev 05:30 < lightlike> vasild: I don't agree with that. 22245 should make just as much sense using just ADDR,ADDRV2 and GETADDR as signs that the node is interested in addr relay. SENDADDRV2 doesn't seem necessary to me 05:30 < jonatack> jnewbery: it indeed would widen the meaning of sendaddrv2, and irreversibly 05:31 < jnewbery> jonatack: I still don't understand why you keep saying that 05:32 < jnewbery> if I don't do anything with addrs received on a link, then asking my peer to send those addrs (which I'm going to ignore) in a certain format, is neither useful, nor irreversible 05:33 -!- gleb [~gleb@178.150.137.228] has joined #bitcoin-core-dev 05:33 < jnewbery> can you help me understand why you think this change is irreverible? 05:39 < jnewbery> lightlike: I think you meant "21528 should make just as much sense using just addr, addrv2 and getaddr" (and I agree) 05:43 < vasild> If we look at 22245 in isolation (as if 21528 does not exist) then it is pointless change. Why make it? The peer is then going to send us addr which we ignore (instead of addrv2 which we ignore). Also we would be lying that we don't support addrv2 when we actually do. Just does not feel right. 05:45 < vasild> But I think the bigger argument here is 22245+21528 and changing the semantic of sendaddrv2 in order to tweak address relay and attempt to fix the black holes problem. But would even that fix it? Some scenarios where it will not: https://github.com/bitcoin/bitcoin/pull/21528#issuecomment-862312851 05:46 < jnewbery> `sendaddrv2` only has meaning within a single connection. It means "I prefer to receive addrv2 messages on this connection". It implies nothing about the implementation of the node (as it shouldn't). Not sending a sendaddrv2 message is not "lying that we don't support addrv2", but simply not expressing a preference for a certain format *on that connection*. 05:47 < jnewbery> If this was a service bit, which is gossiped beyond a single connection, then I'd agree with you, but all p2p messages only have semantic meaning within the connection that they exist in 05:47 < laanwj> the point is that there is no way to "not express a preference" 05:47 < laanwj> either the preference is v1 (not message) or v2 (send a message) 05:47 < jnewbery> the way to "not express a preference" is to not "express a preference" 05:48 < laanwj> there could have been a third option in BIP155 but there wasn't 05:48 < vasild> It means "I prefer to receive addrv2 messages on this connection" -- this is your interpretation. 05:48 < laanwj> no, it's not 05:48 < laanwj> that would mean that a peer that doesn't send the message doesn't fcare what it receives 05:48 < laanwj> whereas it means that it wants to receive v1 05:49 < laanwj> there is no "I don't want to receive addr" signal, nor is there "I don't care" 05:49 < laanwj> maybe there should be! but there isn't 05:49 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has joined #bitcoin-core-dev 05:49 < jnewbery> vasild: the BIP says "Sending such a message indicates that a node can understand and *prefers to receive addrv2 messages*" - yes, my interpretation of this is that it prefers to receive addrv2 messages 05:50 < vasild> INSTEAD 05:50 < laanwj> signaling the new protocol version then *deciding* not to send the message means your node wants to receive v1 05:50 < jnewbery> laanwj: that's not how p2p versions work 05:50 < laanwj> well it is how I intended BIP155, I don't really agree with the new interpretation 05:50 < jnewbery> since they're serial, if a later p2p version adds some other feature, it doesn't automatically mean that anyone using that feature *must* implement addrv2 05:51 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has quit [Ping timeout: 268 seconds] 05:51 < vasild> jnewbery: you skipped the trailing "instead..." which is important. 05:52 < jnewbery> I didn't think I was misrepresenting it. Here you go: "Sending such a message indicates that a node can understand and prefers to receive addrv2 messages instead of addr messages. I.e. "Send me addrv2"." 05:52 < laanwj> jnewbery: which is why there is the option not to send the message: it implies you want to keep reciving v1 05:52 < laanwj> there are two states, not three or four 05:52 < vasild> anyway, if sendaddrv2 signalled preference to receive unrequested address messages, then bitcoin core-pre-bip155 do not want to receive unrequested address messages? 05:53 < jnewbery> laanwj: A BIP cannot specify that not sending something implies some meaning. BIPs are opt-in. I can't say in my BIP "not sending this message implies thing". 05:53 < vasild> because they do not send sendaddrv2 05:53 < laanwj> jnewbery: in this case not sending the message is the old behavior of 'send me v1' 05:53 < jnewbery> vasild: huh? I think you really misunderstand 22245 and 21528. 05:53 < laanwj> there's the old behavior anhd the new behavior 05:54 < jnewbery> a pre-bip155 bitcoin core node will send a getaddr, which implies that it wants to receive addresses 05:54 < laanwj> but I honestly don't understand why this is such a hotbed issue before the v22.0 feature freeze 05:55 < jnewbery> it will also send addr messages, which implies that it's taking part in address relay 05:55 < laanwj> there would have been tons of time to discuss this with less time pressure 05:57 < vasild> jnewbery: would you say that if a node sends getaddr it follows that it wants to also receive unrequested address messages, outside of the response of that getaddr? And from that would you say it follows that this node participates in address relay (gossips to other nodes)? 05:57 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has quit [Ping timeout: 240 seconds] 05:58 < laanwj> I wasn't aware either that sending "getaddr" changed the connection status in that regard 05:59 -!- vnogueira [~vnogueira@user/vnogueira] has quit [Remote host closed the connection] 05:59 < vasild> laanwj: my understanding is that 21528 intends to achieve that (sending getaddr changed the connection status...) 05:59 -!- vnogueira [~vnogueira@user/vnogueira] has joined #bitcoin-core-dev 05:59 < laanwj> but if you can regard "getaddr" as a signaling message to receive unrequested address messages, and a peer never sending it effectively signals it doesn't ever want to see address messages, then you're right, there are three states 06:00 < jnewbery> laanwj: that's what #21528 is proposing. If a peer sends us `addr` OR `addrv2` OR `getaddr` OR `sendaddrv2`, then we should consider it a peer for address gossiping. I think it makes sense to review that PR. 06:00 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 06:00 -!- promag_ is now known as promag 06:00 < jnewbery> The PR description also lists all of the places that the proposal has been discussed previously 06:00 < laanwj> jnewbery: if that is new behavior it needs a BIP as well 06:01 < laanwj> doesn't seem limited to bitcoin core 06:01 < jnewbery> laanwj: please read the PR before making statements like that 06:01 < jnewbery> here's the mailing list post: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2021-April/018784.html 06:01 < laanwj> you're being really agressive now 06:01 < jnewbery> I don't think it's aggressive to expect people to read PR descriptions 06:02 < laanwj> I'm not aware of every single PR description 06:02 < laanwj> I just cannot keep up wit hthat 06:03 < jnewbery> laanwj: this has been discussed in many venues already. It's been raised on the mailing list, discussed in bitcoin core irc and p2p irc meetings, amiti has even done a survey of every other common node implementation to make sure it doesn't break them: https://github.com/bitcoin/bitcoin/pull/21528#issuecomment-809906430 06:03 < vasild> hmm, actually laanwj is right that 21528 is not related to just bitcoin core - it changes the semantics of getaddr, addr, addrv2 and sendaddrv2. I guess that warrants a BIP. 06:04 < laanwj> jnewbery: that's good 06:05 < laanwj> vasild: yes, likely instead of your proposed BIP155 change 06:06 < laanwj> (as it goes the other way) 06:06 < laanwj> which is fine, if you're trying to change something, just be transparent about it, a while before you were still claiming this doesn't make any irreversible change to behavior 06:07 < jnewbery> laanwj: are you saying that I'm not being transparent? 06:07 -!- evias_ [~evias__@196.red-88-6-131.staticip.rima-tde.net] has joined #bitcoin-core-dev 06:07 < jnewbery> 22245 makes no irreversible changes to behavior 06:07 < laanwj> it came across to me as "this was always the case" 06:08 < jnewbery> all of this has been discussed many times on github, irc, mailing list. How could it be more transparent? 06:08 < laanwj> maybe it's just the nthat taking that part out of the parent PR separetely was unclear to people 06:08 < vasild> Today is the first time I hear about it, sorry. 06:09 < laanwj> yeah 06:09 < vasild> it == 21528+22245 06:10 < jnewbery> laanwj: I suggested to amiti that 22245 be separated from 21528, so I apologize if that was confusing for people and take full responsibility 06:10 -!- evias [~evias__@user/evias] has quit [Ping timeout: 268 seconds] 06:10 < jnewbery> it seems to me like a good change independent from 21528 06:10 < vasild> IMO 21528+22245 is a protocol change. 06:10 < jnewbery> vasild: it's not a protocol change 06:11 < vasild> the semantic of e.g. getaddr is changed (extended) 06:11 < laanwj> by itself (without other context) it seemed like a misinterpretation of BIP155 as i had intended it 06:11 < laanwj> seeing it as part of a large change is completely different 06:12 < jnewbery> laanwj: the first sentence in the PR description links to the 21528. I don't understand how it's not transparent 06:13 < laanwj> so okay, you *are* trying to amke a protocol change 06:13 < jnewbery> what? 06:13 < jnewbery> can you please stop the insinuations that I'm trying to do something underhand? 06:14 < laanwj> what, I dont' mean you are trying to make a sneaky protocol change 06:14 < laanwj> I haven't said anything about being underhand 06:14 < laanwj> I just didn't understand 06:14 < jnewbery> you said that I'm not being transparent 06:15 -!- laanwj [~laanwj@user/laanwj] has quit [Quit: WeeChat 3.1] 06:17 < jamesob> seems clear that there's enough unresolved discussion here that merging these changes in before the feature freeze isn't the way to go. and that conversations like these would be much more amicable IRL! 06:18 < jnewbery> laanwj: I'd suggest re-reading the irc meeting logs from when this was discussed, both in the main Bitcoin Core meeting, and in the p2p meeting. You commented in the first meeting in March, so you were aware that this was a proposal. 06:21 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has quit [Ping timeout: 252 seconds] 06:23 < jnewbery> It's very frustrating to see someone put in months of work on something, raise it in both irc meetings, share it on the mailing list, do a huge amount of work to verify compatibility with other nodes, and then see people show up and leave opinionated comments without reading and understanding the full context. 06:24 -!- nfr [~nfr@2001:1c00:31c:5500:f00e:3d5e:2b21:31c1] has joined #bitcoin-core-dev 06:25 -!- nfr [~nfr@2001:1c00:31c:5500:f00e:3d5e:2b21:31c1] has left #bitcoin-core-dev [] 06:26 < jamesob> jnewbery: no doubt man, but to some extent that's just the nature of the beast re: the project's decentralized development process. I haven't seen mention of an irreversible widening of the protocol's semantics earlier than jonatack's comment 8 hours ago, and I think once something like that is raised it deserves to be fleshed out over some time 06:30 < jnewbery> I'm still waiting for jonatack to explain why this is an "irreversible change" 06:31 < michaelfolkson> It is regrettable but if laanwj vasild jonatack are all uncomfortable with it getting in before feature freeze then we should probably move on. Lots of other things to discuss for feature freeze 06:32 -!- Guest9766574 [~Guest9766@ip5f5bf52b.dynamic.kabel-deutschland.de] has joined #bitcoin-core-dev 06:36 -!- Guest9766574 [~Guest9766@ip5f5bf52b.dynamic.kabel-deutschland.de] has quit [Client Quit] 06:39 -!- sibilant_ [~sibilant@194.59.250.58] has joined #bitcoin-core-dev 06:39 -!- sibilant_ [~sibilant@194.59.250.58] has quit [Changing host] 06:39 -!- sibilant_ [~sibilant@user/sibilant] has joined #bitcoin-core-dev 06:41 < _aj_> jnewbery: you said that I'm not being transparent // -!- laanwj [~laanwj@user/laanwj] has quit 06:42 -!- sibilant [~sibilant@user/sibilant] has quit [Ping timeout: 268 seconds] 06:44 -!- goatpig [~goat@blocksettle-gw.cust.31173.se] has quit [Quit: Konversation terminated!] 06:48 -!- evias_ [~evias__@196.red-88-6-131.staticip.rima-tde.net] has quit [Quit: This computer has gone to sleep] 06:55 -!- lightlike [~lightlike@user/lightlike] has quit [Remote host closed the connection] 06:56 -!- lightlike [~lightlike@p200300c7ef121e0069fa21b1771e86de.dip0.t-ipconnect.de] has joined #bitcoin-core-dev 06:56 -!- lightlike [~lightlike@p200300c7ef121e0069fa21b1771e86de.dip0.t-ipconnect.de] has quit [Changing host] 06:56 -!- lightlike [~lightlike@user/lightlike] has joined #bitcoin-core-dev 07:04 -!- Henry151 [~bishop@user/henry151] has quit [Quit: leaving] 07:05 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has joined #bitcoin-core-dev 07:06 < ariard> notwithstanding making the discussion far less heated for the sake of everyone, i agree that's a sender-only change and i don't see how it would restrain a future bip155 client to probe addrv2 support with its selected v22.0+ peers 07:06 < ariard> that's the bip is unclear and not interest in addr format doesn't signal lack of interest in addr-relay sounds a different issue there, imho 07:09 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 07:10 -!- Henry151 [~bishop@user/henry151] has quit [Client Quit] 07:11 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 07:21 -!- jonatack [jonatack@user/jonatack] has quit [Quit: Connection closed] 07:22 -!- goatpig [~goat@h-94-254-2-155.A498.priv.bahnhof.se] has joined #bitcoin-core-dev 07:24 -!- leafy-greens [~leafy-gre@modemcable123.58-83-70.mc.videotron.ca] has joined #bitcoin-core-dev 07:26 -!- evias [~evias__@user/evias] has joined #bitcoin-core-dev 07:27 -!- sibilant_ [~sibilant@user/sibilant] has quit [Ping timeout: 268 seconds] 07:40 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 07:43 -!- stevenroose [~steven@2001:19f0:6801:83a:5b0:2160:99b2:6c0e] has quit [Quit: ZNC 1.7.4 - https://znc.in] 07:43 -!- stevenroose [~steven@2001:19f0:6801:83a:ec57:5e94:994a:afff] has joined #bitcoin-core-dev 07:51 -!- Henry151 [~bishop@user/henry151] has quit [Quit: leaving] 07:52 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 07:53 -!- Henry151 [~bishop@user/henry151] has quit [Client Quit] 07:55 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 07:59 -!- Henry151 [~bishop@user/henry151] has quit [Client Quit] 07:59 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 07:59 -!- Henry151 [~bishop@user/henry151] has quit [Client Quit] 08:00 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has joined #bitcoin-core-dev 08:03 -!- Henry151 [~bishop@user/henry151] has joined #bitcoin-core-dev 08:05 -!- jonatack [jonatack@user/jonatack] has joined #bitcoin-core-dev 08:09 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 08:09 < bitcoin-git> [bitcoin] Sjors opened pull request #22260: Make bech32m the default, except where needed. Update GUI checkbox. (master...2021/06/bech32_gui) https://github.com/bitcoin/bitcoin/pull/22260 08:09 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 08:23 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has quit [Ping timeout: 244 seconds] 08:38 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has joined #bitcoin-core-dev 08:47 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has joined #bitcoin-core-dev 08:53 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has quit [] 08:54 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has joined #bitcoin-core-dev 08:54 -!- lkqwejhhgasdjhgn [~kljkljklk@p200300d46f03bc00adf9c5ad604010d2.dip0.t-ipconnect.de] has quit [Quit: Konversation terminated!] 08:56 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has quit [Client Quit] 08:57 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has joined #bitcoin-core-dev 08:59 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 08:59 < bitcoin-git> [bitcoin] jnewbery opened pull request #22261: [p2p/mempool] Two small fixes to node broadcast logic (master...2021-06-broadcast-fixes) https://github.com/bitcoin/bitcoin/pull/22261 08:59 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 08:59 -!- dunxen [dunxen@gateway/vpn/protonvpn/dunxen] has joined #bitcoin-core-dev 09:01 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has quit [Client Quit] 09:02 -!- Evel-Knievel [~Evel-Knie@d5152f744.static.telenet.be] has joined #bitcoin-core-dev 09:18 -!- sagi [~sagi@bzq-79-180-140-8.red.bezeqint.net] has quit [Ping timeout: 272 seconds] 09:19 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 09:19 < bitcoin-git> [bitcoin] dongcarl closed pull request #20158: tree-wide: De-globalize ChainstateManager (master...2020-06-libbitcoinruntime) https://github.com/bitcoin/bitcoin/pull/20158 09:20 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 09:23 -!- prakash [~prakash@58.182.42.155] has joined #bitcoin-core-dev 09:35 -!- lightlike [~lightlike@user/lightlike] has quit [Quit: Leaving] 09:39 -!- lightlike [~lightlike@user/lightlike] has joined #bitcoin-core-dev 09:52 -!- WS_black22 [~WS_black2@176.67.86.204] has joined #bitcoin-core-dev 09:52 < WS_black22> hello folks 09:53 < WS_black22> i got my old wallet.dat i need to imported to the new bitcoin core, how can i do this? i dont see the replasment of wallet.dat, 09:53 -!- Guest69 [~Guest69@218.212.21.21] has quit [Quit: Client closed] 09:54 -!- WS_black22 [~WS_black2@176.67.86.204] has quit [Quit: IRC sucks ... IRC should Die! all sick!] 09:57 -!- Talkless [~Talkless@mail.dargis.net] has joined #bitcoin-core-dev 10:03 -!- mekster [~mekster@li1564-239.members.linode.com] has quit [Quit: mekster] 10:03 -!- mekster [~mekster@li1564-239.members.linode.com] has joined #bitcoin-core-dev 10:06 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has quit [Ping timeout: 272 seconds] 10:09 < amiti> hey all, looks like I missed lots of convo about my work to mitigate-addr-blackholes. I've caught up on the conversation here & the PRs and want to share two main thoughts: 10:10 < amiti> 1. My biggest question is why are these approach concerns only being raised now? 10:10 < amiti> To review the context of this work: I opened #21528 almost 3 months ago, and soon after brought it up at the weekly bitcoin-core-dev meeting to seek approach feedback. A lot of the concerns that are now being voiced (should there be a separate flag, could we do redundant relay for blackholes rather than selective, what are the implications for other clients) have already been discussed in relation to these changes. My 10:10 < amiti> understanding was the biggest concern was about compatibility for other clients, so I wrote to the mailing list and researched / opened issues in every other bitcoin client I could find. I’ve additionally brought up these changes at a P2P meeting in early April and then again this week. I understand that time zones are hard and that not everyone can attend the meetings, but I’d hope the logs would be read and concerns to 10:10 < amiti> be raised in the week that follows. 10:10 < amiti> It’s pretty disheartening to spend a long time trying to clear the conceptual obstacles for this work, to then loop back and rehash the same conversations. So this brings me back to the question of why are these approach concerns only being raised now? I’m genuinely perplexed because between the PR, irc meetings & the mailing list, I thought I was very vocal about these changes. If our only takeaway is how we avoid these 10:10 < amiti> sort of drawn-out circular conversations, at least this will have been a constructive experience. 10:10 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 10:10 < amiti> 2. In regards to the path forward for #21528 & #22245, it seems like the concerns are all focusing specifically on SENDADDRV2 and the wording of that specific bip. #21528 can proceed without using SENDADDRV2 (aka without #22245), but imo it would make more sense to treat it consistently with ADDR, GETADDR and ADDRV2 messages. While I do think the change is a standalone improvement, the end goal is to support more 10:10 < amiti> block-relay-only connections (and address the concerns I had about the disabletx proposal). In the end, if the only remaining concern is really around SENDADDRV2, I will remove that piece. But I would prefer to keep it logically consistent with the rest of the proposed changes. 10:10 < amiti> Its clear that network-wide addr relay is something filled with murky assumptions, and divergent expectations, but I’m hoping to find a way forward for these goals. 10:10 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 10:10 <@gribble> https://github.com/bitcoin/bitcoin/issues/22245 | [p2p] Stop sending SENDADDRV2 message to block-relay-only peers by amitiuttarwar · Pull Request #22245 · bitcoin/bitcoin · GitHub 10:10 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 10:10 <@gribble> https://github.com/bitcoin/bitcoin/issues/22245 | [p2p] Stop sending SENDADDRV2 message to block-relay-only peers by amitiuttarwar · Pull Request #22245 · bitcoin/bitcoin · GitHub 10:10 < amiti> laanwj, vasild, jonatack: I'm interested in hearing your opinions on these two points. thanks in advance! 10:10 < amiti> also sorry for all the gribbles =P 10:13 -!- martinus__ [~martinus@212095005005.public.telering.at] has quit [Ping timeout: 268 seconds] 10:15 -!- mekster [~mekster@li1564-239.members.linode.com] has quit [Quit: mekster] 10:16 -!- mekster [~mekster@li1564-239.members.linode.com] has joined #bitcoin-core-dev 10:19 -!- mekster [~mekster@li1564-239.members.linode.com] has quit [Changing host] 10:19 -!- mekster [~mekster@user/mekster] has joined #bitcoin-core-dev 10:19 < achow101> amiti: from a cursory review of all of the discussion on this topic that I could find, the issue is with the specific implementation of #22245, which, afaict, is not discussed anywhere except when that PR was opened. I get that it was broken out of #21528 but I don't see where that specific change is discussed. 10:19 <@gribble> https://github.com/bitcoin/bitcoin/issues/22245 | [p2p] Stop sending SENDADDRV2 message to block-relay-only peers by amitiuttarwar · Pull Request #22245 · bitcoin/bitcoin · GitHub 10:20 <@gribble> https://github.com/bitcoin/bitcoin/issues/21528 | [p2p] Reduce addr blackholes by amitiuttarwar · Pull Request #21528 · bitcoin/bitcoin · GitHub 10:20 < sipa> i don't think the sendaddrv2 aspect of the discussion is relevant 10:21 < sipa> (only to the extent that it drew attention to the fact that 21528 is changing behavior for a number of messages including sendaddrv2) 10:21 < achow101> it seems to me that the concerns are with 22245 itself, not with the concept of avoiding addr blackholes 10:21 < sipa> 22245 is utterly harmless on itself 10:22 < sipa> sending sendaddrv2 or not makes no difference, as the received addr messages are just ignored 10:28 < achow101> sipa: but with 21528 it gains an additional meaning? 10:29 < sipa> achow101: not in my opinion, but yes, i think that is the disagreement 10:29 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 10:30 < amiti> this is kinda what I'm trying to clarify- are the concerns solely about the sendaddrv2 message, or are they about the general approach 10:31 < amiti> but yeah, I conceptually agree with what sipa just voiced 10:35 < sipa> amiti: actually, question i have: if 21528 would exclude sendaddrv2 (just not use it as a trigger when received), what would happen for relay to block only peers? 10:38 < amiti> so, if 21528 only used ADDR, ADDRV2, GETADDR to indicate "interest in addr relay", then bitcoin core nodes wouldn't initiate any of those to outbound block-relay-only connections. but nodes who have started up in blocks-only mode would initiate a GETADDR to their outbound peers. 10:38 < amiti> and participate in other ADDR relay 10:42 < sipa> amiti: so what difference would that make? 10:44 < amiti> sipa: right, so I don't quite get why sendaddrv2 would / should be treated differently, and have a preference for logical consistency across p2p messages. BUT if the concern is *exclusively* about sendaddrv2 messages & not matching the intent when writing the bip, I can just leave it out and continue forward. 10:45 < sipa> amiti: yeah, i'm just trying to understand the tradeoffs 10:45 < sipa> not suggesting anything specifically 10:47 < amiti> gotcha, imo the main tradeoff of using sendaddrv2 or not is just "logical consistency" of treating it the same as other address messages. no observable functional difference based on current clients 10:47 < amiti> also in the research of other clients, I wasn't specifically looking, but didn't see much support for sendaddrv2, so suspect it would have minimal impact there too. 10:47 < sipa> got it, thanks 10:50 < amiti> :) 10:52 < achow101> is 22245 required for 21528 or just "makes things consistent"? If the latter, I would suggest just dropping/skipping it for now. 10:52 < achow101> it seems to me it doesn't really get in the way 10:53 -!- dunxen [dunxen@gateway/vpn/protonvpn/dunxen] has quit [Remote host closed the connection] 10:53 -!- dunxen [dunxen@gateway/vpn/protonvpn/dunxen] has joined #bitcoin-core-dev 10:54 -!- kabaum [~kabaum@host-78-77-216-135.mobileonline.telia.com] has joined #bitcoin-core-dev 10:54 < sipa> i think i'm leaning in that direction too; trying to write my thoughts a bit more structured on the PR 11:06 -!- belcher_ [~belcher@user/belcher] has joined #bitcoin-core-dev 11:06 < amiti> yup, not my preference but totally viable. if we're going that route, I'd like to better understand the reasoning around treating sendaddrv2 differently, and then can update 21528 & close 22245. 11:07 -!- belcher [~belcher@user/belcher] has quit [Ping timeout: 244 seconds] 11:09 -!- hiii [~hiii@171.78.176.155] has joined #bitcoin-core-dev 11:11 -!- belcher_ is now known as belcher 11:18 < hiii> Hi, is this the main bitcoin dev channel? 11:19 < michaelfolkson> hiii: "Bitcoin Core development discussion and commit log" 11:26 < hiii> thanks 11:26 < hiii> where can I ask about summerofbitcoin? sorry if this is off topic 11:27 < michaelfolkson> hiii: #bitcoin 11:29 < hiii> michaelfolkson: thank you so much, i asked here since it was a btc core developer program, apologies 11:31 -!- dunxen [dunxen@gateway/vpn/protonvpn/dunxen] has quit [Quit: Leaving...] 11:36 -!- sjaustirni [~sjaustirn@user/sjaustirni] has joined #bitcoin-core-dev 11:37 -!- sjaustirni [~sjaustirn@user/sjaustirni] has left #bitcoin-core-dev [] 11:48 -!- zndtoshi [~zndtoshi@79.112.74.9] has joined #bitcoin-core-dev 11:49 < zndtoshi> I posted the answers (randomized order) for the devs that were kind enough to give their feedback on the Taproot update and future expectations. Thank you! https://twitter.com/zndtoshi/status/1405235804549566464?s=20 11:53 -!- evias [~evias__@user/evias] has quit [Quit: This computer has gone to sleep] 12:06 -!- kabaum [~kabaum@host-78-77-216-135.mobileonline.telia.com] has quit [Ping timeout: 272 seconds] 12:13 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 12:13 < bitcoin-git> [bitcoin] hebasto reopened pull request #19882: depends: Export variables from make to environment explicitly (master...200905-build) https://github.com/bitcoin/bitcoin/pull/19882 12:13 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 12:19 -!- zestymug [uid446484@id-446484.charlton.irccloud.com] has joined #bitcoin-core-dev 12:21 -!- zndtoshi [~zndtoshi@79.112.74.9] has quit [Ping timeout: 250 seconds] 12:26 -!- zndtoshi [~zndtoshi@79.112.74.9] has joined #bitcoin-core-dev 12:27 -!- logarus [~logarus@101.186.60.205] has joined #bitcoin-core-dev 12:28 -!- logarus2 [~logarus@2001:8003:4d47:a500:fc19:6bd2:ed16:1c0d] has quit [Ping timeout: 256 seconds] 12:29 -!- lukedashjr [~luke-jr@user/luke-jr] has joined #bitcoin-core-dev 12:30 -!- luke-jr [~luke-jr@user/luke-jr] has quit [Ping timeout: 244 seconds] 12:31 -!- lukedashjr is now known as luke-jr 12:33 -!- Talkless [~Talkless@mail.dargis.net] has quit [Quit: Konversation terminated!] 12:38 -!- zestymug is now known as zesty 12:39 -!- vasanth2[m] [~vasanth2m@2001:470:69fc:105::3548] has joined #bitcoin-core-dev 12:41 -!- kexkey [~kexkey@static-198-54-132-110.cust.tzulo.com] has joined #bitcoin-core-dev 12:49 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has quit [Ping timeout: 252 seconds] 12:57 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 12:57 -!- nanotube [~nanotube@user/nanotube] has quit [Ping timeout: 244 seconds] 12:58 -!- Guest54 [~Guest54@pool-108-53-136-237.nwrknj.fios.verizon.net] has joined #bitcoin-core-dev 13:09 -!- zndtoshi [~zndtoshi@79.112.74.9] has quit [Quit: Client closed] 13:19 -!- RebelOfBabylon [~RebelOfBa@host-67-204-200-148.public.eastlink.ca] has joined #bitcoin-core-dev 13:30 -!- RebelOfBabylon [~RebelOfBa@host-67-204-200-148.public.eastlink.ca] has quit [Quit: Connection closed] 13:51 -!- nanotube [~nanotube@user/nanotube] has joined #bitcoin-core-dev 14:19 -!- vnogueira [~vnogueira@user/vnogueira] has quit [Remote host closed the connection] 14:19 -!- vnogueira [~vnogueira@user/vnogueira] has joined #bitcoin-core-dev 14:30 -!- ratatosk9 [~ratatosk9@193.138.218.218] has joined #bitcoin-core-dev 14:38 -!- Guyver2 [~Guyver@guyver2.xs4all.nl] has quit [Remote host closed the connection] 14:41 -!- ratatosk9 [~ratatosk9@193.138.218.218] has quit [Quit: Konversation terminated!] 14:42 -!- ratatosk9 [~ratatosk9@193.138.218.218] has joined #bitcoin-core-dev 14:47 -!- ratatosk9 [~ratatosk9@193.138.218.218] has quit [Quit: Konversation terminated!] 14:47 -!- ratatosk9 [~ratatosk9@193.138.218.218] has joined #bitcoin-core-dev 14:47 -!- ratatosk9 [~ratatosk9@193.138.218.218] has quit [Client Quit] 14:48 -!- rejvons [~ratatosk9@193.138.218.218] has joined #bitcoin-core-dev 14:50 -!- rejvons is now known as ola 14:50 -!- ola is now known as rejvons 14:53 -!- rejvons is now known as ratatosk9 14:54 -!- ratatosk9 is now known as rejvons 14:58 -!- hiii [~hiii@171.78.176.155] has quit [Quit: Connection closed] 15:00 -!- rejvons [~ratatosk9@193.138.218.218] has quit [Quit: Konversation terminated!] 15:00 -!- ratatosk9 [~ratatosk9@193.138.218.218] has joined #bitcoin-core-dev 15:02 -!- gene_ [~gene@2a02:6f8:2020:214:100::100e] has joined #bitcoin-core-dev 15:03 -!- ratatosk9 is now known as rejvons 15:13 -!- smartin [~Icedove@88.135.18.171] has quit [Quit: smartin] 15:18 -!- rejvons is now known as ratatosk9 15:22 -!- Guest54 [~Guest54@pool-108-53-136-237.nwrknj.fios.verizon.net] has quit [Quit: Client closed] 15:22 -!- gene_ is now known as gene 15:24 -!- ratatosk9 [~ratatosk9@193.138.218.218] has quit [Quit: Konversation terminated!] 15:29 -!- leafy-greens [~leafy-gre@modemcable123.58-83-70.mc.videotron.ca] has quit [Ping timeout: 264 seconds] 15:34 -!- Neojack [Neojack@pear.bnc4free.com] has joined #bitcoin-core-dev 15:35 -!- lightlike [~lightlike@user/lightlike] has quit [Quit: Leaving] 15:39 -!- l3kk0 [~l3kk0@c-73-22-213-40.hsd1.il.comcast.net] has quit [Ping timeout: 268 seconds] 15:39 -!- l3kk0 [~l3kk0@107.117.175.81] has joined #bitcoin-core-dev 15:48 -!- l3kk0 [~l3kk0@107.117.175.81] has quit [Read error: Connection reset by peer] 15:50 -!- l3kk0 [~l3kk0@c-73-22-213-40.hsd1.il.comcast.net] has joined #bitcoin-core-dev 16:11 -!- provoostenator [~quassel@user/provoostenator] has quit [Ping timeout: 272 seconds] 16:38 -!- sanket1729 [~sanket172@ec2-100-24-255-95.compute-1.amazonaws.com] has joined #bitcoin-core-dev 16:50 -!- gene [~gene@2a02:6f8:2020:214:100::100e] has quit [Quit: gene] 16:52 -!- jarthur [~jarthur@2603-8080-1540-002d-d88a-0d46-4103-a0ae.res6.spectrum.com] has joined #bitcoin-core-dev 17:15 -!- AaronvanW [~AaronvanW@71pc74.sshunet.nl] has quit [Quit: Leaving...] 17:35 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 17:35 < bitcoin-git> [bitcoin] jamesob opened pull request #22263: refactor: wrap CCoinsViewCursor in unique_ptr (master...2021-06-cursor-unique-ptr) https://github.com/bitcoin/bitcoin/pull/22263 17:35 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 17:39 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has quit [Ping timeout: 252 seconds] 18:25 -!- evias [~evias__@196.red-88-6-131.staticip.rima-tde.net] has joined #bitcoin-core-dev 18:25 -!- evias [~evias__@196.red-88-6-131.staticip.rima-tde.net] has quit [Changing host] 18:25 -!- evias [~evias__@user/evias] has joined #bitcoin-core-dev 18:27 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has quit [Quit: Connection closed for inactivity] 18:41 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 19:05 -!- marcinja [~marcinja@152.89.160.187] has joined #bitcoin-core-dev 19:13 -!- marcinja [~marcinja@152.89.160.187] has quit [Quit: Client closed] 19:27 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has quit [Ping timeout: 268 seconds] 19:36 -!- jarthur [~jarthur@2603-8080-1540-002d-d88a-0d46-4103-a0ae.res6.spectrum.com] has quit [Quit: jarthur] 19:40 < ariard> #proposedmeetingtopic CoreDev in 2021, a.k.a project might benefit to reconnect with in-person meetings 20:03 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 20:03 < bitcoin-git> [bitcoin] vechiv opened pull request #22265: Update wallet.cpp (master...patch-2) https://github.com/bitcoin/bitcoin/pull/22265 20:03 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 20:04 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 20:04 < bitcoin-git> [bitcoin] hebasto closed pull request #22265: Update wallet.cpp (master...patch-2) https://github.com/bitcoin/bitcoin/pull/22265 20:04 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 20:21 -!- prakash [~prakash@58.182.42.155] has quit [Quit: Leaving] 20:23 -!- Guest69 [~Guest69@2400:dd01:100f:5f38:7815:fb0d:3004:cf83] has joined #bitcoin-core-dev 20:23 -!- Guest69 [~Guest69@2400:dd01:100f:5f38:7815:fb0d:3004:cf83] has quit [Client Quit] 20:45 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 20:45 < bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6bc1eca01b2f...65c4a36e57c5 20:45 < bitcoin-git> bitcoin/master 1111457 MarcoFalke: build: Disable deprecated-copy warning only when external warnings are ena... 20:45 < bitcoin-git> bitcoin/master 65c4a36 fanquake: Merge bitcoin/bitcoin#22258: build: Disable deprecated-copy warning only w... 20:45 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 20:45 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 20:45 < bitcoin-git> [bitcoin] fanquake merged pull request #22258: build: Disable deprecated-copy warning only when external warnings are enabled (master...2106-buildEnableWarnDeprecatedCopy) https://github.com/bitcoin/bitcoin/pull/22258 20:45 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 20:50 -!- bitdex [~bitdex@gateway/tor-sasl/bitdex] has joined #bitcoin-core-dev 21:22 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has joined #bitcoin-core-dev 21:24 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has quit [Client Quit] 21:24 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has joined #bitcoin-core-dev 21:42 -!- donny [uid133844@id-133844.brockwell.irccloud.com] has quit [] 21:48 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 21:48 < bitcoin-git> [bitcoin] fanquake pushed 7 commits to master: https://github.com/bitcoin/bitcoin/compare/65c4a36e57c5...7c561bea5288 21:48 < bitcoin-git> bitcoin/master fc0eca3 Sjors Provoost: fuzz: fix fuzz binary linking order 21:48 < bitcoin-git> bitcoin/master 7d94530 Sjors Provoost: refactor: clean up external_signer.h includes 21:48 < bitcoin-git> bitcoin/master 5be90c9 Sjors Provoost: build: enable external signer by default 21:48 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 21:48 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 21:48 < bitcoin-git> [bitcoin] fanquake merged pull request #21935: Enable external signer support by default, reduce #ifdef (master...2021/05/hww-qt-compile) https://github.com/bitcoin/bitcoin/pull/21935 21:48 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 21:52 < achow101> \o/ 22:05 -!- prakash [~prakash@58.182.42.155] has joined #bitcoin-core-dev 22:11 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 22:11 < bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/7c561bea5288...d50302625e11 22:11 < bitcoin-git> bitcoin/master 28a9c9b Carl Dong: Make SHA256SUMS fragment right after build 22:11 < bitcoin-git> bitcoin/master 4cc35da Carl Dong: Rewrite guix-{attest,verify} for new hier 22:11 < bitcoin-git> bitcoin/master e2c40a4 Carl Dong: guix-attest: Error out if SHA256SUMS is unexpected 22:11 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 22:11 -!- bitcoin-git [~bitcoin-g@x0f.org] has joined #bitcoin-core-dev 22:11 < bitcoin-git> [bitcoin] fanquake merged pull request #22182: guix: Overhaul how guix-{attest,verify} works and hierarchy (master...2021-05-guix-attestation-overhaul) https://github.com/bitcoin/bitcoin/pull/22182 22:11 -!- bitcoin-git [~bitcoin-g@x0f.org] has left #bitcoin-core-dev [] 22:23 -!- prakash [~prakash@58.182.42.155] has quit [Quit: Leaving] 22:26 -!- prakash [~prakash@58.182.42.155] has joined #bitcoin-core-dev 22:26 -!- prakash [~prakash@58.182.42.155] has quit [Client Quit] 22:35 -!- prakash [~prakash@58.182.42.155] has joined #bitcoin-core-dev 22:41 -!- roconnor [~roconnor@host-184-164-16-124.dyn.295.ca] has quit [Ping timeout: 268 seconds] 22:42 -!- prakash87 [~prakash@58.182.42.155] has joined #bitcoin-core-dev 22:46 -!- goatpig [~goat@h-94-254-2-155.A498.priv.bahnhof.se] has quit [Quit: Konversation terminated!] 22:46 -!- prakash87 [~prakash@58.182.42.155] has quit [Client Quit] 22:53 -!- prakash [~prakash@58.182.42.155] has quit [Changing host] 22:53 -!- prakash [~prakash@user/prakash] has joined #bitcoin-core-dev 23:05 -!- prakash [~prakash@user/prakash] has quit [Quit: Leaving] 23:12 -!- goatpig [~goat@blocksettle-gw.cust.31173.se] has joined #bitcoin-core-dev 23:25 -!- smartin [~Icedove@88.135.18.171] has joined #bitcoin-core-dev 23:43 -!- baldur [~baldur@pool-108-30-51-126.nycmny.fios.verizon.net] has quit [Ping timeout: 268 seconds] 23:45 -!- dviola [~diego@user/dviola] has joined #bitcoin-core-dev 23:48 -!- sagi [~sagi@193.238.189.12] has joined #bitcoin-core-dev 23:50 -!- jespada [~jespada@90.254.247.46] has quit [Ping timeout: 244 seconds] 23:52 -!- jespada [~jespada@90.254.247.46] has joined #bitcoin-core-dev 23:52 -!- Guyver2 [Guyver@guyver2.xs4all.nl] has joined #bitcoin-core-dev 23:53 -!- sipsorcery [~sipsorcer@2a02:8084:6981:7880::3] has joined #bitcoin-core-dev 23:56 -!- baldur [~baldur@pool-108-30-51-126.nycmny.fios.verizon.net] has joined #bitcoin-core-dev --- Log closed Thu Jun 17 00:00:40 2021