Bitcoin Core Github
44 subscribers
121K links
Download Telegram
📝 fanquake unlocked a pull request: "log: expand BCLog::LogFlags (categories) to 64 bits"
(https://github.com/bitcoin/bitcoin/pull/26619)
Increase the maximum number of logging categories from 32 to 64.

We're currently using 29 of the 32 available logging categories (there are only 3 remaining). It would be good to increase the limit soon; the fourth PR to be merged that adds a new logging category will be blocked until something like this is done.

This PR also adds a `TEST` category that uses the new range (`1ULL << 63`) in case there's a hidden assumption somewhere that the `BCLog::LogFlags` type is 32 bits. (Also added a
...
🚀 fanquake merged a pull request: "doc: Update ccache website link"
(https://github.com/bitcoin/bitcoin/pull/30630)
💬 Sjors commented on pull request "Remove Taproot activation height":
(https://github.com/bitcoin/bitcoin/pull/26201#issuecomment-2283399297)
Rebased after #30560 and #29775.
💬 ismaelsadeeq commented on pull request "BlockAssembler: return selected packages vsize and feerate":
(https://github.com/bitcoin/bitcoin/pull/30391#discussion_r1713369779)
That makes sense. I removed `findFeeRate` and explicitly checked the vector for the packages and verify the fee and size were as expected. I also took the opportunity to change the variables to be initialized uniformly.
👍 willcl-ark approved a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625#pullrequestreview-2232381658)
crACK b0ec8716bf27335686471e0ae4c6a34f9a08f33c
🚀 fanquake merged a pull request: "doc: Remove outdated nTx faking comment"
(https://github.com/bitcoin/bitcoin/pull/30624)
🚀 fanquake merged a pull request: "doc: rpc: Use "output script" consistently (2/2)"
(https://github.com/bitcoin/bitcoin/pull/30524)
💬 fanquake commented on pull request "Fixes for GCC 15 compatibility":
(https://github.com/bitcoin/bitcoin/pull/30612#discussion_r1713492439)
> You all can do whatever you want with my suggested fixes.

Thanks. We don't edit commits during merges, so I'll cherry-pick and fixup any nits in a new PR.
📝 fanquake opened a pull request: "Fixes for GCC 15 compatibility"
(https://github.com/bitcoin/bitcoin/pull/30633)
#30612 with changes made.

GCC 15 introduces three build failures:

* Two are related to missing includes. You can't use `uint16_t` et al. without including `<cstdint>`.

* The third is harder to understand but easy to fix. GCC changed something about the way templates are instantiated when checking type constraints, and now there is a dependency loop while checking `std::optional<CFeeRate>`. This manifests as the following compile-time mess:
```
In file included from /usr/lib/gc
...
fanquake closed a pull request: "Fixes for GCC 15 compatibility"
(https://github.com/bitcoin/bitcoin/pull/30612)
💬 fanquake commented on pull request "Fixes for GCC 15 compatibility":
(https://github.com/bitcoin/bitcoin/pull/30612#issuecomment-2283597717)
See #30633.
💬 fanquake commented on pull request "Fixes for GCC 15 compatibility":
(https://github.com/bitcoin/bitcoin/pull/30612#discussion_r1713494639)
Done in #30633.
🚀 fanquake merged a pull request: "lint: Find function calls in default arguments"
(https://github.com/bitcoin/bitcoin/pull/30553)
🚀 fanquake merged a pull request: "doc: Drop no longer needed workaround for WSL"
(https://github.com/bitcoin/bitcoin/pull/30597)
🚀 fanquake merged a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625)
💬 fanquake commented on pull request "ci: Silent Homebrew's reinstall warnings":
(https://github.com/bitcoin/bitcoin/pull/30591#discussion_r1713500252)
Are these Qt workarounds still needed? Would be good if we could drop them, rather than adding more code to suppress warnings coming from workarounds that may no-longer be needed.
🤔 vasild reviewed a pull request: "log: expand BCLog::LogFlags (categories) to 64 bits"
(https://github.com/bitcoin/bitcoin/pull/26619#pullrequestreview-2232592770)
ACK 90c3d904bcbf294a6fcef8bc9ecb9445fd41c4bd

This probably needs a rebase. It is a trivial change.

10 new categories were added in the last 7 years. That is 1-2 per year. 32 extra should suffice for a few decades.
🚀 fanquake merged a pull request: "ci: Silent Homebrew's reinstall warnings"
(https://github.com/bitcoin/bitcoin/pull/30591)
👍 fanquake approved a pull request: "Revert "depends: Fetch miniupnpc sources from an alternative website""
(https://github.com/bitcoin/bitcoin/pull/30603#pullrequestreview-2232604323)
ACK 4c2cc63d84dc1c8bae1b620f34188b7c20b70019 - please don't @ mention anyone in your pull request descriptions, otherwise they are likely to just get spammed with notifications.
🚀 fanquake merged a pull request: "Revert "depends: Fetch miniupnpc sources from an alternative website""
(https://github.com/bitcoin/bitcoin/pull/30603)