bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
grettke has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
AaronvanW has quit [Ping timeout: 248 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 258 seconds]
AaronvanW has joined #bitcoin-core-dev
warren_ is now known as warren
earnestly has quit [Ping timeout: 248 seconds]
AaronvanW has quit [Ping timeout: 268 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git>
[bitcoin] achow101 opened pull request #22680: guix: make git tag and commit known prior to building (master...guix-use-tag) https://github.com/bitcoin/bitcoin/pull/22680
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
Guest12 has joined #bitcoin-core-dev
Guest12 has quit [Client Quit]
Guest66 has joined #bitcoin-core-dev
<Guest66>
when is the next meeting?
Guest66 has quit [Client Quit]
AaronvanW has quit [Ping timeout: 258 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 272 seconds]
AaronvanW has joined #bitcoin-core-dev
belcher_ has joined #bitcoin-core-dev
belcher has quit [Ping timeout: 272 seconds]
babasancheti has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 272 seconds]
AaronvanW has joined #bitcoin-core-dev
commmon has quit [Read error: Connection reset by peer]
AaronvanW has quit [Ping timeout: 248 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 258 seconds]
AaronvanW has joined #bitcoin-core-dev
jarthur has quit [Quit: jarthur]
belcher_ is now known as belcher
AaronvanW has quit [Ping timeout: 268 seconds]
Guyver2 has joined #bitcoin-core-dev
vysn has joined #bitcoin-core-dev
jarthur has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
jarthur has quit [Ping timeout: 268 seconds]
Talkless has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
hex17or has quit [Ping timeout: 244 seconds]
hex17or has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
sipsorcery has joined #bitcoin-core-dev
earnestly has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 248 seconds]
AaronvanW has joined #bitcoin-core-dev
sipsorcery has quit [Ping timeout: 268 seconds]
sipsorcery has joined #bitcoin-core-dev
<jnewbery>
roasbeef: > since the "new" protocol essentially uses the space between version and verack for interactive protocol feature negotiation instead of service bits
<jnewbery>
My mental model is that service bits are for permanent attributes of the node. They communicate what services the node is capable of providing (historic blocks, segwit serialized blocks & txs, compact block filters, etc), and that service support is gossipped around the network in addrs.
<jnewbery>
That allows nodes that require specific services to find peers that support those services.
<jnewbery>
Feature negotiation using individual messages like sendcmpct, wtxidrelay, sendaddrv2, etc are for features for that specific connection. It's perfectly fine for a node to negotiate those features on some of its connections and not on others.
<jnewbery>
If you need to support addrv2 and not wtxidrelay, it's safe to use p2p protocol version 70016 and just not send `wtxidrelay` during the version handshake. The peer will see that you don't support wtxidrelay and won't inv txs to you using wtxid.
<bitcoin-git>
bitcoin/master 4830f49 João Barbosa: qt: Refactor open date range to use std::optional
<bitcoin-git>
bitcoin/master 3d9cdb1 Hennadii Stepanov: Merge bitcoin-core/gui#354: Refactor open date range to use std::optional
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<Talkless>
hebasto: Code review ACK is alos not docummented..? :/
jonatack has joined #bitcoin-core-dev
<laanwj>
"code review ACK" for me is "i've looked at the code changes and they look good to me, no comments, no critical conceptual issues, have run the auto-tests locally but not done any specific testing" (by far my most common kind of ACK)
<Talkless>
laanwj: and utACK ?
<laanwj>
i try to never use that
<laanwj>
it's just too aspecific
<laanwj>
i do "tested ACK" (which means I did some specific testing of the feature or change, and generally code review too, unless mentioned otherwise), "concept ACK" (didn't look at the code but the described concept sounds good, I do not include a commit id with this), "concept and code review ACK" sometimes (I especially like the concept and also reviewed the code)
<Talkless>
OK so I'll not gonna use utACK
<jonatack>
i guess part of the job of maintaining is evaluating what different people mean with their ACKs :)
<hebasto>
the simplest way is just to follow our own dev docs
<jonatack>
("code review ACK" isn't described in CONTRIBUTING.md, much less "crACK" ;)
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
vnogueir- is now known as vnogueira
AaronvanW has quit [Ping timeout: 268 seconds]
yanmaani has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
grettke has joined #bitcoin-core-dev
vasild has quit [Ping timeout: 244 seconds]
gleb has quit [Ping timeout: 272 seconds]
<Talkless>
jonatack: eyah, reviewing docs could be better...
vasild has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
gleb7 has joined #bitcoin-core-dev
muhblockchain has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
lightlike has joined #bitcoin-core-dev
VzxPLnHqr has joined #bitcoin-core-dev
grettke has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
grettke has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 245 seconds]
<achow101>
laanwj: are we doing 0.21.2rc1 soon?
jarthur has joined #bitcoin-core-dev
Naiza has joined #bitcoin-core-dev
lkqwejhhgasdjhgn has quit [Quit: Konversation terminated!]
AaronvanW has joined #bitcoin-core-dev
sipsorcery has quit [Ping timeout: 268 seconds]
lightlike has left #bitcoin-core-dev [Leaving]
sipsorcery has joined #bitcoin-core-dev
vasild has quit [Ping timeout: 244 seconds]
raj has joined #bitcoin-core-dev
kuler has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
vasild has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
<Talkless>
laanwj: about "Tested ACK" means only that you tested GUI for expected behaviour, but did not ran any unit/functional tests..? Here's also ambiguity :/
<sipa_>
really, if you feel there is ambiguity, write a sentence that describes what you did
ghost43 has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git>
[gui] hebasto opened pull request #403: refactor: Make paths to update Encryption and HD wallet statuses simpler (master...210811-hd) https://github.com/bitcoin-core/gui/pull/403
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Aaronvan_ has joined #bitcoin-core-dev
AaronvanW has quit [Read error: Connection reset by peer]
Jackielove4u has quit [Quit: Connection closed for inactivity]
SpellChecker has quit [Quit: bye]
SpellChecker has joined #bitcoin-core-dev
raj has quit [Quit: Leaving]
sipsorcery has quit [Ping timeout: 272 seconds]
janb has joined #bitcoin-core-dev
Naiza has quit [Quit: Connection closed]
VzxPLnHqr has quit [Remote host closed the connection]
Jackielove4u has joined #bitcoin-core-dev
lightlike has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git>
[bitcoin] theStack opened pull request #22684: test: check for invalid `-prune` parameters (master...202108-test-check_invalid_prune_parameters) https://github.com/bitcoin/bitcoin/pull/22684
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Aaronvan_ has quit [Remote host closed the connection]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 252 seconds]
AaronvanW has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
babasancheti has joined #bitcoin-core-dev
babasancheti has quit [Client Quit]
AaronvanW has quit [Ping timeout: 245 seconds]
sipsorcery has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
vasild has quit [Ping timeout: 244 seconds]
meshcollider has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
Guyver2_ has joined #bitcoin-core-dev
Guyver2 has quit [Ping timeout: 248 seconds]
Guyver2_ is now known as Guyver2
vysn has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
mdrollette has joined #bitcoin-core-dev
meshcollider has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
sipa_ is now known as sipa
AaronvanW has quit [Ping timeout: 248 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git>
[bitcoin] dongcarl opened pull request #22685: clientversion: No suffix #if CLIENT_VERSION_IS_RELEASE (master...2021-08-clientversion-is-cursed) https://github.com/bitcoin/bitcoin/pull/22685
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
Guest71 has joined #bitcoin-core-dev
Guest71 has quit [Client Quit]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git>
[bitcoin] achow101 closed pull request #22680: guix: make git tag and commit known prior to building (master...guix-use-tag) https://github.com/bitcoin/bitcoin/pull/22680
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
Guyver2 has quit [Quit: Going offline, see ya! (www.adiirc.com)]
AaronvanW has quit [Ping timeout: 252 seconds]
Lightsword has quit [Quit: ZNC]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
AaronvanW has joined #bitcoin-core-dev
vnogueir- has joined #bitcoin-core-dev
vnogueira has quit [Remote host closed the connection]
Talkless has quit [Quit: Konversation terminated!]
Lightsword has joined #bitcoin-core-dev
AaronvanW has quit [Remote host closed the connection]
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
Ananta-shesha has quit [Ping timeout: 256 seconds]
Ananta-shesha has joined #bitcoin-core-dev
hex17or has quit [Ping timeout: 244 seconds]
hex17or has joined #bitcoin-core-dev
hex17or has quit [Remote host closed the connection]
hex17or has joined #bitcoin-core-dev
sipsorcery has quit [Remote host closed the connection]
sipsorcery has joined #bitcoin-core-dev
lightlike has quit [Quit: Leaving]
<fanquake>
achow101: yes
meshcollider has quit [Changing host]
meshcollider has joined #bitcoin-core-dev
vnogueir- has quit [Remote host closed the connection]