💬 willcl-ark commented on issue "release: ship codesigned MacOS arm64 binaries":
(https://github.com/bitcoin/bitcoin/issues/29749#issuecomment-2511980210)
@edilmedeiros notarization (and codesigning) is being actively worked on by @achow101. It's my understanding (and I could be wrong here) that, if we get the notarization working, we can apply it retroactively to other binaries which would resolve this issue in a preferable way.

💬 furszy commented on pull request "[RFC] descriptors: inference process, do not return unparsable multisig descriptors":
(https://github.com/bitcoin/bitcoin/pull/31404#discussion_r1866150428)
upps, fixed. Thanks

💬 furszy commented on pull request "[RFC] descriptors: inference process, do not return unparsable multisig descriptors":
(https://github.com/bitcoin/bitcoin/pull/31404#discussion_r1866152366)
Sure. Introduced it in the first commit.

✅ danielabrozzoni closed an issue: "Unable to compile for test coverage on Nixos 24.05"
(https://github.com/bitcoin/bitcoin/issues/31087)

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2512046722)
Rebased.

@maflcko comments have been addressed.

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1866162497)
Thanks! [Done](https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2512046722).

💬 hebasto commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1866164093)
Thanks! [Done](https://github.com/bitcoin/bitcoin/pull/31176#issuecomment-2512046722).

However, a few functional tests fail...

💬 maflcko commented on pull request "ci: Skip broken Wine64 tests by default":
(https://github.com/bitcoin/bitcoin/pull/31284#issuecomment-2512064036)
... https://cirrus-ci.com/task/6299029368209408?logs=ci#L4685

💬 l0rinc commented on pull request "util: Improve documentation and negation of args":
(https://github.com/bitcoin/bitcoin/pull/31212#discussion_r1866191667)
optional of optional sounds like a `std::variant` - but I'm fine with doing these in a separate PR, of course

👍 ryanofsky approved a pull request: "test: enable running independent functional test sub-tests"
(https://github.com/bitcoin/bitcoin/pull/30991#pullrequestreview-2473279752)
Code review ACK 409d0d629378c3e23388ed31516376ad1ae536b5. This seems like a good step towards making it easy to run independent tests quickly. I think ideally there would be some naming convention or @ annotation added to test methods that can run independently, so the test framework could provide more functionality like being able to list test methods, being able to show command lines to quickly reproduce problems when tests fails, and calling test methods automatically instead of requiring ind
...

🚀 ryanofsky merged a pull request: "test: enable running independent functional test sub-tests"
(https://github.com/bitcoin/bitcoin/pull/30991)

👍 hebasto approved a pull request: "util: Drop boost posix_time in ParseISO8601DateTime"
(https://github.com/bitcoin/bitcoin/pull/31391#pullrequestreview-2473326256)
ACK faf70cc9941ce2b0ce4fd48ecfdbe28194adb8ba, I have reviewed the code and it looks OK.

💬 ryanofsky commented on pull request "Add multiprocess binaries to release build":
(https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2512207207)
> I disabled ASan + LSan + UBsan again, see previous failure log: https://github.com/bitcoin/bitcoin/actions/runs/11970857809/job/33374462331?pr=30975

I think https://github.com/chaincodelabs/libmultiprocess/pull/121 should fix this, but I want to test the fix a little more and it will require another depends bump like 90b405516f7f3be522ced3e0c4d23b3892df0661 to fix the problem in CI.

If there are more bugs feel free to post to https://github.com/chaincodelabs/libmultiprocess/issues/new or
...

💬 rkrux commented on issue "Add `sat_to_btc()` and conversely `btc_to_sat()` util functions in functional tests":
(https://github.com/bitcoin/bitcoin/issues/31345#issuecomment-2512243262)
@le0nAg Thanks for showing interest in this, there is an open PR for it here: https://github.com/bitcoin/bitcoin/pull/31356
Maybe coordinate with @andremralves and work on the same PR?

💬 theuni commented on pull request "build: Set shared linker flags in toolchain file":
(https://github.com/bitcoin/bitcoin/pull/31395#issuecomment-2512332764)
Post-merge utACK a8e04704f930de168fdddbb1c040dd70fbbe8ff7

👍 danielabrozzoni approved a pull request: "refactor: Move GuessVerificationProgress into ChainstateManager"
(https://github.com/bitcoin/bitcoin/pull/31393#pullrequestreview-2473684721)
light tACK fa5ad4ed2b36fa483385670a210eb8564bc53c9b - i'm not too familiar with the code, but the change is reasonable. Tests are passing locally.

💬 Sjors commented on pull request "Add multiprocess binaries to release build":
(https://github.com/bitcoin/bitcoin/pull/30975#issuecomment-2512564803)
@ryanofsky that one and https://github.com/chaincodelabs/libmultiprocess/issues/122 are the only two I'm aware of at the moment.

📝 mzumsande opened a pull request: "validation: stricter internal handling of invalid blocks"
(https://github.com/bitcoin/bitcoin/pull/31405)
Historically, some fields in validation were set opportunistically by "best effort":
- The `BLOCK_FAILED_CHILD` status (which means that the block index has an invalid predecessor)
- `m_best_header` (the most-work header not known to be invalid).

This means that there were known situations in which these fields were not set when they should've been, or set to wrong values. This was tolerated because the fields were not used for anything consensus-critical and triggering these situations inv
...

💬 maflcko commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1866522517)
IIRC chmod does not work on Windows, so a workaround may be needed for the test failure.

Not sure about the `wallet_migration.py` failure. Another workaround may be needed there.

Also, the `wallet_migration` test won't be running after https://github.com/bitcoin/bitcoin/pull/31248, so the previous releases tests should probably be run as well here.

💬 maflcko commented on pull request "ci: Test cross-built Windows executables on Windows natively":
(https://github.com/bitcoin/bitcoin/pull/31176#discussion_r1866523690)
(Functional tests could be handled in a follow-up, if you want)