💬 maflcko commented on pull request "ci: Drop duplicated compiler flags":
(https://github.com/bitcoin/bitcoin/pull/29800#issuecomment-2040027759)
re-ACK a3485af67da4949c72c45acc608f8746ed0e0848

💬 fjahr commented on pull request "clang-tidy: Enable misc-no-recursion":
(https://github.com/bitcoin/bitcoin/pull/29690#discussion_r1553824794)
That framing makes a lot more sense to me 👍

💬 vostrnad commented on issue "IBD performance regression in 27.0rc1 on Windows":
(https://github.com/bitcoin/bitcoin/issues/29785#issuecomment-2040045243)
@maflcko Just to clarify, what didn't regress was the release binary for Linux (which I assume is built with guix, not WSL) running in WSL.

💬 fanquake commented on issue "IBD performance regression in 27.0rc1 on Windows":
(https://github.com/bitcoin/bitcoin/issues/29785#issuecomment-2040086861)
> which I assume is built with guix

All of the release binaries are built using Guix. Windows is produced in Guix using GCC+Mingw-w64. We don't produce any release binaries using WSL or MSVC.

💬 fanquake commented on pull request "clang-tidy: Enable misc-no-recursion":
(https://github.com/bitcoin/bitcoin/pull/29690#issuecomment-2040102948)
Concept ACK. I think tracking where any recursion currently is, is useful, as well as being alerted to new usage in the future.

👍 fanquake approved a pull request: "refactor: Remove gmtime*"
(https://github.com/bitcoin/bitcoin/pull/29081#pullrequestreview-1983608799)
ACK fa9f36babaceba6ab2f88e64bc4bc2956f58871f - more std lib & even less stuff to port.

🚀 fanquake merged a pull request: "refactor: Remove gmtime*"
(https://github.com/bitcoin/bitcoin/pull/29081)

💬 ismaelsadeeq commented on pull request "Wallet: Add `max_tx_weight` to transaction funding options (take 2)":
(https://github.com/bitcoin/bitcoin/pull/29523#issuecomment-2040171293)
> Any chance to turn the first commit into an scripted-diff?

The commit message was misleading It's not just renaming `max_weight`, it also updates the input parameter docstring to match match the new name.
And also the fuzz test renames has more info like `low_max_selection_weight` and `high_max_selection_weight`.

I've updated the commit message of the first commit https://github.com/bitcoin/bitcoin/pull/29523/commits/270639324a6637e0025e1458221ab488a8a09554

👍 fanquake approved a pull request: "ci: Drop duplicated compiler flags"
(https://github.com/bitcoin/bitcoin/pull/29800#pullrequestreview-1983625214)
ACK a3485af67da4949c72c45acc608f8746ed0e0848 - no-longer a change in behaviour.

💬 fanquake commented on pull request "[WIP] ci: test secp256k1 MSAN asm annotations":
(https://github.com/bitcoin/bitcoin/pull/29742#discussion_r1553941342)
Will be done via #29800.

💬 Sjors commented on pull request "depends: suggest GNU patch for macOS <= 13":
(https://github.com/bitcoin/bitcoin/pull/29814#issuecomment-2040173278)
> Seems weird that this suddenly pops up now after having worked for a long time?

I agree. It's worked fine for me too on this machine back in the day, say 2017-2019. Replacing the built-in patch with GNU patch immediately fixed it, so it does seem _related_.

💬 fanquake commented on pull request "depends: suggest GNU patch for macOS <= 13":
(https://github.com/bitcoin/bitcoin/pull/29814#issuecomment-2040178090)
> say 2017-2019.

So it stopped working in 2019, but you only reported it in 2024?

Given that there are 3 machines here, 2 where it always has worked, and continues to work, and one where it used to work, and now no-longer does, it would seem more likely that something has changed on your machine?

📝 theuni opened a pull request: "crypto: chacha20: always use our fallback timingsafe_bcmp rather than libc's"
(https://github.com/bitcoin/bitcoin/pull/29815)
Looking at libc sources, apple and openbsd implementations match our naive fallback. Only FreeBSD (and only x86_64) seems to [implement an optimized version](https://github.com/freebsd/freebsd-src/blob/main/lib/libc/amd64/string/timingsafe_bcmp.S).

It's not worth the hassle of using a platform-specific function for such little gain.

Additionally, as mentioned below, this is the only case outside of sha2 that requires an autoconf check, and I have upcoming PRs to remove the sha2 ones.



...

💬 Sjors commented on pull request "depends: suggest GNU patch for macOS <= 13":
(https://github.com/bitcoin/bitcoin/pull/29814#issuecomment-2040180002)
> So it stopped working in 2019, but you only reported it in 2024?

No, I bought a faster machine in 2019 which I've been using for most compilation work. I may have run a depends build on the old machine since then, but I know when the last time was.

💬 theuni commented on pull request "crypto: chacha20: always use our fallback timingsafe_bcmp rather than libc's":
(https://github.com/bitcoin/bitcoin/pull/29815#issuecomment-2040180394)
Ping @sipa for a quick concept ACK.

📝 Sjors converted_to_draft a pull request: "depends: suggest GNU patch for macOS <= 13"
(https://github.com/bitcoin/bitcoin/pull/29814)
Fixes #29792

Adds a note to `depends/README.md` that on older versions of macOS you _may_ need to use GNU patch. I phrased as a troubleshooting hint, rather than make it a default recommendation, because:

1. so far only I ran into it, and fanquake couldn't reproduce
2. BDB is only used for the legacy wallet, which itself is deprecated
3. it seems better to replace as few default tools on macOS as possible

The instructions in `build-osx.md` don't even mention depends, so I don't touch
...

🚀 fanquake merged a pull request: "ci: Drop duplicated compiler flags"
(https://github.com/bitcoin/bitcoin/pull/29800)

💬 theuni commented on pull request "refactor: Remove gmtime*":
(https://github.com/bitcoin/bitcoin/pull/29081#issuecomment-2040186745)
Post-merge utACK fa9f36babaceba6ab2f88e64bc4bc2956f58871f

💬 Sjors commented on pull request "depends: suggest GNU patch for macOS <= 13":
(https://github.com/bitcoin/bitcoin/pull/29814#issuecomment-2040194182)
On both machines:

```
/usr/bin/patch --version
patch 2.0-12u11-Apple
```

On the old machine:

```
shasum -a 256 /usr/bin/patch
1c7648162c11d32aa0ee1a05a948a8a548e4a9336106704a43ab986f718efb88 /usr/bin/patch
```

On the new machine:

```
shasum -a 256 /usr/bin/patch
214df67037191e570a95ddfe1427ebaad7c756f53348584d981759309d3dd489 /usr/bin/patch
```

Both are Intel.

💬 kosuodhmwa commented on issue "Tons of Socks5() connect to x.x.x.x:8333 failed: connection refused-messages when i use TOR - why?":
(https://github.com/bitcoin/bitcoin/issues/29759#issuecomment-2040196739)