📝 laanwj opened a pull request: "net: Replace ifname check with IFF_LOOPBACK in Discover"
(https://github.com/bitcoin/bitcoin/pull/29984)
Checking the interface name is kind of brittle. In the age of network namespaces and containers, there is no reason a loopback interface can't be called differently.

Check for the `IFF_LOOPBACK` flag to detect loopback interface instead.

💬 laanwj commented on issue ""Migrate Wallet" is unclear to translators":
(https://github.com/bitcoin/bitcoin/issues/29979#issuecomment-2081416263)
A better user-facing word would be just "upgrade wallet"--but for technical reasons we use a different word, as we already had a concept of upgrading inside the wallet format.

💬 laanwj commented on issue "Add IPv6 pinhole support using UPnP / NAT-PMP":
(https://github.com/bitcoin/bitcoin/issues/17012#issuecomment-2081424660)
The UPnP side of this is progressing, though i've ran into a possible bug/limitaton in miniupnp: https://github.com/miniupnp/miniupnp/issues/731#issuecomment-2081257515

> Meanwhile NAT-PMP support is there, but it has no IPv6 pinhole support.

i looked into this too, we'll actually have to implement PCP support ourselves to do this. The good part is that it's just a matter of sending one fixed-size binary UDP packet to the default gateway and parsing the result. Not worth taking on a depend
...

💬 laanwj commented on pull request "depends: Remove Qt build-time dependencies":
(https://github.com/bitcoin/bitcoin/pull/29923#issuecomment-2081443414)
Looks like we could drop 'bison' as a dependency here, it was used by the `xkbcommon` build which is dropped.

📝 laanwj opened a pull request: "depends: Fix build of Qt for 32-bit platforms"
(https://github.com/bitcoin/bitcoin/pull/29985)
The 32 to 64-bit `time_t` transition causes a build failure in the built-in gzip about conflicting `_TIME_BITS` and `_FILE_OFFSET_BITS`.

Note that gzip doesn't use time_t at all, so it is a false alarm.

Take the following patch from upstream gzip:
https://github.com/madler/zlib/commit/a566e156b3fa07b566ddbf6801b517a9dba04fa3.patch

Closes #29980.

💬 hebasto commented on pull request "depends: Fix build of Qt for 32-bit platforms":
(https://github.com/bitcoin/bitcoin/pull/29985#issuecomment-2081445659)
Concept ACK. I was thinking about picking up the same patch :)

🤔 hebasto reviewed a pull request: "depends: Fix build of Qt for 32-bit platforms with recent glibc"
(https://github.com/bitcoin/bitcoin/pull/29985#pullrequestreview-2026996605)
Approach ACK e4c1a0c62900e8e560b99ee2749460c15355c556, reviewed and tested on Ubuntu 24.04.

I'm going to submit Guix build hashes shortly.

💬 laanwj commented on pull request "depends: Fix build of Qt for 32-bit platforms with recent glibc":
(https://github.com/bitcoin/bitcoin/pull/29985#discussion_r1582109676)
whoops, should probably do s/gzip/zlib here too

💬 laanwj commented on pull request "depends: Fix build of Qt for 32-bit platforms with recent glibc":
(https://github.com/bitcoin/bitcoin/pull/29985#issuecomment-2081463967)
Force-pushed because somehow had gzip and zlib names mixed up in my head, only the branch name remains 😄

💬 theStack commented on pull request "refactor: remove remaining unused code from cpp-subprocess":
(https://github.com/bitcoin/bitcoin/pull/29961#discussion_r1582144839)
Oh sorry, I missed that. Force-pushed now with a commit where only `Popen::kill()` is removed (and `Popen::poll()` is kept as-is).

💬 sipa commented on pull request "net: Replace ifname check with IFF_LOOPBACK in Discover":
(https://github.com/bitcoin/bitcoin/pull/29984#issuecomment-2081471875)
utACK a68fed111be393ddbbcd7451f78bc63601253ee0

📝 sdaftuar opened a pull request: "test: Don't rely on incentive incompatible replacement in mempool_accept_v3.py"
(https://github.com/bitcoin/bitcoin/pull/29986)
In the sibling eviction test, we're currently testing that a transaction with ancestor feerate of 179 is able to replace a transaction with ancestor feerate of 300, due to a shortcoming in our current RBF rules.

In preparation for fixing our RBF rules to not allow such replacements, fix the test by bumping the fee of the replacement to be a bit higher.

💬 sdaftuar commented on pull request "test: Don't rely on incentive incompatible replacement in mempool_accept_v3.py":
(https://github.com/bitcoin/bitcoin/pull/29986#issuecomment-2081474305)
@glozow @instagibbs This was something I noticed when rebasing #28676.

📝 fanquake opened a pull request: "guix: build with glibc 2.31"
(https://github.com/bitcoin/bitcoin/pull/29987)
Set minimum required glibc to 2.31.
The glibc 2.31 branch is still maintained: https://sourceware.org/git/?p=glibc.git;a=shortlog;h=refs/heads/release/2.31/master.

Remove the stack-protector check from test-security-check, as the test
no-longer fails, and given the control we have of the end, the actual
security-check test seems sufficient (this might also be applied to some
of the other checks).

💬 laanwj commented on pull request "guix: build with glibc 2.31":
(https://github.com/bitcoin/bitcoin/pull/29987#issuecomment-2081481022)
Concept ACK.

For context: this is the version in Ubuntu 20.04 LTS: https://packages.ubuntu.com/search?keywords=libc6&searchon=names&suite=focal&section=all

💬 fjahr commented on pull request "refactor: remove remaining unused code from cpp-subprocess":
(https://github.com/bitcoin/bitcoin/pull/29961#discussion_r1582161938)
Ok, strange, I guess I was looking at an outdated version of the code or something.

💬 fjahr commented on pull request "refactor: remove remaining unused code from cpp-subprocess":
(https://github.com/bitcoin/bitcoin/pull/29961#issuecomment-2081495482)
> ACK [908c51f](https://github.com/bitcoin/bitcoin/commit/908c51fe4afeba0af500c6275027b1afa1b3bd19). It is compatible with #29961.

@hebasto #29961 is this PR here, did you mean #29868 there?

💬 fjahr commented on pull request "refactor: remove remaining unused code from cpp-subprocess":
(https://github.com/bitcoin/bitcoin/pull/29961#issuecomment-2081495688)
Code review ACK 8b52e7f628304e83b0e36fd97e617de0f71c5a62

💬 eval-exec commented on issue "qa: Support git worktrees when running the linters locally via Docker":
(https://github.com/bitcoin/bitcoin/issues/29972#issuecomment-2081499778)
I can't reproduce that error:
```
bitcoin on  exec/worktree via 🐍 v3.8.18 via ❄️ impure (btc)
❯ DOCKER_BUILDKIT=1 docker build -t bitcoin-linter --file "./ci/lint_imagefile" ./ && docker run --rm -v $(pwd):/bitcoin -it bitcoin-linter
[+] Building 3.4s (12/12) FINISHED docker:default
=> [internal] load build definition from lint_imagefile
...

📝 Apetree100122 opened a pull request: "Update bitcoin_config.h.in"
(https://github.com/bitcoin/bitcoin/pull/29988)
<!--
*** 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 new tests that improve cove
...