📝 fanquake locked a pull request: "64bit arith"
(https://github.com/bitcoin/bitcoin/pull/29171)
https://github.com/bitcoin/bips/pull/1533

📝 fanquake locked a pull request: "[DONTMERGE] See if just constexpr->consteval for _mst works"
(https://github.com/bitcoin/bitcoin/pull/29167)

💬 hebasto commented on issue "Broken `--enable-suppress-external-warning` for Apple Clang 15 on `x86_64`":
(https://github.com/bitcoin/bitcoin/issues/29174#issuecomment-1877440314)
Here are more details:

1.
```
% clang -cc1 --help | grep -A 1 internal-isystem
-internal-isystem <directory>
Add directory to the internal system include search path; these are assumed to not be user-provided and are used to model system and standard headers' paths.
```


2.
> Is the issue here just the concatenation? Maybe that worked previously, but no-longer does? What happens if you change `-Xclang -internal-isystem/usr/local/include` to `-Xclang -inte
...

📝 glozow locked a pull request: "doc: revert clarify -datacarriersize"
(https://github.com/bitcoin/bitcoin/pull/29173)
The latest update of the help text of `-datacarriersize` is incorrect.

The purpose of this standardization rule is not to target only the data contained in the raw scriptPubKey, but all forms of arbitrary data.

It is incorrect to change the description of this option if an attempt to update it was made without being merged.

Context:

The [first inscription](https://mempool.space/tx/6fb976ab49dcec017f1e201e84395983204ae1a7c2abf7ced0a85d692e442799) appeared December 14, 2022 at the heig
...

🚀 glozow merged a pull request: "Update libsecp256k1 subtree to current master"
(https://github.com/bitcoin/bitcoin/pull/29169)

📝 theuni opened a pull request: "crypto: remove use of BUILD_BITCOIN_INTERNAL macro in sha256"
(https://github.com/bitcoin/bitcoin/pull/29180)
Replace it with a more explicit DISABLE_OPTIMIZED_SHA256 and clean up some.

The macro was originally only used by libbitcoinconsensus which opts out of optimized sha256 for the sake of simplicity. But now that libbitcoinkernel uses it as well, we need more fine-grained control.

As a result of this change, libbitcoinkernel's sha256 usage should now be optimized.

💬 theuni commented on pull request "crypto: remove use of BUILD_BITCOIN_INTERNAL macro in sha256":
(https://github.com/bitcoin/bitcoin/pull/29180#issuecomment-1877480455)
Ping @TheCharlatan @hebasto

I went around in circles several times on this. I think it accomplishes what we need, but please sanity check me :)

💬 hebasto commented on pull request "crypto: remove use of BUILD_BITCOIN_INTERNAL macro in sha256":
(https://github.com/bitcoin/bitcoin/pull/29180#discussion_r1442055466)
Is it needed to keep `-DBUILD_BITCOIN_INTERNAL` now?

💬 theuni commented on pull request "crypto: remove use of BUILD_BITCOIN_INTERNAL macro in sha256":
(https://github.com/bitcoin/bitcoin/pull/29180#discussion_r1442056422)
Yes, this macro is now exclusively used to control exports.

💬 furszy commented on issue "Wallets should update key/descriptor birthdates when txs older than current birthdates are found":
(https://github.com/bitcoin/bitcoin/issues/28897#issuecomment-1877494923)
Can be closed.

✅ maflcko closed an issue: "Wallets should update key/descriptor birthdates when txs older than current birthdates are found"
(https://github.com/bitcoin/bitcoin/issues/28897)

📝 fanquake opened a pull request: "build: remove systemtap variadic patch"
(https://github.com/bitcoin/bitcoin/pull/29181)
We now use C++20.

💬 theuni commented on pull request "crypto: remove use of BUILD_BITCOIN_INTERNAL macro in sha256":
(https://github.com/bitcoin/bitcoin/pull/29180#discussion_r1442077985)
Updated for @fanquake's request: it's now exclusively used to control libbitcoinconsensus exports. libbitcoinkernel will get a new define when the time comes.

📝 L0laL33tz opened a pull request: "test: randomize perturbed files excluding ldb"
(https://github.com/bitcoin/bitcoin/pull/29182)
As discussed in #28831 the ldb files are too small to be randomized for pertubation. This PR excludes ldb files from randomization. Blockfiles are randomly perturbed, ldb file pertubation remains deterministic. For additional rationale see #28612

🤔 ismaelsadeeq reviewed a pull request: "rpc: validate fee estimation mode case insensitive"
(https://github.com/bitcoin/bitcoin/pull/29175#pullrequestreview-1804697537)
Concept ACK

Thanks for trying to solve this issue.

This patch does not fix the issue.

Behaviour on master 65c05db660b2ca1d0076b0d8573a6760b3228068
```terminal
abubakarismail@Abubakars-MacBook-Pro bitcoin % ./src/bitcoind -regtest -fallbackfee=0.0000100 -daemon
Bitcoin Core starting
abubakarismail@Abubakars-MacBook-Pro bitcoin % ./src/bitcoin-cli -regtest loadwallet abubakar-test
{
"name": "abubakar-test"
}
abubakarismail@Abubakars-MacBook-Pro bitcoin % ./src/bitcoin-cli -reg
...

💬 theuni commented on pull request "refactor: C++20: Use std::rotl":
(https://github.com/bitcoin/bitcoin/pull/29085#issuecomment-1877527471)
Any reason not to do `Rotl` in sha3 and `ROTL` in siphash as well?

💬 jonatack commented on pull request "Add OP_TXHASH and OP_CHECKTXHASHVERIFY opcodes":
(https://github.com/bitcoin/bitcoin/pull/29050#discussion_r1442123784)
A working build and green CI may help attract review. The last commit https://github.com/bitcoin/bitcoin/pull/29050/commits/c368d32588ff79efc189fbdcfe559d1ca081c36c doesn't build for me without updating the related fuzz tests.

<details><summary>sample diff</summary><p>

```diff
diff --git a/src/test/fuzz/script_assets_test_minimizer.cpp b/src/test/fuzz/script_assets_test_minimizer.cpp
index 511b581f606..7478897e170 100644
--- a/src/test/fuzz/script_assets_test_minimizer.cpp
+++ b/src/t
...

🤔 1ma reviewed a pull request: "datacarriersize: Match more datacarrying"
(https://github.com/bitcoin/bitcoin/pull/28408#pullrequestreview-1804807600)
In order to be consistent with the documentation of `datacarriersize`, this PR needs to roll back the changes recently made in `src/init.cpp` by this other PR: https://github.com/bitcoin/bitcoin/pull/27832

Not sure about the tests it added.

🤔 furszy reviewed a pull request: "Compressed Bitcoin Transactions"
(https://github.com/bitcoin/bitcoin/pull/29134#pullrequestreview-1804829839)
You could run the lint whitespace task locally. It is inside test/lint folder.

💬 0xB10C commented on pull request "build: remove systemtap variadic patch":
(https://github.com/bitcoin/bitcoin/pull/29181#issuecomment-1877675266)
ACK. That patch isn't needed anymore.