💬 maflcko commented on pull request "test: Add test case for spending bare multisig":
(https://github.com/bitcoin/bitcoin/pull/29120#issuecomment-2074251700)
Are you still working on this?

💬 laanwj commented on pull request "cleanse: switch to SecureZeroMemory for Windows cross-compile, check for usage":
(https://github.com/bitcoin/bitcoin/pull/26950#issuecomment-2074260637)
Concept ACK

💬 laanwj commented on pull request "cleanse: switch to SecureZeroMemory for Windows cross-compile, check for usage":
(https://github.com/bitcoin/bitcoin/pull/26950#discussion_r1577416299)
This checks that the function is there, not whether it is used, in the right places. But it may be enough as a start.

💬 vasild commented on pull request "Broadcast own transactions only via short-lived Tor or I2P connections":
(https://github.com/bitcoin/bitcoin/pull/29415#issuecomment-2074267427)
`cc760207b8...ea1ca3715e`: adjust `feature_config_args.py` after forbidding `-walletbroadcast` when `-privatebroadcast` is enabled.

⚠️ KonradStaniec opened an issue: "Calling `walletprocesspsbt` to sign multisig containing `OP_GREATERTHANOREQUAL` op_code"
(https://github.com/bitcoin/bitcoin/issues/29949)
### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current behaviour

I want to create signature by calling `walletprocesspsbt` endpoint to spend from multisig output. It is worth stating that this is taproot output with one leaf.

Wallet contains one of the key from multisig. Key was created by first calling `getnewaddress` and then calling `getaddressinfo` and retrieving public key.

Now the issue is that if multisig script is in format:
`<key1> OP_C
...

💬 laanwj commented on pull request "contrib: list other binaries in manpage output":
(https://github.com/bitcoin/bitcoin/pull/29585#discussion_r1577434913)
Could use `os.path.basename` it's slightly more readable and will always pick the last path item.

💬 laanwj commented on pull request "contrib: list other binaries in manpage output":
(https://github.com/bitcoin/bitcoin/pull/29585#issuecomment-2074293842)
Good idea, concept ACK.

👍 laanwj approved a pull request: "contrib: list other binaries in manpage output"
(https://github.com/bitcoin/bitcoin/pull/29585#pullrequestreview-2019127833)
Code review ACK 7c3ac598dd9a1f1a506c4931249ff6c9f1c949ba

💬 laanwj commented on pull request "build: add `-Wundef`":
(https://github.com/bitcoin/bitcoin/pull/29876#discussion_r1577456975)
i still think it'd be better, but probably not worth doing this last-minute change for autotools.
Don't know how the CMake port handles these kind of checks anyway, but might consider it there.

👍 laanwj approved a pull request: "netbase: clean up Proxy logging"
(https://github.com/bitcoin/bitcoin/pull/29882#pullrequestreview-2019199241)
ACK fb4cc5f423ce587c1e97377e8afdf92fb4850f59

💬 hebasto commented on pull request "doc: Suggest only necessary Qt packages for installation on OpenBSD":
(https://github.com/bitcoin/bitcoin/pull/29947#issuecomment-2074363052)
> > Verified on a fresh install of OpenBSD 7.5 that these two packages are the minimum requirement to build the GUI:
>
> Please update the version number in line 3 of the doc as well. I don't know if there is a difference between 7.4 and 7.5, but it can't hurt to be accurate here.

Sure. Updated.

💬 laanwj commented on pull request "Remove redundant `-datacarrier` option":
(https://github.com/bitcoin/bitcoin/pull/29942#issuecomment-2074372166)
This particular option was introduced 10 years ago (in e44fea55ea73f46bc9460597c7001e77acb58db7). FWIW i think we would need a stronger reason to remove it than a bit of redundancy.

💬 maflcko commented on pull request "test: Run framework unit tests in parallel":
(https://github.com/bitcoin/bitcoin/pull/29771#issuecomment-2074415872)
ACK f19f0a2e5af6c2a64900f1f229e21b6f1668bd3d 🌽

<details><summary>Show signature</summary>

Signature:

```
untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
trusted comment: ACK f19f0a2e5af6c2a64900f1f229
...

💬 TheCharlatan commented on pull request "refactor: Rename `subprocess.hpp` to follow our header name conventions":
(https://github.com/bitcoin/bitcoin/pull/29910#issuecomment-2074457183)
> Resolved linter warnings.

Can you mention this in the PR description?

💬 Kellydutch1 commented on pull request "doc: Suggest only necessary Qt packages for installation on OpenBSD":
(https://github.com/bitcoin/bitcoin/pull/29947#issuecomment-2074462911)
I have bitcoin hacking software

💬 fanquake commented on pull request "build: Consider `SOURCE_DATE_EPOCH` in Guix environment only":
(https://github.com/bitcoin/bitcoin/pull/29761#issuecomment-2074466635)
What is the status of this?

📝 fanquake converted_to_draft a pull request: "doc: add LLVM instruction for macOS < 13"
(https://github.com/bitcoin/bitcoin/pull/29934)
#29208 bumped clang to 14, which users of old macOS versions need to install manually. This PR adds instructions.

Xcode 14.3.1 ships clang 14.0.3 (14.0.0 is broken, see #29918):
https://en.wikipedia.org/wiki/Xcode#Xcode_11.0_-_14.x_(since_SwiftUI_framework)

The system requirements for that is macOS Ventura 13.0 or later: https://developer.apple.com/documentation/xcode-release-notes/xcode-14_3_1-release-notes#

Homebrew itself officially supports macOS 12 or later, but _may_ still work o
...

⚠️ mouhand-a opened an issue: "0xF2852f0DD3703afc3A5B4343e262839eEf73eD23"
(https://github.com/bitcoin/bitcoin/issues/29950)
### Issues, reports or feature requests related to the GUI should be opened directly on the GUI repo

- [X] I still think this issue should be opened here

### Report

My address [](url)
0xF2852f0DD3703afc3A5B4343e262839eEf73eD23

✅ hebasto closed an issue: "0xF2852f0DD3703afc3A5B4343e262839eEf73eD23"
(https://github.com/bitcoin/bitcoin/issues/29950)

:lock: hebasto locked an issue: "0xF2852f0DD3703afc3A5B4343e262839eEf73eD23"
(https://github.com/bitcoin/bitcoin/issues/29950)