<bitcoin-git>
[bitcoin] Eunovo opened pull request #32344: Bug/Wallet: Non-Ranged Descriptors with Range [0,0] Trigger Unexpected Wallet Errors in AddWalletDescriptor (master...bug-fix-wallet-add-non-ranged-descriptors) https://github.com/bitcoin/bitcoin/pull/32344
<bitcoin-git>
[bitcoin] laanwj closed pull request #30756: run_command: Close non-std fds when execing slave processes (master...subproc_closefds) https://github.com/bitcoin/bitcoin/pull/30756
<bitcoin-git>
[bitcoin] laanwj opened pull request #32343: common: Close non-std fds before exec in RunCommandJSON (master...2025-04-closefds) https://github.com/bitcoin/bitcoin/pull/32343
<bitcoin-git>
[bitcoin] tomasandroil opened pull request #32342: Fix missing error check in `set_clo_on_exec` for FD_CLOEXEC handling (master...fix/update) https://github.com/bitcoin/bitcoin/pull/32342
<laanwj>
dzxzg: oh i see your point now, yes it's less of an issue if we rename the repo to bitcoin-core too
<sipa>
glozow: i think that's much more applicable to bitcoin.org vs bitcoincore.org than github.com/bitcoin vs github.com/bitcoin-core, but yeah
<glozow>
yeah I definitely don't think Bitcoin and Bitcoin Core are the same thing. Really no single piece of software could be == Bitcoin. But most users don't know that, they will search for something called 'bitcoin,' run it, and manage their money with it...
<dzxzg>
I'm not asking whether they'll give it up, just wondering if it's an issue that there's an aural collision with this other org "bitcoincore/bitcoin" (but I'm overall +1 on moving to bitcoin-core)
<sr_gi[m]>
I personally think that the risk of github breaking redirects is higher than someone in the bitcoin org creating a bips repo, and what the implications may be
<sipa>
we like the project to be easy to find, and be prominent, but it isn't Bitcoin
<darosior>
glozow: i think this is again a good point to bring up bitcoin.org
<sipa>
glozow: i hear you - but it also just feels inappropriate that Bitcoin Core ended with the bitcoin/ org name, for historical reasons
<vasild>
so the only solution seems to be to move bitcoin/bips to another org and leave the rest as it is?
<achow101>
(should bitcoin exist that long)
<glozow>
What if github stops doing redirects or kicks us for squatting? Not to be a doomer but what if some random money-stealing or just poorly maintained client takes over bitcoin/bitcoin? bitcoin.org not keeping up with software updates already causes people on the network to run unmaintained software.
<sr_gi[m]>
As opposed to keeping either bitcoin or bips on it
<sr_gi[m]>
I still don't see what is the motivation for leaving bitcoin empty
<darosior>
achow101: i don't think this is a good idea for you all to give/share ownership of bitcoin/
<laanwj>
we need to keep sole ownership of bitcoin to make sure no one else creates a repo named bitcoin
<sipa>
i think the ideal solution is that both move, and bitcoin remains vestigial
<cfields>
glozow: so then what's left in bitcoin/ ?
<achow101>
glozow: we can give ownership to the bip editors too, and make bitcoin/ jointly owned
<glozow>
sipa: I think the cleaner solution to that is to have bips move though. Because assuming we're retaining ownership of the bitcoin org, we're still the ones pressing the ban button for bips
<sipa>
willcl-ark: i think the fact that it's not possible to ban someone from the Bitcoin Core org without also banning them from the BIPs org (whatever those orgs are) is a good motivation. It's not the reason on itself, but it demonstrates there is a mismatch in organisational structure. There is absolutely no reason why people maintaining a piece of software should have the ability to ban someone from
<achow101>
vasild: I don't think we should do that, and I think it's better to separate Bitcoin Core from Bitcoin
<darosior>
This strikes me as a triviality and a bit of a weak motivation to move the repo. I find making the difference between Core and Bitcoin is more compelling a reason.
<vasild>
another possibility is to keep bitcoin/bitcoin as it is and move bitcoin-core/* into bitcoin/; move bitcoin/bips into another org
<achow101>
darosior: we can call it bitcoin-core :p
<darosior>
Can we start bike-shedding the name of the repository in the bitcoin-core org then? :p
<sipa>
i think i'm in favor of moving bitcoin/bitcoin to bitcoin-core/bitcoin, and archiving libbase58/libblkmaker, if the plan is that we retain ownership of bitcoin
<darosior>
fanquake: yeah agree, maintaining the list is sync, as well as separating Bitcoin vs Bitcoin Core, seem more compelling reason to me.
<laanwj>
independently, moving bitcoin's repo to bitcoin-core makes sense, even if just for consistency and to not have to maintain parallel teams/frequent contributor lists
<darosior>
I think the current group of owners of the bitcoin org have shown themselves to be trustworthy and i don't think we should take the risk to change it.
<achow101>
and we can maintain ownership of bitcoin/ in the meanwhile
<sipa>
they can stay under bitcoin org, where we have ownership
<sipa>
fanquake: i think *our* decision here is whether to move bitcoin/bitcoin to bitcoin-core/bitcoin; the question whether bitcoin/bips moved elsewhere is up to the BIP editors
<glozow>
Slightly uncomfortable with the idea that the redirects only work if the owners of bitcoin org don't create a new bitcoin repo + giving admin up to bip editors
<fanquake>
wait why are we sunsetting bitcoin/
<cfields>
achow101: I don't think we want to leave bitcoin/ in a state where it could be perceived as dormant/up for grabs.
<laanwj>
sipa: no i don't think so, if they want an org they can have bitcoin-bips
<achow101>
sipa: I guess so? I'd still prefer to also move bips out and we can sunset bitcoin/
<sipa>
Would the bitcoin org ownership be handed to the BIPs editors then?
<glozow>
if you ban someone from bitcoin-core org, aren't they unable to comment on bitcion-core/meta issues?
<darosior>
Leaving bips in bitcoin/ and having Core in bitcoin-core/ does make sense to me, although i haven't thought about drawbacks.
<cfields>
hmm, looking at what would be left in bitcoin/, it'd just be bips/libblkmaker/libbase58. There's been discussion about moving/deprecating the latter 2 for years. So that'd just leave bips in bitcoin/.
<willcl-ark>
What are the advantages of vacating bitcoin/bitcoin? My natural instinct would be to retain that and move bips/ out from bitcoin/?
<sipa>
Ignoring history, I think having bitcoin/bips and bitcoin-core/bitcoin makes most sense, in that BIPs are actually aiming to be a whole-Bitcoin-ecosystem wide thing.
<laanwj>
"libbase58" is also still in bitcoin i don't know what to do with that
<fanquake>
wouldn't moving the repo break the setup of banning from bitcoin/bitcoin, and then the person can actually comment in bitcoin-core/meta, otherwise if they both exist in the same org, how will the banne comment in meta
<laanwj>
yes could move the bips repo as well, empty out "bitcoin" completely eventually
<laanwj>
sounds good to me, as long as the old git URL will keep working , it's been the plan for as long as the bitcoin-core org exists, really
<achow101>
The main issue that there could be is of course that bitcoin/bitcoin is referenced in a ton of places. However, github already handles this by redirecting all links from the old repo location to the new repo location. All PRs and issues are transferred, all links automatically redirect, and git remotes will automatically redirect too so no one would even have to change their git config. All that needs to happen is that a bitcoin/bitcoin repo
<achow101>
Transferring the repo is trivial - the same people are owners of both bitcoin and bitcoin-core, it's just a matter of clicking the button. However, it is also something that I think requires contributor buy-in, and notice that it's happening.
<achow101>
Moving the repo to bitcoin-core would also reduce administration overhead - primarily that keeping 2 organizations' member and team lists in sync is kind of annoying.
<achow101>
The solution to this issue is to not have both Bitcoin Core and BIPs under the same organization. We already have the bitcoin-core organization where all of our new repos go anyways, it would entirely make sense to move bitcoin/bitcoin to bitcoin-core/bitcoin-core, or something like that. There's also a bitcoin-bips organization that was created a couple of years ago for this, so bips could be moved there. But I think that's for the bip editors
<achow101>
This past week, the topic of moving the repo to bitcoin-core has come up again due to possible issues with conflicting moderation between bitcoin/bitcoin and bitcoin/bips. The main point is that bans are issued at the organization level, but Bitcoin Core and BIPs are really two separate projects. It is therefore conceivable that Bitcoin Core would want to ban someone, but BIPs would not, and vice versa. Under the current setup, any ban from one
<achow101>
#topic Move the repo to bitcoin-core github organization (achow101)
<johnny9dev>
Took a bit longer to get back in a rhythm after being away for 2 weeks but I managed to fix some issues found by Marnix as well as handle the case when there are no Coins to select in my Coin Control PR (bitcoin-core/gui-qml#448).
<bitcoin-git>
[bitcoin] nervana21 opened pull request #32333: doc: Add missing top-level description to pruneblockchain RPC (master...04-23-rpc-pruneblockchain-doc) https://github.com/bitcoin/bitcoin/pull/32333
2025-04-23
<bitcoin-git>
[bitcoin] w0xlt opened pull request #32332: refactor: Update `XOnlyPubKey::GetKeyIDs()` to return a pair of pubkeys (master...xonly_pair_pubkey) https://github.com/bitcoin/bitcoin/pull/32332
<bitcoin-git>
[bitcoin] fanquake merged pull request #32310: test: Run all benchmarks in the sanity check (master...2504-bench-test-all) https://github.com/bitcoin/bitcoin/pull/32310
<bitcoin-git>
bitcoin/master 82d1e94 merge-script: Merge bitcoin/bitcoin#32310: test: Run all benchmarks in the sanity check
<bitcoin-git>
bitcoin/master faca46b MarcoFalke: test: Run all benchmarks in the sanity check
<bitcoin-git>
[bitcoin] maflcko closed pull request #32006: docs: remove passing CI section in guidelines for PR merging as it's common sense (master...fix/typos) https://github.com/bitcoin/bitcoin/pull/32006
<achow101>
#proposedmeetingtopic Move the repo to bitcoin-core github organization
<bitcoin-git>
[bitcoin] hebasto merged pull request #32309: bench: close wallets after migration (master...l0rinc/wallet-migration-bench-fix) https://github.com/bitcoin/bitcoin/pull/32309
<bitcoin-git>
[bitcoin] maflcko opened pull request #32327: test: Add missing check for empty stderr in util tester (master...2504-test-util-empty-err) https://github.com/bitcoin/bitcoin/pull/32327
<bitcoin-git>
[bitcoin] vasild opened pull request #32326: net: improve the interface around FindNode() and avoid a recursive mutex lock (master...findnode) https://github.com/bitcoin/bitcoin/pull/32326
<bitcoin-git>
[bitcoin] maflcko opened pull request #32325: ci: Add missing -Wno-error=array-bounds to valgrind fuzz (master...2504-fuzz-v) https://github.com/bitcoin/bitcoin/pull/32325
<bitcoin-git>
[bitcoin] maflcko reopened pull request #32006: docs: remove passing CI section in guidelines for PR merging as it's common sense (master...fix/typos) https://github.com/bitcoin/bitcoin/pull/32006
<bitcoin-git>
[bitcoin] maflcko closed pull request #32006: docs: remove passing CI section in guidelines for PR merging as it's common sense (master...fix/typos) https://github.com/bitcoin/bitcoin/pull/32006
<bitcoin-git>
[bitcoin] maflcko opened pull request #32310: test: Run all benchmarks in the sanity check (master...2504-bench-test-all) https://github.com/bitcoin/bitcoin/pull/32310
<bitcoin-git>
[bitcoin] l0rinc opened pull request #32309: bench: close wallets after migration (master...l0rinc/wallet-migration-bench-fix) https://github.com/bitcoin/bitcoin/pull/32309
<bitcoin-git>
[bitcoin] hebasto opened pull request #32308: ci: Drop no longer necessary `-Wno-error=array-bounds` (master...250419-array-bound) https://github.com/bitcoin/bitcoin/pull/32308
<bitcoin-git>
[bitcoin] instagibbs opened pull request #32304: test: test MAX_SCRIPT_SIZE for block validity (master...2025-04-max_script_size_test) https://github.com/bitcoin/bitcoin/pull/32304
<bitcoin-git>
[bitcoin] instagibbs opened pull request #32301: test: cover invalid codesep positions for signature in taproot (master...2025-04-codesep_cov) https://github.com/bitcoin/bitcoin/pull/32301
<bitcoin-git>
[bitcoin] l0rinc closed pull request #31923: refactor: use original log string when no suspicious chars found (master...lorinc/optimistic-log-escape-message) https://github.com/bitcoin/bitcoin/pull/31923
<bitcoin-git>
[bitcoin] sipa opened pull request #32300: feefrac: avoid integer overflow in temporary (master...202504_feefrac_div_fix) https://github.com/bitcoin/bitcoin/pull/32300
<bitcoin-git>
[bitcoin] glozow merged pull request #32191: Make TxGraph fuzz tests more deterministic (master...202504_txgraph_deterministic) https://github.com/bitcoin/bitcoin/pull/32191
<bitcoin-git>
bitcoin/master 2835216 Pieter Wuille: txgraph: make GroupClusters use partition numbers directly (optimization)
<bitcoin-git>
bitcoin/master 247e9de merge-script: Merge bitcoin/bitcoin#32191: Make TxGraph fuzz tests more deterministic
<johnny9dev>
for gui-qml: back from 1.5 weeks of travel. Will be focused on answering questions around shutdown that came up and getting bitcoin-core/gui-qml#448 merged.
<bitcoin-git>
[bitcoin] fanquake merged pull request #32288: ci: drop -priority-level from bench in win cross CI (master...win_cross_ci_bench_no_prio) https://github.com/bitcoin/bitcoin/pull/32288
<bitcoin-git>
bitcoin/master 27f1121 fanquake: ci: drop -priority-level from bench in win cross CI
<bitcoin-git>
[bitcoin] mzumsande opened pull request #32290: test: allow all functional tests to be run or skipped with --usecli (master...202505_fix_usecli) https://github.com/bitcoin/bitcoin/pull/32290