💬 hebasto commented on pull request "build: Bump minimum supported macOS to 13.0":
(https://github.com/bitcoin/bitcoin/pull/31048#issuecomment-2413356842)
> > > Running Bitcoin Core on unsupported OSes may expose users to security issues.
> > > macOS Big Sur 11 received its last security update ([11.7.10](https://support.apple.com/en-us/106338)) over a year ago.
> >
> >
> > If this is the reasoning we are going to use, shouldn't this be bumping to `10.13`, given `10.12` is also "unsupported" in terms of security updates?
>
> Yes, good point. I didn't realize 12.0 was eol'd. We shouldn't let qt dictate what we support.
>
> Agree that 13
...

💬 maflcko commented on pull request "build: Bump minimum supported macOS to 13.0":
(https://github.com/bitcoin/bitcoin/pull/31048#issuecomment-2413362340)
lgtm ACK a0e089a71dc449f4cc177d6eb3050400e63f4b3f

💬 maflcko commented on issue "wallet: rpc: `settxfee` sets the wallet feerate not fee":
(https://github.com/bitcoin/bitcoin/issues/31088#issuecomment-2413394142)
In theory, it would be possible to register RPC argument name aliases, IIRC. They are denoted and split by `|`.

💬 m3dwards commented on pull request "build: Bump minimum supported macOS to 13.0":
(https://github.com/bitcoin/bitcoin/pull/31048#issuecomment-2413396346)
ACK a0e089a71dc449f4cc177d6eb3050400e63f4b3f

Tested on ARM Mac 14.6

⚠️ maflcko opened an issue: "doc: Fixup windows-cross build notes"
(https://github.com/bitcoin/bitcoin/issues/31090)
It is also missing bison and patch? Might as well remove it and just refer to the location that is correct?

_Originally posted by @maflcko in https://github.com/bitcoin/bitcoin/pull/31084#discussion_r1799629690_

✅ stickies-v closed an issue: "test: C++ debugger interferes with functional tests that quickly stop/start nodes"
(https://github.com/bitcoin/bitcoin/issues/23651)

💬 stickies-v commented on issue "test: C++ debugger interferes with functional tests that quickly stop/start nodes":
(https://github.com/bitcoin/bitcoin/issues/23651#issuecomment-2413408647)
I'm going to close this issue, if anyone is interested in fixing this please feel free to do so. The issue still persists for me, but it's quite an edge case, I've not ran into anymore since reporting this almost three years ago.

I explored opening a PR for it using the below diff that forces removal of the `.lock` file, but it doesn't quite seem to fix things for me, and I don't feel like spending more time on it at the moment. I'd be happy to review a PR that addresses this, though.

<det
...

✅ stickies-v closed an issue: "test: C++ debugger interferes with functional tests that quickly stop/start nodes"
(https://github.com/bitcoin/bitcoin/issues/23651)

💬 itornaza commented on pull request "build: Bump minimum supported macOS to 13.0":
(https://github.com/bitcoin/bitcoin/pull/31048#issuecomment-2413410099)
reACK a0e089a71dc449f4cc177d6eb3050400e63f4b3f

👍 itornaza approved a pull request: "build: Bump minimum supported macOS to 13.0"
(https://github.com/bitcoin/bitcoin/pull/31048#pullrequestreview-2368758005)
reACK a0e089a71dc449f4cc177d6eb3050400e63f4b3f

💬 maflcko commented on issue "Default -rpcclienttimeout is too low for calls which cause rescan":
(https://github.com/bitcoin/bitcoin/issues/9918#issuecomment-2413423092)
> 3\. Add a warning in the help output for calls which trigger a rescan about the default -rpcclienttimeout being too low.

Seems like this could be an easy doc-only fix?

👍 hodlinator approved a pull request: "bench: add support for custom data directory"
(https://github.com/bitcoin/bitcoin/pull/31000#pullrequestreview-2368774548)
ACK 15cfeebb47587af6ce7be72fd52c57a0483d86d2

### Concept

Thanks @furszy for making the dependency explicit! I don't fully understand the upper/lower level argument as bench_bitcoin has a link dependency on test_util already.

Using ENV-vars instead as suggested in [other comment](https://github.com/bitcoin/bitcoin/pull/31000#discussion_r1800757127) seems too sneaky for me, and contradicts my goal of having things explicit. (IMO one original sin of C was `main` not taking environment args
...

👍 brunoerg approved a pull request: "doc: add doxygen for m_args in tests"
(https://github.com/bitcoin/bitcoin/pull/31083#pullrequestreview-2368790724)
ACK 1fe1b3ba8e9e73987a12fbd9eab7a29459d9f14c

✅ maflcko closed an issue: "Unable to sync blockchain on laptop: ERROR: ReadBlockFromDisk: Deserialize or I/O error"
(https://github.com/bitcoin/bitcoin/issues/29255)

💬 maflcko commented on issue "Unable to sync blockchain on laptop: ERROR: ReadBlockFromDisk: Deserialize or I/O error":
(https://github.com/bitcoin/bitcoin/issues/29255#issuecomment-2413443757)
> It must be a hardware issue, as no one else experiences this issue.

Ok, closing for now, but please do provide a comment with more details, if you find anything.

💬 hebasto commented on pull request "build: Bump minimum supported macOS to 13.0":
(https://github.com/bitcoin/bitcoin/pull/31048#issuecomment-2413448037)
My Guix build:
```
aarch64
11dcf8e401c3cf7a911bf08cf801d354b53d8f4266b139625ef12f502050ee4f guix-build-a0e089a71dc4/output/arm64-apple-darwin/SHA256SUMS.part
d00bfbd05b870fdd4a1b7626bc0e851a88a3b958a1ff78d6822ed85f743c7c8a guix-build-a0e089a71dc4/output/arm64-apple-darwin/bitcoin-a0e089a71dc4-arm64-apple-darwin-unsigned.tar.gz
fc0dcd590d775f6e565ec6d7a005e525891b90b5afdd1f6d429d9f615792ca2b guix-build-a0e089a71dc4/output/arm64-apple-darwin/bitcoin-a0e089a71dc4-arm64-apple-darwin-unsigned
...

✅ maflcko closed an issue: "LevelDB read failure: Corruption: block checksum mismatch"
(https://github.com/bitcoin/bitcoin/issues/30159)

💬 maflcko commented on issue "LevelDB read failure: Corruption: block checksum mismatch":
(https://github.com/bitcoin/bitcoin/issues/30159#issuecomment-2413449154)
Closing for now due to inactivity and because I couldn't reproduce this so far.

If you have *exact* and *full* steps to reproduce, starting from a fresh install of the AWS machine, with all settings, etc, I can take another look.

In the meantime, closing for now, but please do provide a comment with more details, if you find anything.

✅ maflcko closed an issue: "Fatal LevelDB error: Corruption: block checksum mismatch on Linux ext4 SATA SSDs"
(https://github.com/bitcoin/bitcoin/issues/30692)

💬 maflcko commented on issue "Fatal LevelDB error: Corruption: block checksum mismatch on Linux ext4 SATA SSDs":
(https://github.com/bitcoin/bitcoin/issues/30692#issuecomment-2413450378)
Closing for now due to inactivity, but please do provide a comment with more details, if you find anything.