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 broken jobs #407

Merged
merged 6 commits into from
Jun 9, 2024
Merged

ci: Fix broken jobs #407

merged 6 commits into from
Jun 9, 2024

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Jun 8, 2024

See commit messages for details.

@hebasto hebasto marked this pull request as draft June 8, 2024 09:12
@hebasto hebasto marked this pull request as ready for review June 8, 2024 11:11
@hebasto hebasto changed the title ci: Disable fuzz tests for "macOS 13 native" job ci: Fix broken jobs Jun 8, 2024
fanquake and others added 4 commits June 8, 2024 20:12
It's come to light that Boost ASIO (a Boost Process sub dep) has in some
instances, been queitly initialising our network stack on Windows (see
PR #28486 and discussion in #28940).

This has been shielding a bug in our own code, but the larger issue
is that Boost Process/ASIO is running code before main, and doing things
like setting up networking. This undermines our own assumptions about
how our binary works, happens before we get to run any sanity checks,
and also runs before we call our own code to setup networking.

It's also not clear why a feature like external signer would have a
dependency that would be doing anything network/socket related, given it
only exists to spawn a local process.
Can be reverted back after syncing the QML repository with the main one.
Fuzz tests cause timeout for reasons unrelated to the QML GUI
development.

Can be reverted back after syncing the QML repository with the main one.
Copy link
Contributor

@pablomartin4btc pablomartin4btc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d04cfc9

@hebasto hebasto merged commit c065a17 into bitcoin-core:main Jun 9, 2024
9 checks passed
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.

3 participants