📝 JaroslawDomagala opened a pull request: "Create masteron"
(https://github.com/bitcoin/bitcoin/pull/33495)
<!--
*** 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 improv
...

💬 JaroslawDomagala commented on pull request "Create masteron":
(https://github.com/bitcoin/bitcoin/pull/33495#issuecomment-3347001048)
22

👍 vasild approved a pull request: "Introduce SockMan ("lite"): low-level socket handling for HTTP"
(https://github.com/bitcoin/bitcoin/pull/32747#pullrequestreview-3279884844)
ACK e007e1b57d5d42c2a8d932d5b91eec8a3ca76e14

🤔 janb84 reviewed a pull request: "build: Move CMAKE_SKIP_INSTALL_RPATH from CMake to Guix script"
(https://github.com/bitcoin/bitcoin/pull/33470#pullrequestreview-3279929540)
ACK dd5c517757f97b68f7eb07628222c958b47f742b

PR removes CMAKE flag from CMakeLists to config flag section of build script. This change is a follow up on #33247 and seems like a good followup to move the Guix special flag to the Guix special build script.


**Commit:** `dd5c517757f9`

```shell
207120a771188474820e1a98f20780154f53037f12251d08f49582754819ab50 guix-build-dd5c517757f9/output/aarch64-linux-gnu/SHA256SUMS.part
7fd9043d0fcb6f58f3401968f83e47c54f94bd7fd5880951db1918f027ed85
...

💬 vasild commented on pull request "depends: Update URL for `qrencode` package source tarball":
(https://github.com/bitcoin/bitcoin/pull/33494#issuecomment-3347129804)
For me, the checksum is the same as in the PR:
```sh
$ wget -q -O - https://github.com/fukuchi/libqrencode/archive/refs/tags/v4.1.1.tar.gz |sha256
5385bc1b8c2f20f3b91d258bf8ccc8cf62023935df2d2676b5b67049f31a049c
$
```

📝 MirekLabada opened a pull request: "222 "
(https://github.com/bitcoin/bitcoin/pull/33496)
<!--
*** 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 improv
...

💬 MirekLabada commented on pull request "222":
(https://github.com/bitcoin/bitcoin/pull/33496#issuecomment-3347144943)
? fix this please

📝 MirekLabada opened a pull request: "?"
(https://github.com/bitcoin/bitcoin/pull/33497)
<!--
*** 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 improv
...

💬 ryanofsky commented on pull request "multiprocess: Add capnp wrapper for Chain interface":
(https://github.com/bitcoin/bitcoin/pull/29409#issuecomment-3347241863)
> Err, right. Updated the description with your suggested trace filtering.

Thanks created https://github.com/bitcoin-core/libmultiprocess/issues/219 to track this (feel free to create new issues directly in the future). Thanks for updating the rust client, too!

💬 hebasto commented on pull request "build: Move CMAKE_SKIP_INSTALL_RPATH from CMake to Guix script":
(https://github.com/bitcoin/bitcoin/pull/33470#discussion_r2388170464)
> Can this just be `CMAKE_SKIP_RPATH`?

With this suggestion having been adopted, the following lines of code can be dropped as well: https://github.com/bitcoin/bitcoin/blob/dd5c517757f97b68f7eb07628222c958b47f742b/CMakeLists.txt#L632 and https://github.com/bitcoin/bitcoin/blob/d8fe258cd6105704bf4427eda048dd7085ca516d/src/CMakeLists.txt#L420

I understand that this overlaps with https://github.com/bitcoin/bitcoin/pull/33247, so it's OK to leave this branch as is.

👍 hebasto approved a pull request: "build: Move CMAKE_SKIP_INSTALL_RPATH from CMake to Guix script"
(https://github.com/bitcoin/bitcoin/pull/33470#pullrequestreview-3280122695)
ACK dd5c517757f97b68f7eb07628222c958b47f742b, I have reviewed the code and it looks OK.

💬 hebasto commented on pull request "depends: Update URL for `qrencode` package source tarball":
(https://github.com/bitcoin/bitcoin/pull/33494#issuecomment-3347322127)
> For me, the checksum is the same as in the PR:
>
> ```shell
> $ wget -q -O - https://github.com/fukuchi/libqrencode/archive/refs/tags/v4.1.1.tar.gz |sha256
> 5385bc1b8c2f20f3b91d258bf8ccc8cf62023935df2d2676b5b67049f31a049c
> $
> ```
>
> but CI:
>
> ```
> sha256sum: WARNING: 1 computed checksum did NOT match
> /home/admin/actions-runner/_work/_temp/depends/sources/qrencode-4.1.1.tar.gz: FAILED
> ```

I believe it's a cache issue.

@willcl-ark @m3dwards @maflcko

Can the d
...

💬 maflcko commented on pull request "depends: Update URL for `qrencode` package source tarball":
(https://github.com/bitcoin/bitcoin/pull/33494#issuecomment-3347363844)
> Can the dependency CI cache be invalidated: (1) for this PR; (2) globally?

I wouldn't know how, other than by changing the cache key value (both for restore and save): https://github.com/bitcoin/bitcoin/blob/d8fe258cd6105704bf4427eda048dd7085ca516d/.github/actions/restore-caches/action.yml#L21

💬 hebasto commented on pull request "depends: Update URL for `qrencode` package source tarball":
(https://github.com/bitcoin/bitcoin/pull/33494#issuecomment-3347417118)
> I believe it's a cache issue.

Resolved by using the new `$(package)_file_name`.

💬 m3dwards commented on pull request "depends: Update URL for `qrencode` package source tarball":
(https://github.com/bitcoin/bitcoin/pull/33494#issuecomment-3347455469)
> Can the `depends` CI cache be invalidated: (1) for this PR; (2) globally?

I would have thought the cache should have been invalidated automatically, looking into it.

💬 kevkevinpal commented on pull request "ci: check if file or directory already exists for ${HOME}/.bitcoin instead of failing":
(https://github.com/bitcoin/bitcoin/pull/33486#issuecomment-3347489891)
> I don't think it is supported or even encouraged to run this internal CI script on an outside machine.

Yea I understand this, I was just playing around with the scripts to understand better how the CI works

> Also, I fail to see how this could even work, as you did not include any steps how to reproduce.

Yes, I updated the description. Basically, just create a directory in `$HOME/.bitcoin` and then try to run `./ci/test/03_test_script.sh` and you should see the error.

For me it is
...

💬 kevkevinpal commented on pull request "ci: check if file or directory already exists for ${HOME}/.bitcoin instead of failing":
(https://github.com/bitcoin/bitcoin/pull/33486#issuecomment-3347493339)
closing as it doesn't fix any currently existing issue

✅ kevkevinpal closed a pull request: "ci: check if file or directory already exists for ${HOME}/.bitcoin instead of failing"
(https://github.com/bitcoin/bitcoin/pull/33486)

💬 plebhash commented on issue "`bitcoin-node` is unkillable after mining IPC connection is established":
(https://github.com/bitcoin/bitcoin/issues/33463#issuecomment-3347520481)
@ryanofsky I don't really used `gdb` nor `lldb` on a regular basis, so I apologize if I did anything wrong. I hope the logs below are giving you the right info.

Here's the sequence of steps I took:
- do `lldb -- ./build/bin/bitcoin-node -testnet4 -ipcbind=unix`
- inside `lldb` terminal, do `process handle -p true -s false -n false SIGINT` so that the `SIGINT` goes to the process and not to `lldb`
- inside `lldb` terminal, do `run`
- launch my (now outdated) Rust code that does `waitTipChanged`

...

👍 stickies-v approved a pull request: "ci: Drop support for EOL macOS 13"
(https://github.com/bitcoin/bitcoin/pull/33489#pullrequestreview-3280376156)
ACK fab9ddaf554837624fa8f388e046a30d5bf7626f

macOS 13 is EOL, and the minimum Xcode version that ships the documented minimum clang-16 is Xcode 16.