Skip to content

Conversation

@theStack
Copy link
Contributor

With OpenBSD 7.3, the waitid(2) system call is implemented (see openbsd/src@8112871, first mentioned kernel improvement at https://www.openbsd.org/73.html).

This means Boost.Process finally doesn't fail to compile anymore and we can remove the build hint about missing external signer support. Tested on my amd64 machine by reconfiguring / rebuilding master branch and successfully running the functional test wallet_signer.py. ✔️

…able)

With OpenBSD 7.3, the waitid(2) system call is implemented (see
openbsd/src@8112871).

This means Boost.Process finally doesn't fail to compile anymore and we
can remove the build hint about missing external signer support. Tested
on my amd64 machine by reconfiguring / rebuilding master branch and
successfully running the functional test wallet_signer.py.
@DrahtBot
Copy link
Contributor

DrahtBot commented Apr 11, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@DrahtBot DrahtBot added the Docs label Apr 11, 2023
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK 6b17994 - haven't tested, but looks good to me.

@fanquake fanquake merged commit 7f4ab67 into bitcoin:master Apr 12, 2023
@theStack theStack deleted the doc-update_openbsd_build_for_73 branch April 12, 2023 15:19
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 12, 2023
@bitcoin bitcoin locked and limited conversation to collaborators Apr 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants