grettke has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
jarthur has quit [Ping timeout: 272 seconds]
sipsorcery has quit [Ping timeout: 272 seconds]
b10c has quit [Quit: Connection closed for inactivity]
jarthur has joined #bitcoin-core-dev
AaronvanW has quit [Remote host closed the connection]
AaronvanW has joined #bitcoin-core-dev
grettke has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] ljkt7334 opened pull request #402: Create codeql-analysis.yml (master...master) https://github.com/bitcoin-core/gui/pull/402
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] fanquake closed pull request #402: Create codeql-analysis.yml (master...master) https://github.com/bitcoin-core/gui/pull/402
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 joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/0b5344b0d187...c3545a739678
<bitcoin-git> bitcoin/master 6a5ccd6 Hennadii Stepanov: scripted-diff: Rename JoinErrors in more general MakeUnorderedList
<bitcoin-git> bitcoin/master 77a90f0 Hennadii Stepanov: refactor: Move MakeUnorderedList into util/string.h to make it reusable
<bitcoin-git> bitcoin/master bb56486 Hennadii Stepanov: refactor: Reuse MakeUnorderedList where possible
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake merged pull request #22653: refactor: Rename JoinErrors and re-use it (master...210806-err) https://github.com/bitcoin/bitcoin/pull/22653
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #22625: guix: Build out of source tree (master...210804-tag) https://github.com/bitcoin/bitcoin/pull/22625
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
jarthur has quit [Ping timeout: 258 seconds]
jarthur has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #18818: Fix release tarball generated by gitian (master...fix_gitian_src_202004) https://github.com/bitcoin/bitcoin/pull/18818
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has quit [Ping timeout: 272 seconds]
AaronvanW has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
yanmaani has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 258 seconds]
AaronvanW has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake closed pull request #22664: docs: fix typo spelling grammar (master...docs-fixing) https://github.com/bitcoin/bitcoin/pull/22664
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.
AaronvanW has quit [Ping timeout: 248 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
AaronvanW has joined #bitcoin-core-dev
b10c has joined #bitcoin-core-dev
RDK has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 272 seconds]
AaronvanW has joined #bitcoin-core-dev
lkqwejhhgasdjhgn has joined #bitcoin-core-dev
RDK has quit [Quit: Leaving]
SpellChecker has joined #bitcoin-core-dev
bomb-on has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 248 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 268 seconds]
AaronvanW has joined #bitcoin-core-dev
babasancheti has quit [Quit: Client closed]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 3 commits to 0.21: https://github.com/bitcoin/bitcoin/compare/4b523c8f0a72...89d148c8c65b
<bitcoin-git> bitcoin/0.21 e94e433 W. J. van der Laan: doc: Update manual pages for 0.21.2
<bitcoin-git> bitcoin/0.21 bf672ce W. J. van der Laan: build: Bump version to 0.21.2rc1
<bitcoin-git> bitcoin/0.21 89d148c W. J. van der Laan: qt: Translations update for 0.21.2rc1
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has quit [Ping timeout: 258 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] laanwj pushed 1 commit to 22.x: https://github.com/bitcoin/bitcoin/compare/873fbc745d03...d3bd5410f64e
<bitcoin-git> bitcoin/22.x d3bd541 W. J. van der Laan: doc: Manual pages update for 22.0rc2
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
Zenton has quit [Remote host closed the connection]
sipsorcery has quit [Remote host closed the connection]
sipsorcery has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 248 seconds]
common has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
vnogueir- has joined #bitcoin-core-dev
vnogueira has quit [Ping timeout: 244 seconds]
AaronvanW has quit [Ping timeout: 258 seconds]
<Talkless> what does "Code review ACK" mean, how it's different.. from others? :)
<hebasto> Talkless: reviewed only, not actually tested
<Talkless> hebasto: so same as utACK ?
<Talkless> or is there still difference?
<hebasto> yes, but `utACK` is kind of undocumented
<sipa_> and it may mean "i haven't thought if this is a good idea, but the code changes look fine"
AaronvanW has joined #bitcoin-core-dev
<sipa_> i interpret crACK as an utACK without concept/approach ACK
jonatack has quit [Quit: Client closed]
AaronvanW has quit [Ping timeout: 268 seconds]
<jnewbery> sipa_: oh interesting, I'd just started using it instead of utACK (since it seems more precise)
<hebasto> jnewbery: agree, crACK seems more precise and looks fancy
AaronvanW has joined #bitcoin-core-dev
<sipa_> no reason why you can't write a while sentence to elaborate on exactly what you did
<sipa_> *whole
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto merged pull request #354: Refactor open date range to use std::optional (master...2021-06-date-range) https://github.com/bitcoin-core/gui/pull/354
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c3545a739678...3d9cdb16897b
<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" ;)
yanmaani has quit [Ping timeout: 244 seconds]
AaronvanW has quit [Ping timeout: 248 seconds]
vysn has quit [Ping timeout: 258 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto merged pull request #317: Add Direction column to Peers Tab (master...direction-col-peers) https://github.com/bitcoin-core/gui/pull/317
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> bitcoin/master 6971e79 Jon Atack: gui: add Direction column to peers tab
<bitcoin-git> [bitcoin] hebasto pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3d9cdb16897b...77e23ca94503
<bitcoin-git> bitcoin/master 77e23ca Hennadii Stepanov: Merge bitcoin-core/gui#317: Add Direction column to Peers Tab
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
<jonatack> oh, nice to see this cleared up ^ thank you jarolrod for picking it up
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto closed pull request #372: Remove "Type" column from peers window (master...remove-peers-type-column) https://github.com/bitcoin-core/gui/pull/372
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
<jonatack> ty hebasto
<hebasto> thanks to jonatack and jarolrod
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto closed pull request #400: Honor inbound/outbound arrow prefix when sorting peer address column (master...address_prefix_sort) https://github.com/bitcoin-core/gui/pull/400
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]
yanmaani has quit [Ping timeout: 244 seconds]
SpellChecker has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 272 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 256 seconds]
AaronvanW has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 248 seconds]
yanmaani has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/77e23ca94503...e614cc8cd808
<bitcoin-git> bitcoin/master 0237d95 Prateek Sancheti: qt: Add Load PSBT functionaliy with nowallet
<bitcoin-git> bitcoin/master e614cc8 Hennadii Stepanov: Merge bitcoin-core/gui#399: Fix "Load PSBT" functionality when no wallet l...
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
AaronvanW has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto merged pull request #399: Fix "Load PSBT" functionality when no wallet loaded (master...i232-0806) https://github.com/bitcoin-core/gui/pull/399
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]
vnogueira has joined #bitcoin-core-dev
sipsorcery has quit [Ping timeout: 245 seconds]
sipsorcery has joined #bitcoin-core-dev
nanotube has quit [Ping timeout: 248 seconds]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [gui] hebasto merged pull request #390: Add SubFeeFromAmount to options (master...i386_0729) https://github.com/bitcoin-core/gui/pull/390
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
bitcoin-git has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] hebasto pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/e614cc8cd808...9948f114f8e6
<bitcoin-git> bitcoin/master ad28b66 Prateek Sancheti: qt: Add SubFeeFromAmount option
<bitcoin-git> bitcoin/master 62b125f Prateek Sancheti: qt, refactor: Fix indentation
<bitcoin-git> bitcoin/master 9948f11 Hennadii Stepanov: Merge bitcoin-core/gui#390: Add SubFeeFromAmount to options
bitcoin-git has left #bitcoin-core-dev [#bitcoin-core-dev]
nanotube has joined #bitcoin-core-dev
bomb-on has quit [Quit: aллилѹіа!]
bitdex has joined #bitcoin-core-dev