💬 laanwj commented on pull request "net: Replace ifname check with IFF_LOOPBACK in Discover":
(https://github.com/bitcoin/bitcoin/pull/29984#issuecomment-2100006946)
Thanks for checking! it's kind of a belt-and-suspenders check: we only look for routable addresses in Discover, so unless someone assigned a routable address to the loopback interface-which would be really strange-it doesn't actually influence the result.

🚀 fanquake merged a pull request: "test: added test coverage to loadtxoutset could not open file"
(https://github.com/bitcoin/bitcoin/pull/30053)

💬 emsit commented on pull request "Testnet4 including PoW difficulty adjustment fix":
(https://github.com/bitcoin/bitcoin/pull/29775#issuecomment-2100018346)
Is there a plan to replace the path '/testnet3' with '/testnet'? Due to backward compatibility, that might not be a good idea.

https://github.com/fjahr/bitcoin/commit/37e202adc1c734740104c6a5e1d2bd021102764e#diff-583f605f372805d0bc87ebfc893f835b3f66746f560611e682bd917de17586d2
https://github.com/fjahr/bitcoin/commit/37e202adc1c734740104c6a5e1d2bd021102764e#diff-723b9eeeb819047ef03f827ee8a3ccfef48ce7ef6a340a068a68b551993fe70b

💬 paplorinc commented on pull request "test: use assert_greater_than util":
(https://github.com/bitcoin/bitcoin/pull/30019#discussion_r1593622473)
`idx` can equal `MAX_NODES` according to the code

💬 TheCharlatan commented on pull request "Encapsulate warnings in generalized node::Warnings and remove globals":
(https://github.com/bitcoin/bitcoin/pull/30058#issuecomment-2100025531)
Concept ACK on removing the warnings globals.

💬 josibake commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593657407)
Oops! Should have ran it myself before including it. I have a slight preference for keeping the sage code, since its nice to be able to run that snippet and get the same value as a way of verifying. That being said, "be able to run that snippet" implies keeping the comment up to date, but I doubt that will be much (if any) of a maintenance burden.

Also think its more correct to output the x,y coordinate pair since in the comment we are referring to `NUMS_H` as a point, whereas the x-only valu
...

💬 josibake commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593668308)
Updated to fix the hex decoding. Went ahead and left the (x,y) print out, but can drop the y-value or drop the whole sage snippet if others feel strongly.

✅ maflcko closed a pull request: "build: Require libc++-16 or later"
(https://github.com/bitcoin/bitcoin/pull/29077)

📝 maflcko opened a pull request: "ci: Roll clang in test-each-commit task"
(https://github.com/bitcoin/bitcoin/pull/30060)
Needed for https://github.com/bitcoin/bitcoin/pull/29077#issuecomment-2099704210

💬 theStack commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593731959)
Agree that keeping the sage code and printing out the full point makes sense w.r.t. your point vs byte encoding argument. (By the way, at least in our test-framework implementation of GE, [`lift_x` always produces a point with even y coordinate](https://github.com/bitcoin/bitcoin/blob/43a66c55ec8770cf7c21112aac9b997f3f2fb704/test/functional/test_framework/crypto/secp256k1.py#L256), but no idea if this is also the case in the GE of sage.)

👍 theStack approved a pull request: "crypto: add `NUMS_H` const"
(https://github.com/bitcoin/bitcoin/pull/30048#pullrequestreview-2045153103)
Code-review ACK a5867a393cacbe4b2e0de51b842362754e98ab42

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593737753)
nit: It is already a `Vec`, so no need to iter+collect, no?

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593739133)
nit: Can also use `.args([...])` for less code?

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593737106)
nit: Could remove this, as the URL is below as well.

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593736433)
check_output is a helper that assumes the command exists, so I don't think it will work here, no?

Ref:

```
thread 'main' panicked at src/main.rs:23:28:
command error: Os { code: 2, kind: NotFound, message: "No such file or directory" }
stack backtrace:
0: rust_begin_unwind
...
```

I think in a previous version you had a check for `NotFound` directly, so you could restore that?

💬 laanwj commented on pull request "guix: build with glibc 2.31":
(https://github.com/bitcoin/bitcoin/pull/29987#issuecomment-2100177952)
New used symbols since 2.28:

81: 0000000000000000 0 FUNC GLOBAL DEFAULT UND fcntl64@GLIBC_2.28 (15)

`fcntl` function that can handle large file offsets on 32 bit platforms. Used in `unixFileLock`. `unixShmLock` and other posix/unix file functions.

195: 0000000000000000 0 FUNC GLOBAL DEFAULT UND statx@GLIBC_2.28 (15)

Extended file stat. Used in `QFileSystemEngine::fillMetaData`.

533: 0000000000000000 0 FUNC GLOBAL DEFAULT UND r
...

💬 willcl-ark commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593777495)
Reverted in 9d2a8a8b1d67af14e4a5fc8276a1069b6ef9fe4a

💬 willcl-ark commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593777653)
Removed in 9d2a8a8b1d67af14e4a5fc8276a1069b6ef9fe4a

💬 josibake commented on pull request "crypto, refactor: add method for applying the taptweak":
(https://github.com/bitcoin/bitcoin/pull/30051#issuecomment-2100225575)
Updated to use a utility function (instead of a method on `CKey`) per the @theuni 's feedback

💬 maflcko commented on issue "Memory leak with `rest/block` REST endpoint and `getblock` RPC when verbosity >=2":
(https://github.com/bitcoin/bitcoin/issues/30052#issuecomment-2100234965)
Did the same check with

```
valgrind --tool=massif ./src/bitcoind &
while sleep 3; do ./src/bitcoin-cli getblock 00000000000000000002a7c4c1e48d76c5a37902165a270156b7a8d72728a054 2 ; done

...

massif-visualizer ./massif.out.1234
```