📝 stevensmccoy734-commits opened a pull request: "Add .gitignore file with editor and build patterns"
(https://github.com/bitcoin/bitcoin/pull/33719)
<!--
*** 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
...
(https://github.com/bitcoin/bitcoin/pull/33719)
<!--
*** 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
...
💬 151henry151 commented on pull request "build: Remove CMAKE_SKIP_BUILD_RPATH and SKIP_BUILD_RPATH settings":
(https://github.com/bitcoin/bitcoin/pull/33247#issuecomment-3454098765)
Rebased on latest master and cleaned up commit message/PR description. Guix build in progress - will post hashes once complete.
(https://github.com/bitcoin/bitcoin/pull/33247#issuecomment-3454098765)
Rebased on latest master and cleaned up commit message/PR description. Guix build in progress - will post hashes once complete.
📝 stevensmccoy734-commits opened a pull request: "Delete .github/workflows/ci.yml"
(https://github.com/bitcoin/bitcoin/pull/33720)
<!--
*** 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
...
(https://github.com/bitcoin/bitcoin/pull/33720)
<!--
*** 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
...
📝 stevensmccoy734-commits opened a pull request: "Patch 6"
(https://github.com/bitcoin/bitcoin/pull/33721)
<!--
*** 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
...
(https://github.com/bitcoin/bitcoin/pull/33721)
<!--
*** 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
...
💬 waketraindev commented on pull request "addrman, net: filter during address selection via AddrPolicy to avoid underfill":
(https://github.com/bitcoin/bitcoin/pull/33663#issuecomment-3454896610)
Added AddrPolicy coverage in addrman_tests
(https://github.com/bitcoin/bitcoin/pull/33663#issuecomment-3454896610)
Added AddrPolicy coverage in addrman_tests
💬 maflcko commented on issue "Cannot import descriptors with label and internal:false":
(https://github.com/bitcoin/bitcoin/issues/32376#issuecomment-3455029791)
Can be closed after https://github.com/bitcoin/bitcoin/pull/31514?
(https://github.com/bitcoin/bitcoin/issues/32376#issuecomment-3455029791)
Can be closed after https://github.com/bitcoin/bitcoin/pull/31514?
💬 Sjors commented on issue "RFC: Adding bitcoin-{node,gui} binaries for IPC in 30.0 release":
(https://github.com/bitcoin/bitcoin/issues/31756#issuecomment-3455071260)
🎉
The sidecar application that connects to `bitcoin-node` / `bitcoin -m node` now lives here: https://github.com/sjors/sv2-tp
It can connect to the v30 release.
(https://github.com/bitcoin/bitcoin/issues/31756#issuecomment-3455071260)
🎉
The sidecar application that connects to `bitcoin-node` / `bitcoin -m node` now lives here: https://github.com/sjors/sv2-tp
It can connect to the v30 release.
✅ maflcko closed an issue: "RFC: "Insufficient review" tag for closed PRs"
(https://github.com/bitcoin/bitcoin/issues/29839)
(https://github.com/bitcoin/bitcoin/issues/29839)
💬 maflcko commented on issue "RFC: "Insufficient review" tag for closed PRs":
(https://github.com/bitcoin/bitcoin/issues/29839#issuecomment-3455083740)
Anything left to do here?
(https://github.com/bitcoin/bitcoin/issues/29839#issuecomment-3455083740)
Anything left to do here?
💬 maflcko commented on issue "add ability to remove imported public keys/addresses (or even privkeys)":
(https://github.com/bitcoin/bitcoin/issues/23765#issuecomment-3455089395)
So I guess this can be closed?
(https://github.com/bitcoin/bitcoin/issues/23765#issuecomment-3455089395)
So I guess this can be closed?
✅ Sjors closed an issue: "Stratum v2 via IPC Mining Interface tracking issue"
(https://github.com/bitcoin/bitcoin/issues/31098)
(https://github.com/bitcoin/bitcoin/issues/31098)
💬 Sjors commented on issue "Stratum v2 via IPC Mining Interface tracking issue":
(https://github.com/bitcoin/bitcoin/issues/31098#issuecomment-3455099378)
Yes, I might open a fresh tracking issue later for remaining and new interface changes. There are a few things that we want to adjust, including things that aren't urgent, but should be done when we need a breaking change anyway.
The sidecar application lives at https://github.com/Sjors/sv2-tp
(https://github.com/bitcoin/bitcoin/issues/31098#issuecomment-3455099378)
Yes, I might open a fresh tracking issue later for remaining and new interface changes. There are a few things that we want to adjust, including things that aren't urgent, but should be done when we need a breaking change anyway.
The sidecar application lives at https://github.com/Sjors/sv2-tp
💬 Sammie05 commented on pull request "test: Use same rpc timeout for authproxy and cli":
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455330037)
> > I went through the new changes, built the code, ran the specific test p2p_headers_sync_with_minchainwork.py --timeout-factor=0.2 in both RPC and CLI modes and everything looks good .
>
> It shouldn't look "good". They should both timeout with the same timeout value.
>
> > but why did we decide on half the timeout for CLI instead of using the same value?
>
> The goal of this pull is to use the exact same value, which is also what it does.
> > I went through the new changes, bu
...
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455330037)
> > I went through the new changes, built the code, ran the specific test p2p_headers_sync_with_minchainwork.py --timeout-factor=0.2 in both RPC and CLI modes and everything looks good .
>
> It shouldn't look "good". They should both timeout with the same timeout value.
>
> > but why did we decide on half the timeout for CLI instead of using the same value?
>
> The goal of this pull is to use the exact same value, which is also what it does.
> > I went through the new changes, bu
...
💬 maflcko commented on pull request "test: Use same rpc timeout for authproxy and cli":
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455485696)
> Doesn't the // 2 division mean CLI is actually getting half the RPC timeout instead of the same value?
It is explained in the trailing comment, which refers to https://github.com/bitcoin/bitcoin/blob/66667d6512294fd5dd02161b7c68c19af0865865/test/functional/test_framework/test_node.py#L322
If you have a question about a line of code, it is better to leave the question on the line of code, instead of in the general pull request thread.
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455485696)
> Doesn't the // 2 division mean CLI is actually getting half the RPC timeout instead of the same value?
It is explained in the trailing comment, which refers to https://github.com/bitcoin/bitcoin/blob/66667d6512294fd5dd02161b7c68c19af0865865/test/functional/test_framework/test_node.py#L322
If you have a question about a line of code, it is better to leave the question on the line of code, instead of in the general pull request thread.
👍 willcl-ark approved a pull request: "guix: update time-machine to 5cb84f2013c5b1e48a7d0e617032266f1e6059e2"
(https://github.com/bitcoin/bitcoin/pull/33185#pullrequestreview-3387888534)
ACK 59c4898994bde3d86168075f0031c9d5a9ac5c8f
(https://github.com/bitcoin/bitcoin/pull/33185#pullrequestreview-3387888534)
ACK 59c4898994bde3d86168075f0031c9d5a9ac5c8f
💬 Sammie05 commented on pull request "test: Use same rpc timeout for authproxy and cli":
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455781777)
> > Doesn't the // 2 division mean CLI is actually getting half the RPC timeout instead of the same value?
>
> It is explained in the trailing comment, which refers to
>
> https://github.com/bitcoin/bitcoin/blob/66667d6512294fd5dd02161b7c68c19af0865865/test/functional/test_framework/test_node.py#L322
>
> If you have a question about a line of code, it is better to leave the question on the line of code, instead of in the general pull request thread.
Alrighht.
(https://github.com/bitcoin/bitcoin/pull/33698#issuecomment-3455781777)
> > Doesn't the // 2 division mean CLI is actually getting half the RPC timeout instead of the same value?
>
> It is explained in the trailing comment, which refers to
>
> https://github.com/bitcoin/bitcoin/blob/66667d6512294fd5dd02161b7c68c19af0865865/test/functional/test_framework/test_node.py#L322
>
> If you have a question about a line of code, it is better to leave the question on the line of code, instead of in the general pull request thread.
Alrighht.
💬 Sjors commented on pull request "guix: update time-machine to 5cb84f2013c5b1e48a7d0e617032266f1e6059e2":
(https://github.com/bitcoin/bitcoin/pull/33185#issuecomment-3455784718)
Guix build on x86_64:
```
3de417ac1dade848d8b1609adf72c0faefcaf5acf03199259aa3aeb83e6a863f guix-build-59c4898994bd/output/aarch64-linux-gnu/SHA256SUMS.part
c6eda45fce2b34940ea53caa2acf0c1436122f89b85c0bc727834360f78a40f5 guix-build-59c4898994bd/output/aarch64-linux-gnu/bitcoin-59c4898994bd-aarch64-linux-gnu-debug.tar.gz
41584667134bfc5c35851d5005692c447d3fc529a310c7868030d6383d8bd840 guix-build-59c4898994bd/output/aarch64-linux-gnu/bitcoin-59c4898994bd-aarch64-linux-gnu.tar.gz
5c6d784b
...
(https://github.com/bitcoin/bitcoin/pull/33185#issuecomment-3455784718)
Guix build on x86_64:
```
3de417ac1dade848d8b1609adf72c0faefcaf5acf03199259aa3aeb83e6a863f guix-build-59c4898994bd/output/aarch64-linux-gnu/SHA256SUMS.part
c6eda45fce2b34940ea53caa2acf0c1436122f89b85c0bc727834360f78a40f5 guix-build-59c4898994bd/output/aarch64-linux-gnu/bitcoin-59c4898994bd-aarch64-linux-gnu-debug.tar.gz
41584667134bfc5c35851d5005692c447d3fc529a310c7868030d6383d8bd840 guix-build-59c4898994bd/output/aarch64-linux-gnu/bitcoin-59c4898994bd-aarch64-linux-gnu.tar.gz
5c6d784b
...
🤔 TheCharlatan reviewed a pull request: "kernel: Introduce C header API"
(https://github.com/bitcoin/bitcoin/pull/30595#pullrequestreview-3387750815)
Updated 20be96ee696ba8c3085da0877156e534a58c39bf -> 083814948d66aac49b6995de560a48c7889896cc ([kernelApi_76](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_76) -> [kernelApi_77](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_77), [compare](https://github.com/TheCharlatan/bitcoin/compare/kernelApi_76..kernelApi_77))
* Addressed @stringintech's [comment](https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2455010438), expand comment on purpose of the context options.
* Ad
...
(https://github.com/bitcoin/bitcoin/pull/30595#pullrequestreview-3387750815)
Updated 20be96ee696ba8c3085da0877156e534a58c39bf -> 083814948d66aac49b6995de560a48c7889896cc ([kernelApi_76](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_76) -> [kernelApi_77](https://github.com/TheCharlatan/bitcoin/tree/kernelApi_77), [compare](https://github.com/TheCharlatan/bitcoin/compare/kernelApi_76..kernelApi_77))
* Addressed @stringintech's [comment](https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2455010438), expand comment on purpose of the context options.
* Ad
...
💬 TheCharlatan commented on pull request "kernel: Introduce C header API":
(https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2468823291)
Yes! Must have missed this while re-working the way the secondary object gets passed in.
(https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2468823291)
Yes! Must have missed this while re-working the way the secondary object gets passed in.
💬 TheCharlatan commented on pull request "kernel: Introduce C header API":
(https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2468816664)
We could, yes, but I think it is nice to define the interface outside of the required methods required by the template instantiation when the callbacks get wired to this notification interface. Marking these as virtual we get warnings if we don't override correctly. Alternatively we could also make this a concept, similar to the Log, but that felt a bit too complicated here. I did remove the template arguments though, those seemed needlessly confusing to me.
(https://github.com/bitcoin/bitcoin/pull/30595#discussion_r2468816664)
We could, yes, but I think it is nice to define the interface outside of the required methods required by the template instantiation when the callbacks get wired to this notification interface. Marking these as virtual we get warnings if we don't override correctly. Alternatively we could also make this a concept, similar to the Log, but that felt a bit too complicated here. I did remove the template arguments though, those seemed needlessly confusing to me.