💬 TheCharlatan commented on pull request "[refactor] Check CTxMemPool options in ctor":
(https://github.com/bitcoin/bitcoin/pull/28830#issuecomment-1857559238)
Rebased 3aa41a31690da2ae22b6a15a9a5de0de78a01757 -> 571fa4f4c5c9792457a8a6ec77da980ae2d239e7 ([mempoolArgs_2](https://github.com/TheCharlatan/bitcoin/tree/mempoolArgs_2) -> [mempoolArgs_3](https://github.com/TheCharlatan/bitcoin/tree/mempoolArgs_3), [compare](https://github.com/TheCharlatan/bitcoin/compare/mempoolArgs_2..mempoolArgs_3))

Updated 571fa4f4c5c9792457a8a6ec77da980ae2d239e7 -> 364456f6598b135fcc0acab8a660658b4407f837 ([mempoolArgs_3](https://github.com/TheCharlatan/bitcoin/tree/me
...

💬 fanquake commented on pull request "refactor: C++20: Use lambda implicit capture and std::rotl":
(https://github.com/bitcoin/bitcoin/pull/29085#issuecomment-1857585948)
```bash
txmempool.cpp:88:61: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture]
88 | mapTx.modify(mapTx.iterator_to(descendant), [=, this](CTxMemPoolEntry& e) {
| ~~^~~~
txmempool.cpp:99:32: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture]
99 | mapTx.modify(updateIt, [=, this](CTxMemPoolEntry& e) { e.UpdateDescendantState(modifySize, modifyFee, modifyCount)
...

⚠️ fanquake opened an issue: "ci: feature_proxy failing in MSVC job"
(https://github.com/bitcoin/bitcoin/issues/29090)
https://github.com/bitcoin/bitcoin/actions/runs/7219794888/job/19671475361?pr=29088#step:27:73:
```bash
288/288 - feature_proxy.py failed, Duration: 2410 s

stdout:
2023-12-15T08:53:02.026000Z TestFramework (INFO): PRNG seed is: 197024891591373016

2023-12-15T08:53:02.026000Z TestFramework (INFO): Initializing test directory D:\a\_temp\test_runner_₿_🏃_20231215_084456\feature_proxy_168

2023-12-15T09:33:11.259000Z TestFramework (ERROR): Assertion failed

Traceback (most recent call la
...

💬 fanquake commented on pull request "Revert "ci: Only run functional tests on windows in master"":
(https://github.com/bitcoin/bitcoin/pull/29059#issuecomment-1857593249)
Looks like these are still just randomly failing: #29090 ?

💬 0xB10C commented on pull request "Update security.md contact for achow101":
(https://github.com/bitcoin/bitcoin/pull/29087#issuecomment-1857601824)
ACK e7d66509dc393400079eb885a497dd4e968eade6

💬 hebasto commented on pull request "Revert "ci: Only run functional tests on windows in master"":
(https://github.com/bitcoin/bitcoin/pull/29059#issuecomment-1857605559)
> Looks like these are still just randomly failing: #29090 ?

Perhaps, it depends on the image version? That one was `20231211.1.0`.

💬 fanquake commented on pull request "Revert "ci: Only run functional tests on windows in master"":
(https://github.com/bitcoin/bitcoin/pull/29059#issuecomment-1857610650)
I don't think so. The last run on master passed with that image version: https://github.com/bitcoin/bitcoin/actions/runs/7214949583/job/19658167262.

🚀 fanquake merged a pull request: "Update security.md contact for achow101"
(https://github.com/bitcoin/bitcoin/pull/29087)

🤔 fanquake reviewed a pull request: "refactor: Remove gmtime*"
(https://github.com/bitcoin/bitcoin/pull/29081#pullrequestreview-1783532089)
Concept ACK - can we also drop the Boost date_time usage from `ParseISO8601DateTime` in favour of the std lib?

💬 fanquake commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#discussion_r1427805113)
Can also drop `AC_SUBST(HAVE_GMTIME_R)`.

✅ fanquake closed an issue: "Bitcoin's systemic flaw"
(https://github.com/bitcoin/bitcoin/issues/29089)

💬 fanquake commented on issue "Bitcoin's systemic flaw":
(https://github.com/bitcoin/bitcoin/issues/29089#issuecomment-1857627304)
This would be better suited as a post to the mailing list. This issue tracker is used to track technical issues relating to the Bitcoin Core code base. General bitcoin questions and/or support requests are best directed to the [Bitcoin StackExchange](https://bitcoin.stackexchange.com) or the `#bitcoin` IRC channel on freenode.

💬 fanquake commented on pull request "wallet: birth time update during tx scanning":
(https://github.com/bitcoin/bitcoin/pull/28920#issuecomment-1857652507)
Adding to #29011 for backporting to 26.x.

💬 maflcko commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#discussion_r1427839317)
Thx, done

💬 maflcko commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#issuecomment-1857683898)
> Concept ACK - can we also drop the Boost date_time usage from `ParseISO8601DateTime` in favour of the std lib?

Probably no. `std::chrono::parse` isn't released yet in gcc at all, or did you mean something else?

💬 fanquake commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#issuecomment-1857693940)
> Probably no. std::chrono::parse isn't released yet in gcc at all, or did you mean something else?

Damn. I just meant any possibility to get rid of Boost.

💬 eriknylund commented on pull request "test: verify spend from 999-of-999 taproot multisig wallet":
(https://github.com/bitcoin/bitcoin/pull/28212#discussion_r1427850857)
I'll lower it to 100 unless @achow101 thinks the `send` RPC approach would work efficiently here.

👍 maflcko approved a pull request: "logging: Simplify API for level based logging"
(https://github.com/bitcoin/bitcoin/pull/28318#pullrequestreview-1783613918)
re-ACK 16dd9042ea81412607c0645e4a3cc152abdc4d9a 😹

<details><summary>Show signature</summary>

Signature:

```
untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
trusted comment: re-ACK 16dd9042ea81412607c0
...

💬 maflcko commented on pull request "logging: Simplify API for level based logging":
(https://github.com/bitcoin/bitcoin/pull/28318#discussion_r1427853723)
nit (same): "with the startup options `-debug...`."

💬 maflcko commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#issuecomment-1857706842)
> > Probably no. std::chrono::parse isn't released yet in gcc at all, or did you mean something else?
>
> Damn. I just meant any possibility to get rid of Boost.

At least this one is header-only in a single translation unit, so should be fine to stay around for another 3 years