<jeremyrubin>
fwiw for anyone interested, C/F a meeting in Feb-22 https://gnusha.org/bitcoin-core-dev/2022-02-24.log with laanwj pointing out that glozow as mempool maintainer would be a bad idea, given it's her prime area of work
<glozow>
I care about the health of mempool/policy and Bitcoin Core, and I take it very seriously so I'm happy to be held to a high standard. All of the work we do is entirely in public, so you are always free to audit me, tell me I've done something wrong, or request I prioritize other things. If you want to test my knowledge or preparedness, please let me know what the standard is beforehand and apply it equally to everyone in the same role.
<luke-jr>
there is no bitcoin core organization
<jeremyrubin>
i want to spend time on a beach with a pina colada -- however, process work is a part of the development of the "bitcoin core organization", so you can't just stick your head in the sand and ignore it, especially when it comes to doling out commit access
<bytes1440000>
i am not sure if bitcoin core needs another person with commit access with already 6 but always felt it needs more reviewers looking at open pull requests
<bytes1440000>
b10c: its a good point and maintainers funded by different orgs is always better for a project like bitcoin core
<fanquake>
She has been actively working on Bitcoin Core for > 2 years. Predominately in the mempool & validation code.
<bitcoin-git>
[bitcoin] dergoegge opened pull request #25514: net processing: Move CNode::nServices and CNode::nLocalServices to Peer (master...2022-06-move-services) https://github.com/bitcoin/bitcoin/pull/25514
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25383: wallet: don't read db every time that a new 'WalletBatch' is created (master...2022_wallet_db_read) https://github.com/bitcoin/bitcoin/pull/25383
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25511: test: non-positive integer value to `-peertimeout` should throw an error (master...2022-06-peertimeout-positive-integer) https://github.com/bitcoin/bitcoin/pull/25511
<bitcoin-git>
bitcoin/master d22bd54 brunoerg: test: passing a non-positive integer value to `-peertimeout` should throw ...
<bitcoin-git>
bitcoin/master b6cf0f8 MacroFake: Merge bitcoin/bitcoin#25511: test: non-positive integer value to `-peertim...
<laanwj>
well, bips is the thing that can be argued to belong under bitcoin itself
<laanwj>
jamesob: ideally everything should be under bitcoin-core, that said, the bitcoin/bitcoin repo name is hardcoded in so many places
<bitcoin-git>
[bitcoin] brunoerg opened pull request #25511: test: non-positive integer value to `-peertimeout` should throw an error (master...2022-06-peertimeout-positive-integer) https://github.com/bitcoin/bitcoin/pull/25511
<jamesob>
Random question: was there ever a rationale for the bitcoin/bitcoin-core github org split? I.e. why isn't everything just under one or the other?
<bitcoin-git>
[bitcoin] darosior opened pull request #25510: rpc: explicit the range for listsinceblock's filtering by block hash is exclusive (master...doc_listsinceblock) https://github.com/bitcoin/bitcoin/pull/25510
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25506: Rephrase error message for invalid value of `-peertimeout` (master...peertimeout-error-msg) https://github.com/bitcoin/bitcoin/pull/25506
<bitcoin-git>
bitcoin/master 748a10e /dev/fd0: rephrase error for invalid timeout
<bitcoin-git>
bitcoin/master bae8a66 MacroFake: Merge bitcoin/bitcoin#25506: Rephrase error message for invalid value of `...
<bitcoin-git>
[bitcoin] MarcoFalke closed pull request #25505: test: passing a negative value to `-peertimeout` should throw an error (master...2022-06-peertimeout-negative) https://github.com/bitcoin/bitcoin/pull/25505
<bitcoin-git>
[bitcoin] fanquake opened pull request #25508: guix: use elfesteem 2eb1e5384ff7a220fd1afacd4a0170acff54fe56 (master...update_elfesteem) https://github.com/bitcoin/bitcoin/pull/25508
<bitcoin-git>
[bitcoin] S3RK opened pull request #25507: wallet: don't add change fee to target if subtracting fees from output (master...correct_target_with_sffo) https://github.com/bitcoin/bitcoin/pull/25507
<bitcoin-git>
[bitcoin] 1440000bytes opened pull request #25506: add error msg for invalid value of `-peertimeout` (master...peertimeout-error-msg) https://github.com/bitcoin/bitcoin/pull/25506
<bitcoin-git>
[bitcoin] brunoerg opened pull request #25505: test: passing a negative value to `-peertimeout` should throw an error (master...2022-06-peertimeout-negative) https://github.com/bitcoin/bitcoin/pull/25505
<bitcoin-git>
[bitcoin] darosior opened pull request #25504: RPC: allow to track coins by imported descriptors (master...rpc_track_coins_by_descriptor) https://github.com/bitcoin/bitcoin/pull/25504
<bitcoin-git>
bitcoin/master 28a28a0 fanquake: Squashed 'src/minisketch/' changes from 7eeb778fef..47f0a2d26f
<bitcoin-git>
bitcoin/master dc375e5 fanquake: Update minisketch subtree to latest master
<bitcoin-git>
[bitcoin] theStack opened pull request #25503: test: pass `datacarriersize` option for tests using large outputs (instead of `acceptnonstdtxn`) (master...202206-test-replace_acceptnonstdxn_with_datacarriersize_option) https://github.com/bitcoin/bitcoin/pull/25503
<bitcoin-git>
[bitcoin] achow101 merged pull request #25497: wallet: more accurate target for large transactions (master...wallet_noninput_fees) https://github.com/bitcoin/bitcoin/pull/25497
<lightlike>
laanwj: cool - looks like the high lvl prio project is now one level higher (/bitcoin) than the old projects (/bitcoin/bitcoin), is that on purpose?
<bitcoin-git>
[bitcoin] fanquake merged pull request #25495: Revert "bnb: exit selection when best_waste is 0" (master...2022-06-fix-waste-0-bug) https://github.com/bitcoin/bitcoin/pull/25495
<bitcoin-git>
[bitcoin] dergoegge opened pull request #25500: refactor: Move inbound eviction logic to its own translation unit (master...2022-06-evictionman-1ofN) https://github.com/bitcoin/bitcoin/pull/25500
<bitcoin-git>
[bitcoin] S3RK opened pull request #25497: wallet: more accurate target for large transactions (master...wallet_noninput_fees) https://github.com/bitcoin/bitcoin/pull/25497
<bitcoin-git>
[bitcoin] vasild opened pull request #25491: wallet: use Mutex for g_sqlite_mutex instead of GlobalMutex (master...g_sqlite_mutex) https://github.com/bitcoin/bitcoin/pull/25491
<bitcoin-git>
[bitcoin] fanquake opened pull request #25490: guix: more cross arch reproducibility (x86_64 - arm64) (master...24615_rebased_no_libtool) https://github.com/bitcoin/bitcoin/pull/25490
<bitcoin-git>
[bitcoin] laanwj merged pull request #25426: net: add new method Sock::GetSockName() that wraps getsockname() and use it in GetBindAddress() (master...getsockname) https://github.com/bitcoin/bitcoin/pull/25426
<bitcoin-git>
bitcoin/master 748dbcd Vasil Dimov: net: add new method Sock::GetSockName() that wraps getsockname()
<bitcoin-git>
bitcoin/master a8d6abb Vasil Dimov: net: change GetBindAddress() to take Sock argument
<bitcoin-git>
bitcoin/master ba29911 laanwj: Merge bitcoin/bitcoin#25426: net: add new method Sock::GetSockName() that ...
<bitcoin-git>
[bitcoin] dongcarl opened pull request #25487: [kernel 3b/n] Make `{Dump,Load}Mempool` `CTxMemPool` methods, decouple from `ArgsManager` (master...2022-05-kernelargs-mempool-loaddump) https://github.com/bitcoin/bitcoin/pull/25487
<laanwj>
i've migrated the "high priority for review" project to the new github projects it's now at https://github.com/orgs/bitcoin/projects/1, everything should otherwise be the same
<bitcoin-git>
[bitcoin] theStack opened pull request #25486: test: fix failing test `interface_usdt_utxocache.py` (master...202206-test-fix_interface_usdt_utxocache) https://github.com/bitcoin/bitcoin/pull/25486
<bitcoin-git>
[bitcoin] w0xlt opened pull request #25485: rpc: Use enum instead of string for `filtertype_name` (master...filtertype_enum) https://github.com/bitcoin/bitcoin/pull/25485
<bitcoin-git>
[bitcoin] fanquake opened pull request #25484: Guix: enable toolchain hardening by default (master...guix_toolchain_hardening) https://github.com/bitcoin/bitcoin/pull/25484
<bitcoin-git>
[bitcoin] fanquake merged pull request #25483: Revert "ci: Increase CPU number for "Win64 native" task" (master...220627-ci) https://github.com/bitcoin/bitcoin/pull/25483
<bitcoin-git>
bitcoin/master fa34e44 MacroFake: test: Return new_utxo from create_self_transfer in MiniWallet
<bitcoin-git>
bitcoin/master fa04ff6 MacroFake: test: Return new_utxos from create_self_transfer_multi in MiniWallet
<bitcoin-git>
[bitcoin] Crypt-iQ closed pull request #25462: index: ignore BlockConnected if pindex is in chain (master...fix_dupe_filterfile) https://github.com/bitcoin/bitcoin/pull/25462
<bitcoin-git>
[bitcoin] hebasto opened pull request #25483: Revert "ci: Increase CPU number for "Win64 native" task" (master...220627-ci) https://github.com/bitcoin/bitcoin/pull/25483
<bitcoin-git>
[bitcoin] mzumsande opened pull request #25471: rpc: Disallow gettxoutsetinfo queries for a specific block with `use_index=false` (master...202206_gettxoutsetinfo_check) https://github.com/bitcoin/bitcoin/pull/25471
<bitcoin-git>
[bitcoin] hebasto reopened pull request #25460: ci: Update Windows task image up to visualstudio2022 (master...220623-vs2022) https://github.com/bitcoin/bitcoin/pull/25460
<bitcoin-git>
[bitcoin] luke-jr opened pull request #25463: Bugfix: Do not define HAVE_O_CLOEXEC for LevelDB when O_CLOEXEC is not available (master...fix_leveldb_no_cloexec) https://github.com/bitcoin/bitcoin/pull/25463
<bitcoin-git>
[bitcoin] Crypt-iQ opened pull request #25462: index: ignore BlockConnected if pindex is in chain (master...fix_dupe_filterfile) https://github.com/bitcoin/bitcoin/pull/25462
2022-06-23
<bitcoin-git>
[bitcoin] hebasto closed pull request #25460: ci: Update Windows task image up to visualstudio2022 (master...220623-vs2022) https://github.com/bitcoin/bitcoin/pull/25460
<jamesob>
[shill] if anyone is looking to dispose of their glut of spare review time :), I might recommend checking out these nice taproot unittests that first-time contributor david-bakin wrote: https://github.com/bitcoin/bitcoin/pull/25097
<bitcoin-git>
[bitcoin] hebasto opened pull request #25460: ci: Update Windows task image up to visualstudio2022 (master...220623-vs2022) https://github.com/bitcoin/bitcoin/pull/25460
<bitcoin-git>
[bitcoin] MarcoFalke closed pull request #25420: rpc: small description/help improvements in blockchain.cpp (master...rpc-description-improvements-blockchain) https://github.com/bitcoin/bitcoin/pull/25420
<bitcoin-git>
[bitcoin] laanwj merged pull request #25457: Use more specific path when including `memenv.h` header (master...220623-memenv) https://github.com/bitcoin/bitcoin/pull/25457
<bitcoin-git>
[bitcoin] hebasto opened pull request #25457: Use more specific path when including `memenv.h` header (master...220623-memenv) https://github.com/bitcoin/bitcoin/pull/25457
<bitcoin-git>
[bitcoin] MarcoFalke opened pull request #25456: rpc: Use steady_clock for getrpcinfo durations (master...2206-rpc-steady-duration-🎴) https://github.com/bitcoin/bitcoin/pull/25456
<in3rsha>
Is the Compact Size field (e.g. the one used for indicating the number of inputs in a tx) something that Satoshi came up with for bitcoin or is it found in other network protocols?
<bitcoin-git>
[bitcoin] MarcoFalke merged pull request #25451: test: `-whitebind` and `-bind` with `-listen=0` should throw an error (master...2022-06-test-whitebind-bind-listen0) https://github.com/bitcoin/bitcoin/pull/25451
<bitcoin-git>
bitcoin/master ceec680 brunoerg: test: `-whitebind` and `-bind` with `-listen=0` should throw an error
<bitcoin-git>
bitcoin/master 01e9e2d MacroFake: Merge bitcoin/bitcoin#25451: test: `-whitebind` and `-bind` with `-listen...
<bitcoin-git>
bitcoin/master 3f51d0b Hennadii Stepanov: qt6: Fix type registration
<bitcoin-git>
bitcoin/master ad73447 Hennadii Stepanov: qt6: Do not use deprecated `QLibraryInfo::path` in Qt 6
<bitcoin-git>
[gui] laanwj merged pull request #623: Getting ready to Qt 6 (9/n). Apply Qt 6 specific changes (master...220622-qt6) https://github.com/bitcoin-core/gui/pull/623
2022-06-22
<bitcoin-git>
[bitcoin] sdaftuar opened pull request #25454: p2p, refactor: Avoid multiple getheaders messages in flight to the same peer (master...2022-06-single-getheaders) https://github.com/bitcoin/bitcoin/pull/25454
<bitcoin-git>
[bitcoin] brunoerg opened pull request #25451: test: `-whitebind` and `-bind` with `-listen=0` should throw an error (master...2022-06-test-whitebind-bind-listen0) https://github.com/bitcoin/bitcoin/pull/25451
<bitcoin-git>
[bitcoin] luke-jr opened pull request #25449: [21.x] Qt: Handle exceptions instead of crash (0.21...qt_handle_exceptions_pr260-21-corepr) https://github.com/bitcoin/bitcoin/pull/25449