Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI fix #35

Closed
wants to merge 28 commits into from
Closed

CI fix #35

wants to merge 28 commits into from

Conversation

Sjors
Copy link
Owner

@Sjors Sjors commented Feb 15, 2024

Obviously bumping macOS CI is not actually a fix.

Sjors and others added 27 commits February 15, 2024 11:05
Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
The proposed Stratum v2 integration introduces a new Transport subclass. This commit makes it easier in the future to move this integration to its own process.

net.cpp is currently part of libbitcoin_node. Avoid a dependency on the node by moving the Transport abstract class to common.
This helps reduce the size and scope net.h, as well as to make v1 and v2 transport usable outside libbitcoin_node. This is useful for utilities that need a one-shot p2p connection, e.g. to broadcast a transaction or fetch a specific block.

This commit is not necessary for Stratum v2 process separation.
Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
Implemented starting from a copy of V2Transport and the V2TransportTester, modifying it to fit Stratum v2 and Noise Protocol requirements.

Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
Co-Authored-By: Fi3
Co-authored-by: Vasil Dimov <vd@FreeBSD.org>
Co-authored-by: Christopher Coverdale <chris.coverdale24@gmail.com>
Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
The template provider will listen for a Job Declarator client.
It can establish a connection and detect various protocol errors.

Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
Co-Authored-By: Fi3
Co-Authored-By: Christopher Coverdale <chris.coverdale24@gmail.com>
When this script is used as part of a Github Action, there is no master branch, only origin/master.
Co-authored-by: MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz>
@Sjors
Copy link
Owner Author

Sjors commented Feb 15, 2024

The extra delay in c8d10af does seem to help. Trying that with macOS 13 CI now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant