💬 hebasto commented on pull request "build: use `-no_exported_symbols` on macOS":
(https://github.com/bitcoin/bitcoin/pull/29072#issuecomment-2240312745)
Ported to the CMake-based build system in https://github.com/hebasto/bitcoin/pull/271.

👍 theuni approved a pull request: "Fix MSVC warning C4273 "inconsistent dll linkage""
(https://github.com/bitcoin/bitcoin/pull/30491#pullrequestreview-2189430254)
Microsoft says on Windows it's coming from `Stdlib.h`, which something else must be including. (Well, Microsoft says it's called `_environ`, but I'm pretty sure the linker fixes that up for us).

From what I can tell, this will simply fail to link if it's not found _somewhere_. Also, this is just another opportunistic entropy source for us, so setting aside a Debian+OpenSSL-type whoopsie, I don't think it's a huge deal if this is null anyway.

So.. as long as this doesn't break anything, fi
...

💬 davidgumberg commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#discussion_r1685047415)
It seems to me the cache entry is not necessarily newly created here, if e.g. cache is not flushed and an entry is `DIRTY` and spent when a reorg takes place.

🤔 furszy reviewed a pull request: "index: Check all necessary block data is available before starting to sync"
(https://github.com/bitcoin/bitcoin/pull/29770#pullrequestreview-2189455441)
> But the index still needs to have an understanding of what block data is and what it needs in order to decide if it can handle what the the kernel gives it. So I don't think it can be avoided that the index has knowledge of the block status so this would need to be shared. That doesn't mean that they can't run isolation, but the kernel needs to share this knowledge with them. We can only avoid this if we forget about the concept suggested here and let the index ask for data from the kernel unt
...

💬 ryanofsky commented on pull request "refactor: Implement missing error checking for ArgsManager flags":
(https://github.com/bitcoin/bitcoin/pull/16545#issuecomment-2240599386)
re: https://github.com/bitcoin/bitcoin/pull/16545#issuecomment-2232487097

> Are you still working on this, or do you want someone to pick it up?

I would like code review, and I probably need to respond to some old comments and questions. There are two parts to this PR:

- The first part of this PR, implemented in the first commit, uses the ALLOW_BOOL, ALLOW_INT, and ALLOW_STRING flags to validate command line and config file settings, and save them into UniValue values as typed values, r
...

💬 hebasto commented on pull request "build: Introduce CMake-based buid system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1685163279)
This diff will gone after https://github.com/hebasto/bitcoin/pull/272.

💬 hebasto commented on issue "bump boost test to 1.59?":
(https://github.com/bitcoin/bitcoin/issues/19128#issuecomment-2240787323)
> To skip tests.

See https://github.com/hebasto/bitcoin/pull/272.

📝 FisZika opened a pull request: "Update translation_process.md"
(https://github.com/bitcoin/bitcoin/pull/30492)
B5690EEEBB952194

<!--
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test improvements or n
...

✅ achow101 closed a pull request: "Update translation_process.md"
(https://github.com/bitcoin/bitcoin/pull/30492)

📝 achow101 locked a pull request: "Update translation_process.md"
(https://github.com/bitcoin/bitcoin/pull/30492)
B5690EEEBB952194

<!--
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test improvements or n
...

📝 1440000bytes opened a pull request: "policy: enable full-rbf by default"
(https://github.com/bitcoin/bitcoin/pull/30493)
This pull request enables full rbf (mempool policy) by default. Commits are same as #28132 however it was closed recently with this [comment](https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-2225369634).

---

Rationale:

- Full RBF config option was added in July 2022: https://github.com/bitcoin/bitcoin/pull/25353

- It is used regularly: https://mempool.space/rbf#fullrbf

- Most mining pools are using it: https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-2059120917
...

💬 hebasto commented on pull request "build: Introduce CMake-based buid system":
(https://github.com/bitcoin/bitcoin/pull/30454#discussion_r1685289194)
Thanks! Fixed in https://github.com/hebasto/bitcoin/pull/273.

💬 hebasto commented on pull request "[DO NOT MERGE] cmake: Migrate CI scripts to CMake-based build system -- WIP":
(https://github.com/bitcoin/bitcoin/pull/29790#issuecomment-2240971961)
Temporarily reopening this PR to demonstrate the correctness of the https://github.com/hebasto/bitcoin/pull/273.

📝 hebasto reopened a pull request: "[DO NOT MERGE] cmake: Migrate CI scripts to CMake-based build system -- WIP"
(https://github.com/bitcoin/bitcoin/pull/29790)
This PR goal is to facilitate testing CI scripts migration to the CMake.

💬 BitcoinErrorLog commented on pull request "policy: enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/30493#issuecomment-2241046005)
mempoolfullrbf should be removed, and RBF is a redundant unenforceable feature. If we believe miners only care about the the largest fee/rate then simply providing them with all txns is good enough. Miners can choose how to sort themselves.

💬 hebasto commented on issue "Fuzzing related configuration/build options can be improved":
(https://github.com/bitcoin/bitcoin/issues/30318#issuecomment-2241046345)
> There are a few things that `--enable-fuzz` does that we definitely want it to do:
>
> * As you said, this option disables all other binaries besides the `fuzz` binary. This is nice when you only want to do fuzzing and nothing else (e.g. oss-fuzz). It is also required for the use of libFuzzer (`--with-sanitizers=fuzzer` i.e. `-fsanitize=fuzzer` with clang), because the other binaries provide a main function.

This functionality does not require any build options. The fuzz binary can be
...

💬 jrakibi commented on pull request "test, assumeutxo: Remove resolved todo comments and add new test":
(https://github.com/bitcoin/bitcoin/pull/30403#issuecomment-2241097572)
ACK [d63ef73](https://github.com/bitcoin/bitcoin/commit/d63ef738001fb69ce04134cc8645dcd1e1cbccd1)

💬 andrewtoth commented on pull request "policy: enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/30493#issuecomment-2241100701)
> If we believe miners only care about the the largest fee/rate then simply providing them with all txns is good enough. Miners can choose how to sort themselves.

This patch is necessary to provide miners with all txns. As of now if a node running default settings is between you and a miner, and you send it a txn not flagged as rbf enabled, that node will not relay another txn spending the same inputs as the first txn to the miner even if it pays a larger fee and fee rate. The miner will not be
...

💬 tdb3 commented on pull request "policy: enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/30493#issuecomment-2241107700)
Concept ACK

🚀 fanquake merged a pull request: "depends: bump libmultiprocess for CMake fixes"
(https://github.com/bitcoin/bitcoin/pull/30490)