📝 fjahr opened a pull request: "test: Remove dead code from interface_zmq test"
(https://github.com/bitcoin/bitcoin/pull/30942)
The loop removed here appears to be effectively dead code: In case `get_raw_seq` is behind `zmq_mem_seq` the loop runs and tries to get a more recent (higher) number for `get_raw_seq`. However, the exact number of `get_raw_seq` is asserted in the line above: `assert_equal(get_raw_seq, 6)`. If the loop would actually achieve its purpose this assert would need to be racy. This does not seem to be the case and 6 appears to be the final number. `zmq_mem_seq` however does take some time to catch up (
...

💬 hebasto commented on pull request "depends: Fix build with `MULTIPROCESS=1` in Guix environment":
(https://github.com/bitcoin/bitcoin/pull/30940#issuecomment-2365310194)
@Sjors
> can you rebase your `240921-guix-mp.DEMO` branch so it builds directly on this PR? That makes it easier to compare guix builds.

Done. The PR description has been updated accordingly.

💬 kegdeg commented on issue "rpc auth fails 'Error parsing command line arguments: Invalid parameter -rpcpasssword=password":
(https://github.com/bitcoin/bitcoin/issues/30939#issuecomment-2365310228)
changed it to ./bitcoin-cli -rpcuser=user -stdinrpcpass -netinfo, 'error: Authorization failed: Incorrect rpcuser or rpcpassword

Also trying:
curl --user user:password --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getblockchaininfo", "params": [] }' -H 'content-type: text/plain;' http://127.0.0.1:8332/

the server shows
2024-09-21T20:21:44Z ThreadRPCServer incorrect password attempt from 127.0.0.1:44938

💬 fjahr commented on pull request "test: Introduce ensure helper":
(https://github.com/bitcoin/bitcoin/pull/30893#discussion_r1769650699)
> hmm, so you are thinking about a condition that passes at the beginning, does not passes somewhere during the test sleep time and ends up passing at the end. Maybe the test condition should be changed if such scenario exist?

Yes, exactly, and in order to know that it should be changed the person writing the test will see that `ensure_for` fails. This is the protection against introducing flakiness that I mentioned.

> which would flood the active logs

I don't know if that's a problem b
...

💬 fjahr commented on pull request "test: Introduce ensure helper":
(https://github.com/bitcoin/bitcoin/pull/30893#issuecomment-2365313958)
I have introduced a delay parameter that allows to adjust the delay between checking the predicate. The default is set at 0.2 instead of previously 0.05.

I have also added a second commit adding the delay parameter to `wait_until` and using it in places where a `time.sleep` is currently used and a high frequency of checking does not seem appropriate.

There are still a few sleeps spread across the tests, I think some of them require some further thought on how to remove them and some of the
...

⚠️ agzht opened an issue: "bitcoin core发送后，链上没有"
(https://github.com/bitcoin/bitcoin/issues/30943)
### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current behaviour

用bitcoin core 發送比特幣後,比特幣余额扣了，
但blockchain 找不到這交易

### Expected behaviour

退回bitcoin

### Steps to reproduce

运行bitcoin core

### Relevant log output

_No response_

### How did you obtain Bitcoin Core

Other

### What version of Bitcoin Core are you using?

25

### Operating system and version

window

### Machine specifications

thinkpad

✅ pinheadmz closed an issue: "bitcoin core发送后，链上没有"
(https://github.com/bitcoin/bitcoin/issues/30943)

⚠️ Peroqu opened an issue: "Minar"
(https://github.com/bitcoin/bitcoin/issues/30944)

💬 fjahr commented on pull request "sync: improve CCoinsViewCache ReallocateCache - 2nd try":
(https://github.com/bitcoin/bitcoin/pull/30370#issuecomment-2365335400)
> this seems to completely contradict what ReallocateCache was introduced in the first place

Can you explain what you mean by this? I don't understand how making it optional contradicts it.

💬 l0rinc commented on pull request "sync: improve CCoinsViewCache ReallocateCache - 2nd try":
(https://github.com/bitcoin/bitcoin/pull/30370#issuecomment-2365336998)
> Can you explain what you mean by this

I expanded on it in https://github.com/bitcoin/bitcoin/pull/30370#discussion_r1753749012.

My understanding is that `ReallocateCache` was introduced to save memory when shrinking the map after evictions, and this PR prohibits shrinking and reallocates the same capacity we were just trying to get rid of. If I'm mistaken here, I'd appreciate and explanation for why we can't solve it in a simpler way - e.g. by getting rid of `ReallocateCache` instead or
...

💬 KristijanSajenko commented on pull request "sync: improve CCoinsViewCache ReallocateCache - 2nd try":
(https://github.com/bitcoin/bitcoin/pull/30370#issuecomment-2365353646)
Thy

✅ willcl-ark closed an issue: "Minar"
(https://github.com/bitcoin/bitcoin/issues/30944)

💬 melvincarvalho commented on pull request "Testnet4 including PoW difficulty adjustment fix":
(https://github.com/bitcoin/bitcoin/pull/29775#issuecomment-2365976866)
Sorry if this is the wrong spot for feedback. Post-merge, Testnet4 seems solid overall. One issue: difficulty appears to be rising exponentially, likely due to someone spoofing timestamps and pushing in 6 blocks every 20 minutes. I could be off here, but if difficulty keeps spiking, mining might become impractical, making reorgs easier. A possible fix: setting up a pool that builds on genuine 20-minute blocks (not spoofed ones), and having some miners point their hashpower to it.

💬 David79-bot commented on pull request "Stratum v2 Template Provider (take 3)":
(https://github.com/bitcoin/bitcoin/pull/29432#issuecomment-2366210368)
Hi

📝 tsnob opened a pull request: "doc: fix broken link to release"
(https://github.com/bitcoin/bitcoin/pull/30945)
Description:
While checking the previous releases, I figure out that the link to release 0.10.0 is broken.
The link is updated now with the correct address.

No additional test is needed because there was no code change

💬 l0rinc commented on pull request "doc: fix broken link to release":
(https://github.com/bitcoin/bitcoin/pull/30945#discussion_r1770365385)
Shouldn't this pont to `https://bitcoin.org/bin/insecure/bitcoin-core-0.10.0` instead?

Please fix the rest of them as well with a scripted diff:
```bash
-BEGIN VERIFY SCRIPT-
sed -i --regexp-extended 's#(https://bitcoin\.org/bin/)([.0-9]+)#\1insecure/bitcoin-core-\2#g' ./doc/release-notes/*.md
-END VERIFY SCRIPT-
```

✅ maflcko closed a pull request: "rpc: show P2(W)SH redeemScript in getrawtransaction #27637"
(https://github.com/bitcoin/bitcoin/pull/27638)

💬 maflcko commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#issuecomment-2366451640)
Closing for now due to inactivity. You can leave a comment below to have this reopened. Also, a new pull request can be opened.

💬 maflcko commented on issue "rpc auth fails 'Error parsing command line arguments: Invalid parameter -rpcpasssword=password":
(https://github.com/bitcoin/bitcoin/issues/30939#issuecomment-2366495985)
What are the exact steps to reproduce. Including the start of the server. Without full and exact steps to reproduce, there is nothing that can be done here.

💬 tsnob commented on pull request "doc: fix broken link to release":
(https://github.com/bitcoin/bitcoin/pull/30945#discussion_r1770495990)
> Shouldn't this pont to `https://bitcoin.org/bin/insecure/bitcoin-core-0.10.0` instead?
>
> Please fix the rest of them as well with a scripted diff:
>
> ```shell
> -BEGIN VERIFY SCRIPT-
> sed -i --regexp-extended 's#(https://bitcoin\.org/bin/)([.0-9]+)#\1insecure/bitcoin-core-\2#g' ./doc/release-notes/*.md
> -END VERIFY SCRIPT-
> ```

Hi @l0rinc, thanks for your suggestion. however I think the directory you provided (https://bitcoin.org/bin/insecure/) only includes limited releases
...